mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-04 00:03:54 -06:00
hw/ppc/e500: Use SysBusDevice API to access TYPE_CCSR's internal resources
Rather than accessing the attributes of TYPE_CCSR directly, use the SysBusDevice API which exists exactly for that purpose. Furthermore, registering the memory region with the SysBusDevice API makes it show up in QMP's `info qom-tree` command. Signed-off-by: Bernhard Beschow <shentey@gmail.com> Reviewed-by: BALATON Zoltan <balaton@eik.bme.hu> [balaton: rebased] Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu> Acked-by: Bernhard Beschow <shentey@gmail.com> Message-ID: <619a58d1f83d2aad5b4feec930d46c64abff0977.1748012109.git.balaton@eik.bme.hu> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
This commit is contained in:
parent
3c32fa9fb2
commit
a35391ba87
2 changed files with 8 additions and 8 deletions
|
@ -16,7 +16,6 @@
|
|||
|
||||
#include "qemu/osdep.h"
|
||||
#include "hw/irq.h"
|
||||
#include "hw/ppc/e500-ccsr.h"
|
||||
#include "hw/qdev-properties.h"
|
||||
#include "migration/vmstate.h"
|
||||
#include "hw/pci/pci_device.h"
|
||||
|
@ -418,11 +417,12 @@ static const VMStateDescription vmstate_ppce500_pci = {
|
|||
static void e500_pcihost_bridge_realize(PCIDevice *d, Error **errp)
|
||||
{
|
||||
PPCE500PCIBridgeState *b = PPC_E500_PCI_BRIDGE(d);
|
||||
PPCE500CCSRState *ccsr = CCSR(
|
||||
SysBusDevice *ccsr = SYS_BUS_DEVICE(
|
||||
object_resolve_path_component(qdev_get_machine(), "e500-ccsr"));
|
||||
MemoryRegion *ccsr_space = sysbus_mmio_get_region(ccsr, 0);
|
||||
|
||||
memory_region_init_alias(&b->bar0, OBJECT(ccsr), "e500-pci-bar0", &ccsr->ccsr_space,
|
||||
0, int128_get64(ccsr->ccsr_space.size));
|
||||
memory_region_init_alias(&b->bar0, OBJECT(ccsr), "e500-pci-bar0",
|
||||
ccsr_space, 0, int128_get64(ccsr_space->size));
|
||||
pci_register_bar(d, 0, PCI_BASE_ADDRESS_SPACE_MEMORY, &b->bar0);
|
||||
}
|
||||
|
||||
|
|
|
@ -931,7 +931,6 @@ void ppce500_init(MachineState *machine)
|
|||
CPUPPCState *firstenv = NULL;
|
||||
MemoryRegion *ccsr_addr_space;
|
||||
SysBusDevice *s;
|
||||
PPCE500CCSRState *ccsr;
|
||||
I2CBus *i2c;
|
||||
|
||||
irqs = g_new0(IrqLines, smp_cpus);
|
||||
|
@ -993,10 +992,10 @@ void ppce500_init(MachineState *machine)
|
|||
memory_region_add_subregion(address_space_mem, 0, machine->ram);
|
||||
|
||||
dev = qdev_new("e500-ccsr");
|
||||
s = SYS_BUS_DEVICE(dev);
|
||||
object_property_add_child(OBJECT(machine), "e500-ccsr", OBJECT(dev));
|
||||
sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal);
|
||||
ccsr = CCSR(dev);
|
||||
ccsr_addr_space = &ccsr->ccsr_space;
|
||||
sysbus_realize_and_unref(s, &error_fatal);
|
||||
ccsr_addr_space = sysbus_mmio_get_region(s, 0);
|
||||
memory_region_add_subregion(address_space_mem, pmc->ccsrbar_base,
|
||||
ccsr_addr_space);
|
||||
|
||||
|
@ -1284,6 +1283,7 @@ static void e500_ccsr_initfn(Object *obj)
|
|||
PPCE500CCSRState *ccsr = CCSR(obj);
|
||||
memory_region_init(&ccsr->ccsr_space, obj, "e500-ccsr",
|
||||
MPC8544_CCSRBAR_SIZE);
|
||||
sysbus_init_mmio(SYS_BUS_DEVICE(ccsr), &ccsr->ccsr_space);
|
||||
}
|
||||
|
||||
static const TypeInfo e500_ccsr_info = {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue