[coreboot-gerrit] New patch to review for coreboot: libpayload: don't overwrite CFLAGS

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Jul 6 11:14:53 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10804

-gerrit

commit d68d16a79ae323fa33b82974df277676e0e569b3
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Mon Jul 6 09:09:30 2015 +0000

    libpayload: don't overwrite CFLAGS
    
    Makefile already sets it to contain the architecture specific flags,
    don't drop them, but add to that instead.
    
    Change-Id: I147e6480ab2b3c1ee4f4ace511197b4ba94280b8
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
---
 payloads/libpayload/Makefile.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/payloads/libpayload/Makefile.inc b/payloads/libpayload/Makefile.inc
index 05754fe..c8a49fc 100644
--- a/payloads/libpayload/Makefile.inc
+++ b/payloads/libpayload/Makefile.inc
@@ -57,7 +57,7 @@ subdirs-$(CONFIG_LP_LZMA) += liblzma
 
 INCLUDES := -Iinclude -Iinclude/$(ARCHDIR-y) -I$(obj) -include include/kconfig.h
 
-CFLAGS =  $(EXTRA_CFLAGS) $(INCLUDES) -Os -pipe -nostdinc -ggdb3
+CFLAGS +=  $(EXTRA_CFLAGS) $(INCLUDES) -Os -pipe -nostdinc -ggdb3
 CFLAGS += -nostdlib -fno-builtin -ffreestanding -fomit-frame-pointer
 CFLAGS += -ffunction-sections -fdata-sections
 CFLAGS += -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes



More information about the coreboot-gerrit mailing list