Hi Laurent,

On Wed, Oct 5, 2016 at 12:28 PM, Laurent Pinchart
<laurent.pinchart+rene...@ideasonboard.com> wrote:
> The SMP initialization function is only called if CONFIG_SMP is defined.
> Remove the duplicate check.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
> ---
>  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 -- ge...@linux-m68k.org

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

Reply via email to