RE: [PATCH] ARM: EXYNOS: call scu_enable() only in case of cortex-A9 processor

[Kukjin Kim <kgene.kim@samsung.com>]

Doug Anderson wrote:
> 
> Leela,
> 
> On Fri, May 31, 2013 at 2:43 AM, Leela Krishna Amudala
> <l.krishna@samsung.com> wrote:
> > This patch reads the cpuid part number and if it matches with
> > cortex-A9, calls scu_enable()
> >
> > Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com>
> > ---
> >  arch/arm/mach-exynos/platsmp.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-exynos/platsmp.c b/arch/arm/mach-
> exynos/platsmp.c
> > index a0e8ff7..d9c6d0a 100644
> > --- a/arch/arm/mach-exynos/platsmp.c
> > +++ b/arch/arm/mach-exynos/platsmp.c
> > @@ -200,7 +200,7 @@ static void __init exynos_smp_prepare_cpus(unsigned
> int max_cpus)
> >  {
> >         int i;
> >
> > -       if (!(soc_is_exynos5250() || soc_is_exynos5440()))
> > +       if (read_cpuid_part_number() == ARM_CPU_PART_CORTEX_A9)
> >                 scu_enable(scu_base_addr());
> >
> >         /*
> 
> Thanks for sending upstream.  For reference, our local version is here
> <https://gerrit.chromium.org/gerrit/#/c/56804/>.
> 
> This is much better than listing every single non-A9 exynos in a big
> "if" test.  ;)
> 
Yes :)

> Reviewed-by: Doug Anderson <dianders@chromium.org>

Applied, thanks.

- Kukjin