qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 10/50] dino: return PCIBus from dino_init() using qdev_get


From: Richard Henderson
Subject: Re: [PATCH v2 10/50] dino: return PCIBus from dino_init() using qdev_get_child_bus()
Date: Wed, 4 May 2022 11:56:47 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.0

On 5/4/22 02:25, Mark Cave-Ayland wrote:
This allows access to the PCI bus without having to reference parent_obj 
directly.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Acked-by: Helge Deller <deller@gmx.de>
---
  hw/hppa/dino.c | 7 +++----
  1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/hw/hppa/dino.c b/hw/hppa/dino.c
index 77b8089664..d89b48493f 100644
--- a/hw/hppa/dino.c
+++ b/hw/hppa/dino.c
@@ -523,14 +523,12 @@ PCIBus *dino_init(MemoryRegion *addr_space,
  {
      DeviceState *dev;
      DinoState *s;
-    PCIBus *b;
+    PCIBus *pci_bus;
dev = qdev_new(TYPE_DINO_PCI_HOST_BRIDGE);
      object_property_set_link(OBJECT(dev), "memory-as", OBJECT(addr_space),
                               &error_fatal);
      s = DINO_PCI_HOST_BRIDGE(dev);
-
-    b = s->parent_obj.bus;
      sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal);
memory_region_add_subregion(addr_space, DINO_HPA,
@@ -539,7 +537,8 @@ PCIBus *dino_init(MemoryRegion *addr_space,
      *p_rtc_irq = qemu_allocate_irq(dino_set_timer_irq, s, 0);
      *p_ser_irq = qemu_allocate_irq(dino_set_serial_irq, s, 0);
- return b;
+    pci_bus = PCI_BUS(qdev_get_child_bus(dev, "pci"));
+    return pci_bus;
  }
static void dino_pcihost_reset(DeviceState *dev)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~



reply via email to

[Prev in Thread] Current Thread [Next in Thread]