diff mbox

[v1] ARM: do not select ARCH_VEXPRESS by default on ARCH_MULTI_V7

Message ID 1366634604-12637-1-git-send-email-srinivas.kandagatla@st.com (mailing list archive)
State New, archived
Headers show

Commit Message

Srinivas KANDAGATLA April 22, 2013, 12:43 p.m. UTC 
From: Srinivas Kandagatla <srinivas.kandagatla@st.com>

This patch removes ARCH_VEXPRESS selection as default when ARCH_MULTI_V7
is selected. ARCH_VEXPRESS should be selected from
mach-vexpress/Kconfig.

Without this patch, any multiplatform build will have ARCH_VEXPRESS
selected, as a result all the vexpress associated dtbs are build when we
make dtbs.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@st.com>
---
 arch/arm/Kconfig |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

Comments

Pawel Moll April 22, 2013, 4:59 p.m. UTC | #1 
Hi Srinivas,

On Mon, 2013-04-22 at 13:43 +0100, Srinivas KANDAGATLA wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@st.com>
> 
> This patch removes ARCH_VEXPRESS selection as default when ARCH_MULTI_V7
> is selected. ARCH_VEXPRESS should be selected from
> mach-vexpress/Kconfig.
> 
> Without this patch, any multiplatform build will have ARCH_VEXPRESS
> selected, as a result all the vexpress associated dtbs are build when we
> make dtbs.
> 
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@st.com>

Fabio was looking at this some time ago:
http://thread.gmane.org/gmane.linux.ports.arm.kernel/201499
He may comment on the relevant issues.

Cheers!

Pawe?
Fabio Estevam April 22, 2013, 5:10 p.m. UTC | #2 
On Mon, Apr 22, 2013 at 1:59 PM, Pawel Moll <pawel.moll@arm.com> wrote:

> Fabio was looking at this some time ago:
> http://thread.gmane.org/gmane.linux.ports.arm.kernel/201499
> He may comment on the relevant issues.

Arnd has recently posted a patch to address this issue:
https://patchwork.kernel.org/patch/2436131/

Regards,

Fabio Estevam
Russell King - ARM Linux April 22, 2013, 5:49 p.m. UTC | #3 
On Mon, Apr 22, 2013 at 01:43:24PM +0100, Srinivas KANDAGATLA wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@st.com>
> 
> This patch removes ARCH_VEXPRESS selection as default when ARCH_MULTI_V7
> is selected. ARCH_VEXPRESS should be selected from
> mach-vexpress/Kconfig.
> 
> Without this patch, any multiplatform build will have ARCH_VEXPRESS
> selected, as a result all the vexpress associated dtbs are build when we
> make dtbs.
> 
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@st.com>

There are reasons that this is done at the moment; Arnd has this under
control, and will remove this when stuff is ready to survive without
VEXPRESS enabled.
Srinivas KANDAGATLA April 23, 2013, 6:48 a.m. UTC | #4 
On 22/04/13 18:10, Fabio Estevam wrote:
> On Mon, Apr 22, 2013 at 1:59 PM, Pawel Moll <pawel.moll@arm.com> wrote:
>
>> Fabio was looking at this some time ago:
>> http://thread.gmane.org/gmane.linux.ports.arm.kernel/201499
>> He may comment on the relevant issues.
Thankyou for the patch, now I understand the reasoning.
> Arnd has recently posted a patch to address this issue:
> https://patchwork.kernel.org/patch/2436131/
>
> Regards,
>
> Fabio Estevam
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
diff mbox

Patch

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 1cacda4..e67d49d 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1014,7 +1014,6 @@  config ARCH_MULTI_V7
 	bool "ARMv7 based platforms (Cortex-A, PJ4, Scorpion, Krait)"
 	default y
 	select ARCH_MULTI_V6_V7
-	select ARCH_VEXPRESS
 	select CPU_V7
 
 config ARCH_MULTI_V6_V7