Re: [PATCH 12/13] ARM: OMAP5: Add the build support

[Tony Lindgren <tony@atomide.com>]

* R Sricharan <r.sricharan@ti.com> [120503 00:30]:
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -9,7 +9,7 @@ config ARCH_OMAP2PLUS_TYPICAL
>  	select REGULATOR
>  	select PM_RUNTIME
>  	select VFP
> -	select NEON if ARCH_OMAP3 || ARCH_OMAP4
> +	select NEON if ARCH_OMAP3 || ARCH_OMAP4 || ARCH_OMAP5
>  	select SERIAL_OMAP
>  	select SERIAL_OMAP_CONSOLE
>  	select I2C

If we add CONFIG_SOC_OMAP3PLUS and CONFIG_SOC_OMAP4PLUS, then
this becomes more future proof with select NEON if SOC_OMAP3PLUS.

> @@ -55,6 +55,13 @@ config ARCH_OMAP4
>  	select USB_ARCH_HAS_EHCI if USB_SUPPORT
>  	select ARM_CPU_SUSPEND if PM
>  
> +config ARCH_OMAP5
> +	bool "TI OMAP5"
> +	depends on ARCH_OMAP2PLUS
> +	select CPU_V7
> +	select ARM_GIC
> +	select HAVE_SMP

No need to have depends on ARCH_OMAP2PLUS, it's all inside
if ARCH_OMAP2PLUS anyways. I removed those already once, but that
had to be reverted because the patch was doing other things too
that did not work out too well..

> @@ -343,6 +350,10 @@ config MACH_OMAP4_PANDA
>  	select OMAP_PACKAGE_CBS
>  	select REGULATOR_FIXED_VOLTAGE if REGULATOR
>  
> +config MACH_OMAP5_SEVM
> +	bool "OMAP5 sevm Board"
> +	depends on ARCH_OMAP5
> +
>  config OMAP3_EMU
>  	bool "OMAP3 debugging peripherals"
>  	depends on ARCH_OMAP3

No need for it here either. Actually, I think this whole chunk
can be now left out since it's DT based?

Regards,

Tony