diff mbox

ARM: mpu: Ensure that MPU depends on CPU_V7

Message ID 1370625473-21628-1-git-send-email-jonathan.austin@arm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jonathan Austin June 7, 2013, 5:17 p.m. UTC
The support for the MPU is currently implemented only for R-class
(PMSAv7/R). Since the merge of V7M support in to the kernel it is possible
to select MPU support on V7M.

This patch ensures that until MPU support for M-class processors is
implemented, the MPU can only be selected with R-class CPUs

Signed-off-by: Jonathan Austin <jonathan.austin@arm.com>
Cc: Uwe Kleine-Konig <u.kleine-koenig@pengutronix.de>
---

This patch is necessary now that Russell has merged the V7M work in to his
devel-stable branch.

I don't have any immediate plans to implement MPU support for M-class, so this
patch is important to ensure we don't break building of V7M platforms.

 arch/arm/Kconfig-nommu |    1 +
 1 file changed, 1 insertion(+)

Comments

Uwe Kleine-König June 7, 2013, 6:11 p.m. UTC | #1
On Fri, Jun 07, 2013 at 06:17:53PM +0100, Jonathan Austin wrote:
> The support for the MPU is currently implemented only for R-class
> (PMSAv7/R). Since the merge of V7M support in to the kernel it is possible
> to select MPU support on V7M.
> 
> This patch ensures that until MPU support for M-class processors is
> implemented, the MPU can only be selected with R-class CPUs
> 
> Signed-off-by: Jonathan Austin <jonathan.austin@arm.com>
> Cc: Uwe Kleine-Konig <u.kleine-koenig@pengutronix.de>
> ---
> 
> This patch is necessary now that Russell has merged the V7M work in to his
> devel-stable branch.
> 
> I don't have any immediate plans to implement MPU support for M-class, so this
> patch is important to ensure we don't break building of V7M platforms.
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

I didn't check, but that might affect other nommu configs for pre-v7
cpu's, too. (I think there is support for an at91 in the tree?!)

Best regards
Uwe

>  arch/arm/Kconfig-nommu |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/Kconfig-nommu b/arch/arm/Kconfig-nommu
> index db597e1..aed66d5 100644
> --- a/arch/arm/Kconfig-nommu
> +++ b/arch/arm/Kconfig-nommu
> @@ -53,6 +53,7 @@ config REMAP_VECTORS_TO_RAM
>  
>  config ARM_MPU
>         bool 'Use the ARM v7 PMSA Compliant MPU'
> +       depends on CPU_V7
>         default y
>         help
>           Some ARM systems without an MMU have instead a Memory Protection
> -- 
> 1.7.9.5
> 
> 
>
Jonathan Austin June 7, 2013, 6:20 p.m. UTC | #2
On 07/06/13 19:11, Uwe Kleine-König wrote:
> On Fri, Jun 07, 2013 at 06:17:53PM +0100, Jonathan Austin wrote:
>> The support for the MPU is currently implemented only for R-class
>> (PMSAv7/R). Since the merge of V7M support in to the kernel it is possible
>> to select MPU support on V7M.
>>
>> This patch ensures that until MPU support for M-class processors is
>> implemented, the MPU can only be selected with R-class CPUs
>>
>> Signed-off-by: Jonathan Austin <jonathan.austin@arm.com>
>> Cc: Uwe Kleine-Konig <u.kleine-koenig@pengutronix.de>
>> ---
>>
>> This patch is necessary now that Russell has merged the V7M work in to his
>> devel-stable branch.
>>
>> I don't have any immediate plans to implement MPU support for M-class, so this
>> patch is important to ensure we don't break building of V7M platforms.
> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
>
> I didn't check, but that might affect other nommu configs for pre-v7
> cpu's, too. (I think there is support for an at91 in the tree?!)
>

Thanks Uwe,

You're right - I just checked and without this patch we cause issues 
with at91x40_defconfig. This patch *does* fix that too.

I've added your acked-by and this time I've got the 'ö' :)

Jonny
diff mbox

Patch

diff --git a/arch/arm/Kconfig-nommu b/arch/arm/Kconfig-nommu
index db597e1..aed66d5 100644
--- a/arch/arm/Kconfig-nommu
+++ b/arch/arm/Kconfig-nommu
@@ -53,6 +53,7 @@  config REMAP_VECTORS_TO_RAM
 
 config ARM_MPU
        bool 'Use the ARM v7 PMSA Compliant MPU'
+       depends on CPU_V7
        default y
        help
          Some ARM systems without an MMU have instead a Memory Protection