libpayload: Move EXTRA_CFLAGS to enable option override
Before this patch EXTRA_CFLAGS were placed before many other options. This made overriding impossible even, when necessary. This patch moves EXTRA_CFLAGS to be placed after original CFLAGS, thus making option overriding possible. Signed-off-by: Jakub Czapiga <jacz@semihalf.com> Change-Id: If8394b151696eee4bd736d2fb1ad340209e05fbb Reviewed-on: https://review.coreboot.org/c/coreboot/+/57181 Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
		
				
					committed by
					
						
						Patrick Georgi
					
				
			
			
				
	
			
			
			
						parent
						
							b8461aab52
						
					
				
				
					commit
					c35659d930
				
			@@ -265,7 +265,7 @@ ifn$(EMPTY)def $(1)-objs_$(2)_template
 | 
				
			|||||||
de$(EMPTY)fine $(1)-objs_$(2)_template
 | 
					de$(EMPTY)fine $(1)-objs_$(2)_template
 | 
				
			||||||
$(obj)/$$(1).$(1).o: $$(1).$(2) $(obj)/libpayload-config.h $(4)
 | 
					$(obj)/$$(1).$(1).o: $$(1).$(2) $(obj)/libpayload-config.h $(4)
 | 
				
			||||||
	@printf "    CC         $$$$(subst $$$$(obj)/,,$$$$(@))\n"
 | 
						@printf "    CC         $$$$(subst $$$$(obj)/,,$$$$(@))\n"
 | 
				
			||||||
	$(CC) $(3) -MMD $$$$(CFLAGS) -c -o $$$$@ $$$$<
 | 
						$(CC) $(3) -MMD $$$$(CFLAGS) $(EXTRA_CFLAGS) -c -o $$$$@ $$$$<
 | 
				
			||||||
en$(EMPTY)def
 | 
					en$(EMPTY)def
 | 
				
			||||||
end$(EMPTY)if
 | 
					end$(EMPTY)if
 | 
				
			||||||
endef
 | 
					endef
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -58,7 +58,7 @@ subdirs-$(CONFIG_LP_LZ4) += liblz4
 | 
				
			|||||||
INCLUDES := -Iinclude -Iinclude/$(ARCHDIR-y) -I$(obj)
 | 
					INCLUDES := -Iinclude -Iinclude/$(ARCHDIR-y) -I$(obj)
 | 
				
			||||||
INCLUDES += -include include/kconfig.h -include include/compiler.h
 | 
					INCLUDES += -include include/kconfig.h -include include/compiler.h
 | 
				
			||||||
 | 
					
 | 
				
			||||||
CFLAGS +=  $(EXTRA_CFLAGS) $(INCLUDES) -Os -pipe -nostdinc -ggdb3
 | 
					CFLAGS += $(INCLUDES) -Os -pipe -nostdinc -ggdb3
 | 
				
			||||||
CFLAGS += -nostdlib -fno-builtin -ffreestanding -fomit-frame-pointer
 | 
					CFLAGS += -nostdlib -fno-builtin -ffreestanding -fomit-frame-pointer
 | 
				
			||||||
CFLAGS += -ffunction-sections -fdata-sections
 | 
					CFLAGS += -ffunction-sections -fdata-sections
 | 
				
			||||||
CFLAGS += -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes -Wvla
 | 
					CFLAGS += -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes -Wvla
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user