diff mbox

[v2] ARM: Kconfig: let ARM9TDMI and ARM7TDMI invisible.

Message ID 51775D88.2000406@asianux.com (mailing list archive)
State New, archived
Headers show

Commit Message

Chen Gang April 24, 2013, 4:20 a.m. UTC
If CONFIG_MMU is disabled, it will let ARM9TDMI and ARM7TDMI visible,
but in fact, only AT91X40 need ARM7TDMI, so need not let them visible.

Signed-off-by: Chen Gang <gang.chen@asianux.com>
---
 arch/arm/mach-at91/Kconfig.non_dt |    1 +
 arch/arm/mm/Kconfig               |    4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

Comments

Russell King - ARM Linux April 24, 2013, 5:46 p.m. UTC | #1
On Wed, Apr 24, 2013 at 12:20:24PM +0800, Chen Gang wrote:
> If CONFIG_MMU is disabled, it will let ARM9TDMI and ARM7TDMI visible,
> but in fact, only AT91X40 need ARM7TDMI, so need not let them visible.

I'll apply it myself this evening.  Can I add:

Tested-by: Chen Gang <gang.chen@asianux.com>

to it?
Jean-Christophe PLAGNIOL-VILLARD April 24, 2013, 5:59 p.m. UTC | #2
On Apr 25, 2013, at 1:46 AM, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote:

> On Wed, Apr 24, 2013 at 12:20:24PM +0800, Chen Gang wrote:
>> If CONFIG_MMU is disabled, it will let ARM9TDMI and ARM7TDMI visible,
>> but in fact, only AT91X40 need ARM7TDMI, so need not let them visible.
> 
> I'll apply it myself this evening.  Can I add:
> 
> Tested-by: Chen Gang <gang.chen@asianux.com>
you add it mine too

Tested-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

Best Regards,
J.
> 
> to it?
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Chen Gang April 25, 2013, 1:16 a.m. UTC | #3
On 2013?04?25? 01:46, Russell King - ARM Linux wrote:
> On Wed, Apr 24, 2013 at 12:20:24PM +0800, Chen Gang wrote:
>> > If CONFIG_MMU is disabled, it will let ARM9TDMI and ARM7TDMI visible,
>> > but in fact, only AT91X40 need ARM7TDMI, so need not let them visible.
> I'll apply it myself this evening.  Can I add:
> 
> Tested-by: Chen Gang <gang.chen@asianux.com>
> 
> to it?
> 
> 

Ok, thanks. this patch is really provided by you (Arnd also provided the
same patch, but it seems a little later than you). And I really only
reported and tested it.

And next, I should continue to find and make another patches for ARM.

Thanks.
Chen Gang April 25, 2013, 1:16 a.m. UTC | #4
On 2013?04?25? 01:59, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On Apr 25, 2013, at 1:46 AM, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote:
> 
>> > On Wed, Apr 24, 2013 at 12:20:24PM +0800, Chen Gang wrote:
>>> >> If CONFIG_MMU is disabled, it will let ARM9TDMI and ARM7TDMI visible,
>>> >> but in fact, only AT91X40 need ARM7TDMI, so need not let them visible.
>> > 
>> > I'll apply it myself this evening.  Can I add:
>> > 
>> > Tested-by: Chen Gang <gang.chen@asianux.com>
> you add it mine too
> 
> Tested-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

Thanks.
diff mbox

Patch

diff --git a/arch/arm/mach-at91/Kconfig.non_dt b/arch/arm/mach-at91/Kconfig.non_dt
index 6c24985..dc972e1 100644
--- a/arch/arm/mach-at91/Kconfig.non_dt
+++ b/arch/arm/mach-at91/Kconfig.non_dt
@@ -47,6 +47,7 @@  config ARCH_AT91X40
 	select ARCH_USES_GETTIMEOFFSET
 	select MULTI_IRQ_HANDLER
 	select SPARSE_IRQ
+	select CPU_ARM7TDMI
 
 endchoice
 
diff --git a/arch/arm/mm/Kconfig b/arch/arm/mm/Kconfig
index 9e8101e..3397d05 100644
--- a/arch/arm/mm/Kconfig
+++ b/arch/arm/mm/Kconfig
@@ -6,7 +6,7 @@  comment "Processor Type"
 
 # ARM7TDMI
 config CPU_ARM7TDMI
-	bool "Support ARM7TDMI processor"
+	bool
 	depends on !MMU
 	select CPU_32v4T
 	select CPU_ABRT_LV4T
@@ -56,7 +56,7 @@  config CPU_ARM740T
 
 # ARM9TDMI
 config CPU_ARM9TDMI
-	bool "Support ARM9TDMI processor"
+	bool
 	depends on !MMU
 	select CPU_32v4T
 	select CPU_ABRT_NOMMU