[U-Boot] [PATCH 8/8] arm64: Do not double -mstrict-align option

Masahiro Yamada yamada.m at jp.panasonic.com
Wed Feb 26 06:51:05 CET 2014


Seeing arch/arm/cpu/armv8/config.mk, -mstrict-align option
was added to both PLATFORM_NO_UNALIGNED and PLATFORM_CPPFLAGS.

$(PLATFORM_CPPFLAGS) is given to all compiled objects.
Adding the same option to also PLATFORM_NO_UNALIGNED is
redundant.

For example, common/hush.o was compiled with double -mstrict-align.
You can comfirm it by checking common/.hush.o.cmd file.

Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
Cc: David Feng <fenghua at phytium.com.cn>
Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>
Cc: Tom Rini <trini at ti.com>
---

 arch/arm/cpu/armv8/config.mk | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/cpu/armv8/config.mk b/arch/arm/cpu/armv8/config.mk
index 027a68c..7e59b4d 100644
--- a/arch/arm/cpu/armv8/config.mk
+++ b/arch/arm/cpu/armv8/config.mk
@@ -8,7 +8,6 @@ PLATFORM_RELFLAGS += -fno-common -ffixed-x18
 
 # SEE README.arm-unaligned-accesses
 PF_NO_UNALIGNED := $(call cc-option, -mstrict-align)
-PLATFORM_NO_UNALIGNED := $(PF_NO_UNALIGNED)
 
 PF_CPPFLAGS_ARMV8 := $(call cc-option, -march=armv8-a)
 PLATFORM_CPPFLAGS += $(PF_CPPFLAGS_ARMV8)
-- 
1.8.3.2



More information about the U-Boot mailing list