[Buildroot] [PATCH] arch/Config.in: armeb no-MMU is not supported

[Buildroot] [PATCH] arch/Config.in: armeb no-MMU is not supported

[Thomas Petazzoni via buildroot]

elf2flt does not support ARM big-endian, so supporting Cortex M3/M4/M7
with armeb is not possibly.

Therefore this commit makes:

 - MMU mandatory on armeb
 - Prevents from seeing Cortex M3/M4/M7 on armeb

Fixes:

  http://autobuild.buildroot.net/results/9bca0cbfb6a66c455e74ad194526bca942665978/

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
 arch/Config.in     | 2 +-
 arch/Config.in.arm | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/arch/Config.in b/arch/Config.in
index 49e16fce38..da499b37ab 100644
--- a/arch/Config.in
+++ b/arch/Config.in
@@ -49,7 +49,7 @@ config BR2_arm
 
 config BR2_armeb
 	bool "ARM (big endian)"
-	# MMU support is set by the subarchitecture file, arch/Config.in.arm
+	select BR2_ARCH_HAS_MMU_MANDATORY
 	help
 	  ARM is a 32-bit reduced instruction set computer (RISC)
 	  instruction set architecture (ISA) developed by ARM Holdings.
diff --git a/arch/Config.in.arm b/arch/Config.in.arm
index fa2e882550..eb71d70430 100644
--- a/arch/Config.in.arm
+++ b/arch/Config.in.arm
@@ -242,6 +242,8 @@ config BR2_pj4
 	select BR2_ARM_CPU_HAS_VFPV3
 	select BR2_ARM_CPU_ARMV7A
 
+# Cortex-M cores are only supported for little endian configurations
+if BR2_arm
 comment "armv7m cores"
 config BR2_cortex_m3
 	bool "cortex-M3"
@@ -258,6 +260,7 @@ config BR2_cortex_m7
 	select BR2_ARM_CPU_MAYBE_HAS_FPV5
 	select BR2_ARM_CPU_ARMV7M
 	select BR2_ARCH_NEEDS_GCC_AT_LEAST_5
+endif # BR2_arm
 endif # !BR2_ARCH_IS_64
 
 comment "armv8 cores"
-- 
2.35.3

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] arch/Config.in: armeb no-MMU is not supported

[Thomas Petazzoni via buildroot]

On Wed,  8 Jun 2022 11:48:02 +0200
Thomas Petazzoni via buildroot <buildroot@buildroot.org> wrote:

> elf2flt does not support ARM big-endian, so supporting Cortex M3/M4/M7
> with armeb is not possibly.
> 
> Therefore this commit makes:
> 
>  - MMU mandatory on armeb
>  - Prevents from seeing Cortex M3/M4/M7 on armeb
> 
> Fixes:
> 
>   http://autobuild.buildroot.net/results/9bca0cbfb6a66c455e74ad194526bca942665978/
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> ---
>  arch/Config.in     | 2 +-
>  arch/Config.in.arm | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)

I have mistakenly pushed my own patch, sorry about this. Hopefully
people are OK with this, if not I can revert. Let me know.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] arch/Config.in: armeb no-MMU is not supported

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Petazzoni via buildroot <buildroot@buildroot.org> writes:

 > elf2flt does not support ARM big-endian, so supporting Cortex M3/M4/M7
 > with armeb is not possibly.

 > Therefore this commit makes:

 >  - MMU mandatory on armeb
 >  - Prevents from seeing Cortex M3/M4/M7 on armeb

 > Fixes:

 >   http://autobuild.buildroot.net/results/9bca0cbfb6a66c455e74ad194526bca942665978/

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

Committed to 2022.05.x and 2022.02.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot