On Sat, 8 Feb 2020 17:56:41 +0100
Philippe Mathieu-Daudé <f4...@amsat.org> wrote:
> raspi_machine_init() access to board_rev via RaspiMachineClass.
> raspi2_init() and raspi3_init() do nothing. Call raspi_machine_init
> directly.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> Squash with previous?
> ---
> hw/arm/raspi.c | 16 +++-------------
> 1 file changed, 3 insertions(+), 13 deletions(-)
>
> diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
> index fbfcd29732..1628b0dda7 100644
> --- a/hw/arm/raspi.c
> +++ b/hw/arm/raspi.c
> @@ -228,7 +228,7 @@ static void setup_boot(MachineState *machine, int
> version, size_t ram_size)
> arm_load_kernel(ARM_CPU(first_cpu), machine, &binfo);
> }
>
> -static void raspi_init(MachineState *machine)
> +static void raspi_machine_init(MachineState *machine)
> {
> RaspiMachineClass *mc = RASPI_MACHINE_GET_CLASS(machine);
> RaspiMachineState *s = RASPI_MACHINE(machine);
> @@ -280,11 +280,6 @@ static void raspi_init(MachineState *machine)
> setup_boot(machine, version, machine->ram_size - vcram_size);
> }
>
> -static void raspi2_init(MachineState *machine)
> -{
> - raspi_init(machine);
> -}
> -
> static void raspi2_machine_class_init(ObjectClass *oc, void *data)
> {
> MachineClass *mc = MACHINE_CLASS(oc);
> @@ -293,7 +288,7 @@ static void raspi2_machine_class_init(ObjectClass *oc,
> void *data)
>
> rmc->board_rev = board_rev;
> mc->desc = "Raspberry Pi 2B";
> - mc->init = raspi2_init;
> + mc->init = raspi_machine_init;
[...]
> @@ -319,7 +309,7 @@ static void raspi3_machine_class_init(ObjectClass *oc,
> void *data)
>
> rmc->board_rev = board_rev;
> mc->desc = "Raspberry Pi 3B";
> - mc->init = raspi3_init;
> + mc->init = raspi_machine_init;
[...]
you could set it once in base class_init,
since there is not reason lest to do it per leaf class.
Either with this change or without
Reviewed-by: Igor Mammedov <imamm...@redhat.com>
