On 31/05/2023 14:55, Philippe Mathieu-Daudé wrote:
On 31/5/23 14:53, Mark Cave-Ayland wrote:
Also change the instantiation of the macfb device to use
object_initialize_child().
Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk>
---
hw/m68k/q800.c | 6 ++++--
include/hw/m68k/q800.h | 2 ++
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/hw/m68k/q800.c b/hw/m68k/q800.c
index 946cb09e30..0e7451e77a 100644
--- a/hw/m68k/q800.c
+++ b/hw/m68k/q800.c
@@ -449,7 +449,9 @@ static void q800_machine_init(MachineState *machine)
/* framebuffer in nubus slot #9 */
- dev = qdev_new(TYPE_NUBUS_MACFB);
+ object_initialize_child(OBJECT(machine), "macfb", &m->macfb,
Alternatively "framebuffer" or "fb". Regardless,
I'd prefer to keep this as macfb if possible, simply because it matches the
convention of using the device type as the basis for the object child property name
throughout the series.
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
+ TYPE_NUBUS_MACFB);
ATB,
Mark.