Skip to contentSkip to conversation

arch/Config.in.xtensa: provide BR2_ENDIAN symbol

PATCH New Max Filippov 2 participants 2 messages May 5, 2015

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 arch/Config.in.xtensa: provide BR2_ENDIAN symbol

Packages get hints from the buildroot at configure time, e.g. ac_cv_c_bigendian=yes/no. This particular hint is based on the value of BR2_ENDIAN and when it doesn't match actual toolchain endiannes the build may break.
Provide BR2_ENDIAN for xtensa to fix this sort of build errors.

Fixes:
http://autobuild.buildroot.net/results/31115fe8d88f52d77ed0f2da769eb8896a1b34a2/

---
Changes v1->v2:
- add spaces between configure items;
- expand changelog.

 arch/Config.in.xtensa | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
arch/Config.in.xtensa +17 -0
index ff71e0b..0687319 100644
@@ -33,6 +33,23 @@ config BR2_XTENSA_OVERLAY_DIR
 	  configurations. They are provided by the processor vendor or
 	  directly from Tensilica.
 
+choice
+	prompt "Target Architecture Endianness"
+	depends on BR2_XTENSA_CUSTOM
+	default BR2_XTENSA_LITTLE_ENDIAN
+
+config BR2_XTENSA_LITTLE_ENDIAN
+	bool "Little endian"
+
+config BR2_XTENSA_BIG_ENDIAN
+	bool "Big endian"
+
+endchoice
+
+config BR2_ENDIAN
+	default "LITTLE"	if BR2_XTENSA_LITTLE_ENDIAN
+	default "BIG"		if BR2_xtensa_fsf || BR2_XTENSA_BIG_ENDIAN
+
 config BR2_ARCH
 	default "xtensa"	if BR2_xtensa
 
Signature · 1 line
1.8.1.4
#1 lore raw

PATCH v2 1/1 arch/Config.in.xtensa: provide BR2_ENDIAN symbol

Dear Max Filippov,

Quoted 6 lines · Max Filippov

Packages get hints from the buildroot at configure time, e.g. ac_cv_c_bigendian=yes/no. This particular hint is based on the value of BR2_ENDIAN and when it doesn't match actual toolchain endiannes the build may break.
Provide BR2_ENDIAN for xtensa to fix this sort of build errors.

Quoted 8 lines

Fixes:
http://autobuild.buildroot.net/results/31115fe8d88f52d77ed0f2da769eb8896a1b34a2/

---
Changes v1->v2:
- add spaces between configure items;
- expand changelog.

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com