On 03/13/2018 04:34 PM, Peter Maydell wrote: > For the rpi1 and 2 we want to boot the Linux kernel via some > custom setup code that makes sure that the SMC instruction > acts as a no-op, because it's used for cache maintenance. > The rpi3 boots AArch64 kernels, which don't need SMC for > cache maintenance and always expect to be booted non-secure. > Don't fill in the aarch32-specific parts of the binfo struct. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > hw/arm/raspi.c | 17 +++++++++++++---- > 1 file changed, 13 insertions(+), 4 deletions(-) > > diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c > index a37881433c..1ac0737149 100644 > --- a/hw/arm/raspi.c > +++ b/hw/arm/raspi.c > @@ -82,10 +82,19 @@ static void setup_boot(MachineState *machine, int > version, size_t ram_size) > binfo.board_id = raspi_boardid[version]; > binfo.ram_size = ram_size; > binfo.nb_cpus = smp_cpus; > - binfo.board_setup_addr = BOARDSETUP_ADDR; > - binfo.write_board_setup = write_board_setup; > - binfo.secure_board_setup = true; > - binfo.secure_boot = true; > + > + if (version <= 2) { > + /* The rpi1 and 2 require some custom setup code to run in Secure > + * mode before booting a kernel (to set up the SMC vectors so > + * that we get a no-op SMC; this is used by Linux to call the > + * firmware for some cache maintenance operations. > + * The rpi3 doesn't need this. > + */
I was expecting a much complicated fix... neat. Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > + binfo.board_setup_addr = BOARDSETUP_ADDR; > + binfo.write_board_setup = write_board_setup; > + binfo.secure_board_setup = true; > + binfo.secure_boot = true; > + } > > /* Pi2 and Pi3 requires SMP setup */ > if (version >= 2) { >