On 18/05/2020 06:03, Markus Armbruster wrote: > macio_oldworld_init() creates a "macio-nvram", sysbus device, but > neglects to but it on a bus. > > Put it on the macio bus. Affects machine g3beige. Visible in "info > qtree": > > bus: macio.0 > type macio-bus > [...] > + dev: macio-nvram, id "" > + size = 8192 (0x2000) > + it_shift = 4 (0x4) > > This also makes it a QOM child of macio-oldworld. Visible in "info > qom-tree": > > /machine (g3beige-machine) > [...] > /unattached (container) > [...] > /device[6] (macio-oldworld) > [...] > - /device[7] (macio-nvram) > - /macio-nvram[0] (qemu:memory-region) > + /nvram (macio-nvram) > + /macio-nvram[0] (qemu:memory-region) > [rest of device[*] renumbered...] > > Cc: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> > Cc: David Gibson <da...@gibson.dropbear.id.au> > Cc: qemu-...@nongnu.org > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > hw/misc/macio/macio.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/hw/misc/macio/macio.c b/hw/misc/macio/macio.c > index b3dddf8be7..ebc96cc8f6 100644 > --- a/hw/misc/macio/macio.c > +++ b/hw/misc/macio/macio.c > @@ -245,7 +245,8 @@ static void macio_oldworld_init(Object *obj) > > macio_init_child_obj(s, "cuda", &s->cuda, sizeof(s->cuda), TYPE_CUDA); > > - object_initialize(&os->nvram, sizeof(os->nvram), TYPE_MACIO_NVRAM); > + macio_init_child_obj(s, "nvram", &os->nvram, sizeof(os->nvram), > + TYPE_MACIO_NVRAM); > dev = DEVICE(&os->nvram); > qdev_prop_set_uint32(dev, "size", 0x2000); > qdev_prop_set_uint32(dev, "it_shift", 4);
Reviewed-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> ATB, Mark.
