On 04/18/2017 12:59 AM, Eduardo Habkost wrote:
The function requires a PCI_HOST_BRIDGE object, so change the parameter type to reflect that.Cc: "Michael S. Tsirkin" <m...@redhat.com> Cc: Marcel Apfelbaum <mar...@redhat.com> Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> --- hw/pci/pci.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index 259483b1c0..25118fb91d 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c @@ -312,11 +312,9 @@ static void pcibus_reset(BusState *qbus) } } -static void pci_host_bus_register(DeviceState *host) +static void pci_host_bus_register(PCIHostState *phb) { - PCIHostState *host_bridge = PCI_HOST_BRIDGE(host); - - QLIST_INSERT_HEAD(&pci_host_bridges, host_bridge, next); + QLIST_INSERT_HEAD(&pci_host_bridges, phb, next); } PCIBus *pci_find_primary_bus(void) @@ -377,7 +375,7 @@ static void pci_bus_init(PCIBus *bus, DeviceState *parent, /* host bridge */ QLIST_INIT(&bus->child); - pci_host_bus_register(parent); + pci_host_bus_register(PCI_HOST_BRIDGE(host)); } bool pci_bus_is_express(PCIBus *bus)
Reviewed-by: Marcel Apfelbaum <mar...@redhat.com> Thanks, Marcel
