pc: ich9 lpc: make it work with global/compat properties
Propeties of object should be available after its instances_init() callback is finished and not added in PCIDeviceClass.init which is roughly corresponds to realize() method. Moving properties adding into instances_init will fix missing property error when global/compat property mechanism is used. Signed-off-by: Igor Mammedov <imammedo@redhat.com> Acked-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
parent
34774320c3
commit
d6b38b6611
@ -563,7 +563,14 @@ static void ich9_lpc_add_properties(ICH9LPCState *lpc)
|
||||
ich9_pm_add_properties(OBJECT(lpc), &lpc->pm, NULL);
|
||||
}
|
||||
|
||||
static int ich9_lpc_initfn(PCIDevice *d)
|
||||
static void ich9_lpc_initfn(Object *obj)
|
||||
{
|
||||
ICH9LPCState *lpc = ICH9_LPC_DEVICE(obj);
|
||||
|
||||
ich9_lpc_add_properties(lpc);
|
||||
}
|
||||
|
||||
static int ich9_lpc_init(PCIDevice *d)
|
||||
{
|
||||
ICH9LPCState *lpc = ICH9_LPC_DEVICE(d);
|
||||
ISABus *isa_bus;
|
||||
@ -589,9 +596,6 @@ static int ich9_lpc_initfn(PCIDevice *d)
|
||||
memory_region_add_subregion_overlap(pci_address_space_io(d),
|
||||
ICH9_RST_CNT_IOPORT, &lpc->rst_cnt_mem,
|
||||
1);
|
||||
|
||||
ich9_lpc_add_properties(lpc);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -641,7 +645,7 @@ static void ich9_lpc_class_init(ObjectClass *klass, void *data)
|
||||
|
||||
set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
|
||||
dc->reset = ich9_lpc_reset;
|
||||
k->init = ich9_lpc_initfn;
|
||||
k->init = ich9_lpc_init;
|
||||
dc->vmsd = &vmstate_ich9_lpc;
|
||||
k->config_write = ich9_lpc_config_write;
|
||||
dc->desc = "ICH9 LPC bridge";
|
||||
@ -660,6 +664,7 @@ static const TypeInfo ich9_lpc_info = {
|
||||
.name = TYPE_ICH9_LPC_DEVICE,
|
||||
.parent = TYPE_PCI_DEVICE,
|
||||
.instance_size = sizeof(struct ICH9LPCState),
|
||||
.instance_init = ich9_lpc_initfn,
|
||||
.class_init = ich9_lpc_class_init,
|
||||
};
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user