Subject: [PATCH-V3 3/3] ARM: OMAP2+: CLEANUP: Remove unnecessary ifdef around __omap2_set_globals

From: [email-masked] (Benoit Cousson)
Date: Thu, 28 Jun 2012 17:05:38 +0200

Hi Vaibhav,

One small comment.

On 06/28/2012 04:59 PM, Vaibhav Hiremath wrote:
> The function __omap2_set_globals() can be common across all
> platforms/architectures, even in case of omap4, internally it
> calls same set of functions as in __omap2_set_globals() function
> (except for sdrc).
> This patch adds new config flag SOC_HAS_OMAP2_SDRC to handle sdrc,
> so that we can reuse same function across omap2/3/4...
>
> Signed-off-by: Vaibhav Hiremath <[email-masked]>
> Cc: Tony Lindgren <[email-masked]>
> Cc: Kevin Hilman <[email-masked]>
> Cc: Paul Walmsley <[email-masked]>
>
> Changes from V1:
> 	- Added Makefile rule to build sdrc.c file for new
> 	  option SOC_HAS_OMAP2_SDRC (as per Kevin's comment).
> ---
>   arch/arm/mach-omap2/Kconfig  |    8 ++++++++
>   arch/arm/mach-omap2/Makefile |    3 ++-
>   arch/arm/mach-omap2/common.c |    8 +-------
>   arch/arm/mach-omap2/common.h |    5 +++++
>   4 files changed, 16 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index 208b950..82d9d18 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -21,12 +21,16 @@ config ARCH_OMAP2PLUS_TYPICAL
>   	help
>   	  Compile a kernel suitable for booting most boards
>
> +config SOC_HAS_OMAP2_SDRC
> +	bool "OMAP2 SDRAM Controller support"
> +
>   config ARCH_OMAP2
>   	bool "TI OMAP2"
>   	depends on ARCH_OMAP2PLUS
>   	default y
>   	select CPU_V6
>   	select MULTI_IRQ_HANDLER
> +	select SOC_HAS_OMAP2_SDRC
>
>   config ARCH_OMAP3
>   	bool "TI OMAP3"
> @@ -39,6 +43,7 @@ config ARCH_OMAP3
>   	select PM_OPP if PM
>   	select ARM_CPU_SUSPEND if PM
>   	select MULTI_IRQ_HANDLER
> +	select SOC_HAS_OMAP2_SDRC
>
>   config ARCH_OMAP4
>   	bool "TI OMAP4"
> @@ -66,16 +71,19 @@ config SOC_OMAP2420
>   	depends on ARCH_OMAP2
>   	default y
>   	select OMAP_DM_TIMER
> +	select SOC_HAS_OMAP2_SDRC

OMAP4 does not have the SDRC but the EMIF like TI8XX.

Regards,
Benoit