Skip to contentSkip to conversation

uboot: arm64 arch build support

PATCH New Matt Weber 2 participants 2 messages Oct 12, 2015

lore
Share & tools
lore

Matt Weber

author
#0 lore raw

PATCH v2 1/1 uboot: arm64 arch build support

From: Ronak Desai <[email protected]>

For a 64bit arm architecture, Buildroot uses arm64 terminology and from the top-level Makefile KERNEL_ARCH is set to arm64 which is then passed to the uboot build.
This causes a compilation issue as uboot uses the top-level system architecture as it's $(ARCH). So arch/$(ARCH)/Makefile doesn't work with arm64 unless we adjust the arch.

Signed-off-by Ronak Desai <[email protected]>
Signed-off-by Matt Weber <[email protected]>
---
 boot/uboot/uboot.mk | 10 ++++++++++
 1 file changed, 10 insertions(+)
boot/uboot/uboot.mk +10 -0
index eee0dba..d5cfd60 100644
@@ -71,7 +71,17 @@ UBOOT_BIN = u-boot.bin
 UBOOT_BIN_IFT = $(UBOOT_BIN).ift
 endif
 
+# For a 64bit arm architecture, Buildroot uses arm64 terminology
+# and from the top-level Makefile KERNEL_ARCH is set to
+# arm64 which is then passed to the uboot build.
+# This causes a compilation issue as uboot uses the top-level
+# system architecture as it's $(ARCH).  So arch/$(ARCH)/Makefile
+# doesn't work with arm64 unless we adjust the arch.
+ifeq ($(KERNEL_ARCH),arm64)
+UBOOT_ARCH = arm
+else
 UBOOT_ARCH = $(KERNEL_ARCH)
+endif
 
 UBOOT_MAKE_OPTS += \
 	CROSS_COMPILE="$(TARGET_CROSS)" \
Signature · 1 line
1.9.1
#1 lore raw

PATCH v2 1/1 uboot: arm64 arch build support

Matt, Ronak,

Quoted 9 lines · Matt Weber

From: Ronak Desai <[email protected]>

For a 64bit arm architecture, Buildroot uses arm64 terminology and from the top-level Makefile KERNEL_ARCH is set to arm64 which is then passed to the uboot build.
This causes a compilation issue as uboot uses the top-level system architecture as it's $(ARCH). So arch/$(ARCH)/Makefile doesn't work with arm64 unless we adjust the arch.

Quoted 5 lines
Signed-off-by Ronak Desai <[email protected]>
Signed-off-by Matt Weber <[email protected]>
---
 boot/uboot/uboot.mk | 10 ++++++++++
 1 file changed, 10 insertions(+)

Applied with some minor tweaks:

[Thomas:
- rewrap commit message text.
- simplify comment in the code.]

Thanks!

Thomas

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