RE: [PATCH] ARM: EXYNOS: Fix the check for non-smp configuration

2014-07-04 Thread Kukjin Kim
Abhilash Kesavan wrote:
 
 Commit 1754c42e3db5(ARM: exynos: move sysram info to exynos.c) missed
 out the CONFIG_ prefix causing exynos_sysram_init() to get called twice
 for SMP configurations.
 
 Signed-off-by: Abhilash Kesavan a.kesa...@samsung.com
 ---
  arch/arm/mach-exynos/exynos.c |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/arch/arm/mach-exynos/exynos.c b/arch/arm/mach-exynos/exynos.c
 index f38cf7c..95cad25 100644
 --- a/arch/arm/mach-exynos/exynos.c
 +++ b/arch/arm/mach-exynos/exynos.c
 @@ -297,7 +297,7 @@ static void __init exynos_dt_machine_init(void)
* This is called from smp_prepare_cpus if we've built for SMP, but
* we still need to set it up for PM and firmware ops if not.
*/
 - if (!IS_ENABLED(SMP))
 + if (!IS_ENABLED(CONFIG_SMP))
   exynos_sysram_init();
 
   exynos_cpuidle_init();
 --
 1.7.9.5

Oh, OK. will apply into fixes.

Thanks,
Kukjin

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] ARM: EXYNOS: Fix the check for non-smp configuration

2014-07-04 Thread Sachin Kamat
On Thu, Jul 3, 2014 at 10:33 AM, Abhilash Kesavan a.kesa...@samsung.com wrote:
 Commit 1754c42e3db5(ARM: exynos: move sysram info to exynos.c) missed
 out the CONFIG_ prefix causing exynos_sysram_init() to get called twice
 for SMP configurations.

 Signed-off-by: Abhilash Kesavan a.kesa...@samsung.com
 ---
  arch/arm/mach-exynos/exynos.c |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/arch/arm/mach-exynos/exynos.c b/arch/arm/mach-exynos/exynos.c
 index f38cf7c..95cad25 100644
 --- a/arch/arm/mach-exynos/exynos.c
 +++ b/arch/arm/mach-exynos/exynos.c
 @@ -297,7 +297,7 @@ static void __init exynos_dt_machine_init(void)
  * This is called from smp_prepare_cpus if we've built for SMP, but
  * we still need to set it up for PM and firmware ops if not.
  */
 -   if (!IS_ENABLED(SMP))
 +   if (!IS_ENABLED(CONFIG_SMP))
 exynos_sysram_init();

 exynos_cpuidle_init();

If it's not too late
Reviewed-by: Sachin Kamat sachin.ka...@samsug.com

-- 
Regards,
Sachin.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html