Hi, This series is the result of a long thread with Peter: https://www.mail-archive.com/qemu-devel@nongnu.org/msg788366.html and IRC chats...
AddressSpace are physical address view and shouldn't be using non-zero base address. The correct way to map a MR used as AS root is to use a MR alias. Fix the current incorrect uses, then forbid further use. Since v1: - Split the Raven patch in multiple changes, easier to follow/review (https://www.mail-archive.com/qemu-devel@nongnu.org/msg791116.html) Note, the Aspeed patches are already queued in Cédric tree. I had to cherry-pick them from his tree to have the series pass CI. Cédric Le Goater (1): hw/aspeed/smc: Use the RAM memory region for DMAs Peter Xu (1): memory: Make sure root MR won't be added as subregion Philippe Mathieu-Daudé (9): hw/arm/aspeed: Do not directly map ram container onto main address bus hw/arm/aspeed: Do not sysbus-map mmio flash region directly, use alias hw/pci-host: Rename Raven ASIC PCI bridge as raven.c hw/pci-host/raven: Add PCI_IO_BASE_ADDR definition hw/pci-host/raven: Assert PCI I/O AddressSpace is based at 0x80000000 hw/pci-host/raven: Use MR alias for AS root, not sysbus mapped MR hw/pci-host/raven: Remove pointless alias mapping onto system bus hw/pci-host/prep: Do not directly map bus-master region onto main bus hw/pci-host/raven: Remove temporary assertion 'root MR is zero-based' include/exec/memory.h | 1 + include/hw/ssi/aspeed_smc.h | 1 + hw/arm/aspeed.c | 8 ++++++-- hw/pci-host/{prep.c => raven.c} | 19 ++++++++++--------- hw/ssi/aspeed_smc.c | 10 +++++----- softmmu/memory.c | 2 ++ MAINTAINERS | 2 +- hw/pci-host/Kconfig | 2 +- hw/pci-host/meson.build | 2 +- hw/ppc/Kconfig | 2 +- 10 files changed, 29 insertions(+), 20 deletions(-) rename hw/pci-host/{prep.c => raven.c} (96%) -- 2.26.3