Hi Laurent, On Wed, Oct 5, 2016 at 12:28 PM, Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> wrote: > The SMP initialization function is only called if CONFIG_SMP is defined. > Remove the duplicate check. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > --- > arch/arm/mach-shmobile/platsmp.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/arch/arm/mach-shmobile/platsmp.c b/arch/arm/mach-shmobile/platsmp.c > index 02e21bceb085..48595a397220 100644 > --- a/arch/arm/mach-shmobile/platsmp.c > +++ b/arch/arm/mach-shmobile/platsmp.c > @@ -39,9 +39,5 @@ bool shmobile_smp_cpu_can_disable(unsigned int cpu) > > bool __init shmobile_smp_init_fallback_ops(void) > { > - /* fallback on PSCI/smp_ops if no other DT based method is detected */ > - if (!IS_ENABLED(CONFIG_SMP)) > - return false; > - > return platform_can_secondary_boot() ? true : false; > } This check was added recently, because the UP build was broken. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds