All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: EXYNOS: call scu_enable() only in case of cortex-A9 processor
@ 2013-05-31  9:43 Leela Krishna Amudala
  2013-05-31 15:14 ` Doug Anderson
  0 siblings, 1 reply; 3+ messages in thread
From: Leela Krishna Amudala @ 2013-05-31  9:43 UTC (permalink / raw)
  To: linux-samsung-soc; +Cc: kgene.kim, dianders

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());
 
 	/*
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: EXYNOS: call scu_enable() only in case of cortex-A9 processor
  2013-05-31  9:43 [PATCH] ARM: EXYNOS: call scu_enable() only in case of cortex-A9 processor Leela Krishna Amudala
@ 2013-05-31 15:14 ` Doug Anderson
  2013-06-05 12:45   ` Kukjin Kim
  0 siblings, 1 reply; 3+ messages in thread
From: Doug Anderson @ 2013-05-31 15:14 UTC (permalink / raw)
  To: Leela Krishna Amudala
  Cc: linux-samsung-soc, Kukjin Kim, Tomasz Figa, Olof Johansson

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.  ;)

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [PATCH] ARM: EXYNOS: call scu_enable() only in case of cortex-A9 processor
  2013-05-31 15:14 ` Doug Anderson
@ 2013-06-05 12:45   ` Kukjin Kim
  0 siblings, 0 replies; 3+ messages in thread
From: Kukjin Kim @ 2013-06-05 12:45 UTC (permalink / raw)
  To: 'Doug Anderson', 'Leela Krishna Amudala'
  Cc: 'linux-samsung-soc', 'Tomasz Figa',
	'Olof Johansson'

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-06-05 12:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-31  9:43 [PATCH] ARM: EXYNOS: call scu_enable() only in case of cortex-A9 processor Leela Krishna Amudala
2013-05-31 15:14 ` Doug Anderson
2013-06-05 12:45   ` Kukjin Kim

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.