From 2a81112abde284ba09020db6afa363169911a7f6 Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Tue, 12 Nov 2013 14:46:07 -0800 Subject: [PATCH] arm: Remove some pointless CFLAGS This patch removes the -ffixed-r8 CFLAG from the coreboot and libpayload Makefiles. This seems to be a relic from U-Boot, which uses that register to keep it's global data structure pointer. There's no reason for us to throw away a perfectly fine register on this already pretty constrained architecture. Also removed a config.h inclusion from the Makefile because that should really be done inside the C files. BUG=None TEST=Nyan still boots. Change-Id: Ia176c0f323c1be07cddf88fa5488788786a27cdf Signed-off-by: Julius Werner Reviewed-on: https://chromium-review.googlesource.com/177110 Reviewed-by: Gabe Black --- payloads/libpayload/arch/arm/Makefile.inc | 2 +- src/arch/arm/Makefile.inc | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/payloads/libpayload/arch/arm/Makefile.inc b/payloads/libpayload/arch/arm/Makefile.inc index cd909bb5a6..decb331496 100644 --- a/payloads/libpayload/arch/arm/Makefile.inc +++ b/payloads/libpayload/arch/arm/Makefile.inc @@ -27,7 +27,7 @@ ## SUCH DAMAGE. ## -CFLAGS += -ffixed-r8 -mfloat-abi=hard -marm -mabi=aapcs-linux +CFLAGS += -mfloat-abi=hard -marm -mabi=aapcs-linux head.o-y += head.S libc-y += main.c sysinfo.c diff --git a/src/arch/arm/Makefile.inc b/src/arch/arm/Makefile.inc index ac4e5c9c3d..a1f0b1168a 100644 --- a/src/arch/arm/Makefile.inc +++ b/src/arch/arm/Makefile.inc @@ -82,7 +82,7 @@ ramstage-srcs += src/mainboard/$(MAINBOARDDIR)/mainboard.c ################################################################################ # Common recipes for all stages -CFLAGS += -ffixed-r8 -mno-unaligned-access +CFLAGS += -mno-unaligned-access $(objcbfs)/%.bin: $(objcbfs)/%.elf @printf " OBJCOPY $(subst $(obj)/,,$(@))\n" @@ -162,7 +162,7 @@ $(objcbfs)/bootblock.debug: $(src)/arch/arm/bootblock.ld $(obj)/ldoptions $$(boo ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y) $(LD) -m armelf_linux_eabi -static -o $@ -L$(obj) $< -T $(src)/arch/arm/bootblock.ld else - $(CC) $(CFLAGS) -nostartfiles -include $(obj)/config.h -static -o $@ -L$(obj) -T $(src)/arch/arm/bootblock.ld -Wl,--start-group $(bootblock-objs) -Wl,--end-group + $(CC) $(CFLAGS) -nostartfiles -static -o $@ -L$(obj) -T $(src)/arch/arm/bootblock.ld -Wl,--start-group $(bootblock-objs) -Wl,--end-group endif ################################################################################