diff mbox

[2/2] ARM: DRA7XX: Add support for DRA7XX only build

Message ID 1389347850-22435-3-git-send-email-t-kristo@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Tero Kristo Jan. 10, 2014, 9:57 a.m. UTC
SOC_DRA7XX was under wrong menu within Kconfig file, which prevented
DRA7XX only build. Fixed the kconfig options for this SoC as we are
there. voltage.c needs to be added to the DRA7XX build also, otherwise
DRA7XX only build will fail.

Signed-off-by: Tero Kristo <t-kristo@ti.com>
---
 arch/arm/mach-omap2/Kconfig  |   18 ++++++++++--------
 arch/arm/mach-omap2/Makefile |    1 +
 2 files changed, 11 insertions(+), 8 deletions(-)

Comments

Felipe Balbi Jan. 10, 2014, 4:15 p.m. UTC | #1
On Fri, Jan 10, 2014 at 11:57:30AM +0200, Tero Kristo wrote:
> SOC_DRA7XX was under wrong menu within Kconfig file, which prevented
> DRA7XX only build. Fixed the kconfig options for this SoC as we are
> there. voltage.c needs to be added to the DRA7XX build also, otherwise
> DRA7XX only build will fail.
> 
> Signed-off-by: Tero Kristo <t-kristo@ti.com>

one comment below, other than that:

Reviewed-by: Felipe Balbi <balbi@ti.com>

> ---
>  arch/arm/mach-omap2/Kconfig  |   18 ++++++++++--------
>  arch/arm/mach-omap2/Makefile |    1 +
>  2 files changed, 11 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index dc21df1..e65948a 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -76,6 +76,16 @@ config SOC_AM43XX
>  	select ARM_GIC
>  	select MACH_OMAP_GENERIC
>  
> +config SOC_DRA7XX
> +	bool "TI DRA7XX"
> +	depends on ARCH_MULTI_V7
> +	select ARCH_OMAP2PLUS
> +	select ARM_CPU_SUSPEND if PM
> +	select ARM_GIC
> +	select CPU_V7
> +	select HAVE_SMP
> +	select HAVE_ARM_ARCH_TIMER
> +
>  config ARCH_OMAP2PLUS
>  	bool
>  	select ARCH_HAS_BANDGAP
> @@ -128,14 +138,6 @@ config SOC_HAS_REALTIME_COUNTER
>  	depends on SOC_OMAP5 || SOC_DRA7XX
>  	default y
>  
> -config SOC_DRA7XX
> -	bool "TI DRA7XX"
> -	select ARM_ARCH_TIMER
> -	select CPU_V7
> -	select ARM_GIC
> -	select HAVE_SMP
> -	select COMMON_CLK

seems like you lost COMMON_CLK when moving this up, was that on purpose?
Tero Kristo Jan. 10, 2014, 4:32 p.m. UTC | #2
On 01/10/2014 06:15 PM, Felipe Balbi wrote:
> On Fri, Jan 10, 2014 at 11:57:30AM +0200, Tero Kristo wrote:
>> SOC_DRA7XX was under wrong menu within Kconfig file, which prevented
>> DRA7XX only build. Fixed the kconfig options for this SoC as we are
>> there. voltage.c needs to be added to the DRA7XX build also, otherwise
>> DRA7XX only build will fail.
>>
>> Signed-off-by: Tero Kristo <t-kristo@ti.com>
>
> one comment below, other than that:
>
> Reviewed-by: Felipe Balbi <balbi@ti.com>
>
>> ---
>>   arch/arm/mach-omap2/Kconfig  |   18 ++++++++++--------
>>   arch/arm/mach-omap2/Makefile |    1 +
>>   2 files changed, 11 insertions(+), 8 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
>> index dc21df1..e65948a 100644
>> --- a/arch/arm/mach-omap2/Kconfig
>> +++ b/arch/arm/mach-omap2/Kconfig
>> @@ -76,6 +76,16 @@ config SOC_AM43XX
>>   	select ARM_GIC
>>   	select MACH_OMAP_GENERIC
>>
>> +config SOC_DRA7XX
>> +	bool "TI DRA7XX"
>> +	depends on ARCH_MULTI_V7
>> +	select ARCH_OMAP2PLUS
>> +	select ARM_CPU_SUSPEND if PM
>> +	select ARM_GIC
>> +	select CPU_V7
>> +	select HAVE_SMP
>> +	select HAVE_ARM_ARCH_TIMER
>> +
>>   config ARCH_OMAP2PLUS
>>   	bool
>>   	select ARCH_HAS_BANDGAP
>> @@ -128,14 +138,6 @@ config SOC_HAS_REALTIME_COUNTER
>>   	depends on SOC_OMAP5 || SOC_DRA7XX
>>   	default y
>>
>> -config SOC_DRA7XX
>> -	bool "TI DRA7XX"
>> -	select ARM_ARCH_TIMER
>> -	select CPU_V7
>> -	select ARM_GIC
>> -	select HAVE_SMP
>> -	select COMMON_CLK
>
> seems like you lost COMMON_CLK when moving this up, was that on purpose?
>

Yes. It comes through ARCH_OMAP2PLUS which is now selected.

-Tero
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index dc21df1..e65948a 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -76,6 +76,16 @@  config SOC_AM43XX
 	select ARM_GIC
 	select MACH_OMAP_GENERIC
 
+config SOC_DRA7XX
+	bool "TI DRA7XX"
+	depends on ARCH_MULTI_V7
+	select ARCH_OMAP2PLUS
+	select ARM_CPU_SUSPEND if PM
+	select ARM_GIC
+	select CPU_V7
+	select HAVE_SMP
+	select HAVE_ARM_ARCH_TIMER
+
 config ARCH_OMAP2PLUS
 	bool
 	select ARCH_HAS_BANDGAP
@@ -128,14 +138,6 @@  config SOC_HAS_REALTIME_COUNTER
 	depends on SOC_OMAP5 || SOC_DRA7XX
 	default y
 
-config SOC_DRA7XX
-	bool "TI DRA7XX"
-	select ARM_ARCH_TIMER
-	select CPU_V7
-	select ARM_GIC
-	select HAVE_SMP
-	select COMMON_CLK
-
 comment "OMAP Core Type"
 	depends on ARCH_OMAP2
 
diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
index 96a990f..8ebe9f3 100644
--- a/arch/arm/mach-omap2/Makefile
+++ b/arch/arm/mach-omap2/Makefile
@@ -132,6 +132,7 @@  obj-$(CONFIG_SOC_AM33XX)		+= $(voltagedomain-common)
 obj-$(CONFIG_SOC_AM43XX)		+= $(voltagedomain-common)
 obj-$(CONFIG_SOC_OMAP5)			+= $(voltagedomain-common)
 obj-$(CONFIG_SOC_OMAP5)                += voltagedomains54xx_data.o
+obj-$(CONFIG_SOC_DRA7XX)		+= $(voltagedomain-common)
 
 # OMAP powerdomain framework
 powerdomain-common			+= powerdomain.o powerdomain-common.o