[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 6/6] sabre: don't call sysbus_mmio_map() in sabre_realize()
From: |
Mark Cave-Ayland |
Subject: |
[PATCH v2 6/6] sabre: don't call sysbus_mmio_map() in sabre_realize() |
Date: |
Sat, 26 Sep 2020 15:02:16 +0100 |
The device should not map itself but instead should be mapped to sysbus by the
sun4u machine.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
hw/pci-host/sabre.c | 8 --------
hw/sparc64/sun4u.c | 7 +++++++
2 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/hw/pci-host/sabre.c b/hw/pci-host/sabre.c
index 5ac6283623..5394ad5cd0 100644
--- a/hw/pci-host/sabre.c
+++ b/hw/pci-host/sabre.c
@@ -378,16 +378,8 @@ static void sabre_realize(DeviceState *dev, Error **errp)
{
SabreState *s = SABRE(dev);
PCIHostState *phb = PCI_HOST_BRIDGE(dev);
- SysBusDevice *sbd = SYS_BUS_DEVICE(s);
PCIDevice *pci_dev;
- /* sabre_config */
- sysbus_mmio_map(sbd, 0, s->special_base);
- /* PCI configuration space */
- sysbus_mmio_map(sbd, 1, s->special_base + 0x1000000ULL);
- /* pci_ioport */
- sysbus_mmio_map(sbd, 2, s->special_base + 0x2000000ULL);
-
memory_region_init(&s->pci_mmio, OBJECT(s), "pci-mmio", 0x100000000ULL);
memory_region_add_subregion(get_system_memory(), s->mem_base,
&s->pci_mmio);
diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
index ad5ca2472a..ebb3b89250 100644
--- a/hw/sparc64/sun4u.c
+++ b/hw/sparc64/sun4u.c
@@ -588,6 +588,13 @@ static void sun4uv_init(MemoryRegion *address_space_mem,
&error_abort);
sysbus_realize_and_unref(SYS_BUS_DEVICE(sabre), &error_fatal);
+ /* sabre_config */
+ sysbus_mmio_map(SYS_BUS_DEVICE(sabre), 0, PBM_SPECIAL_BASE);
+ /* PCI configuration space */
+ sysbus_mmio_map(SYS_BUS_DEVICE(sabre), 1, PBM_SPECIAL_BASE + 0x1000000ULL);
+ /* pci_ioport */
+ sysbus_mmio_map(SYS_BUS_DEVICE(sabre), 2, PBM_SPECIAL_BASE + 0x2000000ULL);
+
/* Wire up PCI interrupts to CPU */
for (i = 0; i < IVEC_MAX; i++) {
qdev_connect_gpio_out_named(DEVICE(sabre), "ivec-irq", i,
--
2.20.1
- [PATCH v2 0/6] QOM minor fixes, Mark Cave-Ayland, 2020/09/26
- [PATCH v2 2/6] sparc32-ledma: use object_initialize_child() for lance child object, Mark Cave-Ayland, 2020/09/26
- [PATCH v2 3/6] sparc32-espdma: use object_initialize_child() for esp child object, Mark Cave-Ayland, 2020/09/26
- [PATCH v2 1/6] sparc32-dma: use object_initialize_child() for espdma and ledma child objects, Mark Cave-Ayland, 2020/09/26
- [PATCH v2 4/6] sparc32-ledma: don't reference nd_table directly within the device, Mark Cave-Ayland, 2020/09/26
- [PATCH v2 5/6] macio: don't reference serial_hd() directly within the device, Mark Cave-Ayland, 2020/09/26
- [PATCH v2 6/6] sabre: don't call sysbus_mmio_map() in sabre_realize(),
Mark Cave-Ayland <=