build: CPPFLAGS is more common than INCLUDES
Rename INCLUDES to CPPFLAGS since the latter is more commonly used for preprocessor options. Change-Id: I522bb01c44856d0eccf221fa43d2d644bdf01d69 Signed-off-by: Patrick Georgi <patrick@georgi-clan.de> Reviewed-on: http://review.coreboot.org/5764 Tested-by: build bot (Jenkins) Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
This commit is contained in:
parent
c2a0b7d990
commit
98f49d2823
|
@ -214,12 +214,11 @@ ifneq ($(CONFIG_LOCALVERSION),"")
|
||||||
COREBOOT_EXTRA_VERSION := -$(call strip_quotes,$(CONFIG_LOCALVERSION))
|
COREBOOT_EXTRA_VERSION := -$(call strip_quotes,$(CONFIG_LOCALVERSION))
|
||||||
endif
|
endif
|
||||||
|
|
||||||
INCLUDES := -Isrc -Isrc/include -I$(obj)
|
CPPFLAGS := -Isrc -Isrc/include -I$(obj)
|
||||||
INCLUDES += -Isrc/device/oprom/include
|
CPPFLAGS += -Isrc/device/oprom/include
|
||||||
# abspath is a workaround for romcc
|
CPPFLAGS += -include $(src)/include/kconfig.h
|
||||||
INCLUDES += -include $(src)/include/kconfig.h
|
|
||||||
|
|
||||||
CFLAGS_common = $(INCLUDES) -Os -pipe -g -nostdinc
|
CFLAGS_common = $(CPPFLAGS) -Os -pipe -g -nostdinc
|
||||||
CFLAGS_common += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes
|
CFLAGS_common += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes
|
||||||
CFLAGS_common += -Wwrite-strings -Wredundant-decls -Wno-trigraphs
|
CFLAGS_common += -Wwrite-strings -Wredundant-decls -Wno-trigraphs
|
||||||
CFLAGS_common += -Wstrict-aliasing -Wshadow
|
CFLAGS_common += -Wstrict-aliasing -Wshadow
|
||||||
|
|
|
@ -41,7 +41,7 @@ stages_o = $(obj)/arch/armv7/stages.o
|
||||||
|
|
||||||
$(stages_o): $(stages_c) $(obj)/config.h
|
$(stages_o): $(stages_c) $(obj)/config.h
|
||||||
@printf " CC $(subst $(obj)/,,$(@))\n"
|
@printf " CC $(subst $(obj)/,,$(@))\n"
|
||||||
$(CC_armv7) -I. $(INCLUDES) $(INCLUDES_armv7) -c -o $@ $< -marm
|
$(CC_armv7) -I. $(CPPFLAGS) $(CPPFLAGS_armv7) -c -o $@ $< -marm
|
||||||
|
|
||||||
endif # CONFIG_ARCH_ARMV7
|
endif # CONFIG_ARCH_ARMV7
|
||||||
|
|
||||||
|
@ -86,10 +86,10 @@ $(objgenerated)/bootblock.s: $(objgenerated)/bootblock_inc.S $(obj)/config.h $(o
|
||||||
|
|
||||||
$(objgenerated)/bootblock.inc: $(src)/arch/armv7/$(subst ",,$(CONFIG_BOOTBLOCK_SOURCE)) $(bootblock_custom) $(OPTION_TABLE_H) $(obj)/config.h
|
$(objgenerated)/bootblock.inc: $(src)/arch/armv7/$(subst ",,$(CONFIG_BOOTBLOCK_SOURCE)) $(bootblock_custom) $(OPTION_TABLE_H) $(obj)/config.h
|
||||||
@printf " CC $(subst $(obj)/,,$(@))\n"
|
@printf " CC $(subst $(obj)/,,$(@))\n"
|
||||||
$(CC_bootblock) $(bootblock-c-ccopts) $(INCLUDES) $(INCLUDES_bootblock) -MM \
|
$(CC_bootblock) $(bootblock-c-ccopts) $(CPPFLAGS) $(CPPFLAGS_bootblock) -MM \
|
||||||
-MT$(objgenerated)/bootblock.inc \
|
-MT$(objgenerated)/bootblock.inc \
|
||||||
$< > $(objgenerated)/bootblock.inc.d
|
$< > $(objgenerated)/bootblock.inc.d
|
||||||
$(CC_bootblock) $(bootblock-c-ccopts) -c -S $(CFLAGS_bootblock) -I. $(INCLUDES) $(INCLUDES_bootblock) $< -o $@
|
$(CC_bootblock) $(bootblock-c-ccopts) -c -S $(CFLAGS_bootblock) -I. $(CPPFLAGS) $(CPPFLAGS_bootblock) $< -o $@
|
||||||
|
|
||||||
$(objcbfs)/bootblock.debug: $(objgenerated)/bootblock.o $(objgenerated)/bootblock.ld $$(bootblock-objs) $(stages) $(obj)/config.h
|
$(objcbfs)/bootblock.debug: $(objgenerated)/bootblock.o $(objgenerated)/bootblock.ld $$(bootblock-objs) $(stages) $(obj)/config.h
|
||||||
@printf " LINK $(subst $(obj)/,,$(@))\n"
|
@printf " LINK $(subst $(obj)/,,$(@))\n"
|
||||||
|
|
|
@ -123,9 +123,9 @@ $(objgenerated)/bootblock.s: $(objgenerated)/bootblock_inc.S $(obj)/config.h $(o
|
||||||
|
|
||||||
$(objgenerated)/bootblock.inc: $(src)/arch/x86/init/$(subst ",,$(CONFIG_BOOTBLOCK_SOURCE)) $(objutil)/romcc/romcc $(OPTION_TABLE_H)
|
$(objgenerated)/bootblock.inc: $(src)/arch/x86/init/$(subst ",,$(CONFIG_BOOTBLOCK_SOURCE)) $(objutil)/romcc/romcc $(OPTION_TABLE_H)
|
||||||
@printf " ROMCC $(subst $(obj)/,,$(@))\n"
|
@printf " ROMCC $(subst $(obj)/,,$(@))\n"
|
||||||
$(CC_bootblock) $(INCLUDES) $(INCLUDES_bootblock) -MM -MT$(objgenerated)/bootblock.inc \
|
$(CC_bootblock) $(CPPFLAGS) $(CPPFLAGS_bootblock) -MM -MT$(objgenerated)/bootblock.inc \
|
||||||
$< > $(objgenerated)/bootblock.inc.d
|
$< > $(objgenerated)/bootblock.inc.d
|
||||||
$(ROMCC) -c -S $(bootblock_romccflags) -I. $(INCLUDES) $(INCLUDES_bootblock) $< -o $@
|
$(ROMCC) -c -S $(bootblock_romccflags) -I. $(CPPFLAGS) $(CPPFLAGS_bootblock) $< -o $@
|
||||||
|
|
||||||
$(objcbfs)/bootblock.debug: $(objgenerated)/bootblock.o $(objgenerated)/bootblock.ld
|
$(objcbfs)/bootblock.debug: $(objgenerated)/bootblock.o $(objgenerated)/bootblock.ld
|
||||||
@printf " LINK $(subst $(obj)/,,$(@))\n"
|
@printf " LINK $(subst $(obj)/,,$(@))\n"
|
||||||
|
@ -188,7 +188,7 @@ $(objcbfs)/romstage_%.elf: $(objcbfs)/romstage_%.debug
|
||||||
|
|
||||||
$(obj)/mainboard/$(MAINBOARDDIR)/romstage.inc: $(src)/mainboard/$(MAINBOARDDIR)/romstage.c $(objutil)/romcc/romcc $(OPTION_TABLE_H) $(obj)/build.h $(obj)/config.h
|
$(obj)/mainboard/$(MAINBOARDDIR)/romstage.inc: $(src)/mainboard/$(MAINBOARDDIR)/romstage.c $(objutil)/romcc/romcc $(OPTION_TABLE_H) $(obj)/build.h $(obj)/config.h
|
||||||
printf " ROMCC romstage.inc\n"
|
printf " ROMCC romstage.inc\n"
|
||||||
$(ROMCC) -c -S $(ROMCCFLAGS) -D__PRE_RAM__ -I. $(INCLUDES) $(INCLUDES_romstage) $< -o $@
|
$(ROMCC) -c -S $(ROMCCFLAGS) -D__PRE_RAM__ -I. $(CPPFLAGS) $(CPPFLAGS_romstage) $< -o $@
|
||||||
else
|
else
|
||||||
|
|
||||||
$(obj)/mainboard/$(MAINBOARDDIR)/romstage.pre.inc: $(src)/mainboard/$(MAINBOARDDIR)/romstage.c $(OPTION_TABLE_H) $(obj)/build.h $(obj)/config.h
|
$(obj)/mainboard/$(MAINBOARDDIR)/romstage.pre.inc: $(src)/mainboard/$(MAINBOARDDIR)/romstage.c $(OPTION_TABLE_H) $(obj)/build.h $(obj)/config.h
|
||||||
|
@ -256,7 +256,7 @@ $(objgenerated)/crt0.romstage.o: $(objgenerated)/crt0.s
|
||||||
|
|
||||||
$(objgenerated)/crt0.s: $(objgenerated)/crt0.romstage.S $(obj)/config.h $(obj)/build.h
|
$(objgenerated)/crt0.s: $(objgenerated)/crt0.romstage.S $(obj)/config.h $(obj)/build.h
|
||||||
@printf " CC $(subst $(obj)/,,$(@))\n"
|
@printf " CC $(subst $(obj)/,,$(@))\n"
|
||||||
$(CC_romstage) $(INCLUDES) -MMD -x assembler-with-cpp -E -I$(src)/include -I$(src)/arch/x86/include -I$(obj) -include $(obj)/config.h -include $(obj)/build.h -I. -I$(src) $< -o $@
|
$(CC_romstage) $(CPPFLAGS) -MMD -x assembler-with-cpp -E -I$(src)/include -I$(src)/arch/x86/include -I$(obj) -include $(obj)/config.h -include $(obj)/build.h -I. -I$(src) $< -o $@
|
||||||
|
|
||||||
endif # CONFIG_ARCH_ROMSTAGE_X86_32
|
endif # CONFIG_ARCH_ROMSTAGE_X86_32
|
||||||
|
|
||||||
|
|
|
@ -9,6 +9,6 @@ cpu_microcode-$(CONFIG_CPU_MICROCODE_CBFS_GENERATE) += microcode_blob.c
|
||||||
|
|
||||||
ifneq ($(CONFIG_MICROCODE_INCLUDE_PATH),)
|
ifneq ($(CONFIG_MICROCODE_INCLUDE_PATH),)
|
||||||
ifneq ($(wildcard $(shell realpath -L "$(top)/$(CONFIG_MICROCODE_INCLUDE_PATH)")),)
|
ifneq ($(wildcard $(shell realpath -L "$(top)/$(CONFIG_MICROCODE_INCLUDE_PATH)")),)
|
||||||
INCLUDES += -I$(CONFIG_MICROCODE_INCLUDE_PATH)
|
CPPFLAGS += -I$(CONFIG_MICROCODE_INCLUDE_PATH)
|
||||||
endif
|
endif
|
||||||
endif
|
endif
|
||||||
|
|
|
@ -22,7 +22,7 @@ romstage-y += fsp_util.c
|
||||||
ramstage-$(CONFIG_ENABLE_MRC_CACHE) += fastboot_cache.c
|
ramstage-$(CONFIG_ENABLE_MRC_CACHE) += fastboot_cache.c
|
||||||
romstage-$(CONFIG_ENABLE_MRC_CACHE) += fastboot_cache.c
|
romstage-$(CONFIG_ENABLE_MRC_CACHE) += fastboot_cache.c
|
||||||
|
|
||||||
INCLUDES += -Isrc/drivers/intel/fsp
|
CPPFLAGS += -Isrc/drivers/intel/fsp
|
||||||
|
|
||||||
ifeq ($(CONFIG_USE_GENERIC_FSP_CAR_INC),y)
|
ifeq ($(CONFIG_USE_GENERIC_FSP_CAR_INC),y)
|
||||||
cpu_incs += $(src)/drivers/intel/fsp/cache_as_ram.inc
|
cpu_incs += $(src)/drivers/intel/fsp/cache_as_ram.inc
|
||||||
|
|
|
@ -32,6 +32,6 @@ romstage-y += ../../../arch/x86/lib/walkcbfs.S
|
||||||
smm-$(CONFIG_HAVE_SMI_HANDLER) += udelay.c
|
smm-$(CONFIG_HAVE_SMI_HANDLER) += udelay.c
|
||||||
smm-$(CONFIG_HAVE_SMI_HANDLER) += finalize.c
|
smm-$(CONFIG_HAVE_SMI_HANDLER) += finalize.c
|
||||||
|
|
||||||
INCLUDES += -I$(src)/northbridge/intel/fsp_sandybridge/fsp
|
CPPFLAGS += -I$(src)/northbridge/intel/fsp_sandybridge/fsp
|
||||||
|
|
||||||
$(obj)/northbridge/intel/fsp_sandybridge/acpi.ramstage.o : $(obj)/build.h
|
$(obj)/northbridge/intel/fsp_sandybridge/acpi.ramstage.o : $(obj)/build.h
|
||||||
|
|
|
@ -53,7 +53,7 @@ ramstage-$(CONFIG_ELOG) += elog.c
|
||||||
# Remove as ramstage gets fleshed out
|
# Remove as ramstage gets fleshed out
|
||||||
ramstage-y += placeholders.c
|
ramstage-y += placeholders.c
|
||||||
|
|
||||||
INCLUDES += -Isrc/soc/intel/baytrail/
|
CPPFLAGS += -Isrc/soc/intel/baytrail/
|
||||||
|
|
||||||
# Run an intermediate step when producing coreboot.rom
|
# Run an intermediate step when producing coreboot.rom
|
||||||
# that adds additional components to the final firmware
|
# that adds additional components to the final firmware
|
||||||
|
|
|
@ -70,4 +70,4 @@ endif
|
||||||
|
|
||||||
PHONY += bd82x6x_add_me
|
PHONY += bd82x6x_add_me
|
||||||
|
|
||||||
INCLUDES += -I$(src)/southbridge/intel/fsp_bd82x6x
|
CPPFLAGS += -I$(src)/southbridge/intel/fsp_bd82x6x
|
||||||
|
|
|
@ -48,8 +48,7 @@ VB_FIRMWARE_ARCH := $(ARCHDIR-$(ARCH-ROMSTAGE-y))
|
||||||
VB_SOURCE := vboot_reference
|
VB_SOURCE := vboot_reference
|
||||||
|
|
||||||
# Add the vboot include paths.
|
# Add the vboot include paths.
|
||||||
VB_INCLUDES += -I$(VB_SOURCE)/firmware/include
|
CPPFLAGS += -I$(VB_SOURCE)/firmware/include
|
||||||
INCLUDES += $(VB_INCLUDES)
|
|
||||||
|
|
||||||
VBOOT_STUB_ELF = $(obj)/vendorcode/google/chromeos/vbootstub.elf
|
VBOOT_STUB_ELF = $(obj)/vendorcode/google/chromeos/vbootstub.elf
|
||||||
VBOOT_STUB = $(VBOOT_STUB_ELF).rmod
|
VBOOT_STUB = $(VBOOT_STUB_ELF).rmod
|
||||||
|
|
|
@ -38,7 +38,7 @@ CFLAGS_armv7 = \
|
||||||
|
|
||||||
toolchain_to_dir = \
|
toolchain_to_dir = \
|
||||||
$(foreach arch,$(ARCH_SUPPORTED),\
|
$(foreach arch,$(ARCH_SUPPORTED),\
|
||||||
$(eval INCLUDES_$(ARCH_TO_TOOLCHAIN_$(arch)) = \
|
$(eval CPPFLAGS_$(ARCH_TO_TOOLCHAIN_$(arch)) = \
|
||||||
-Isrc/arch/$(ARCHDIR-$(ARCH_TO_TOOLCHAIN_$(arch)))/include))
|
-Isrc/arch/$(ARCHDIR-$(ARCH_TO_TOOLCHAIN_$(arch)))/include))
|
||||||
|
|
||||||
# set_stage_toolchain: Decides the toolchain to be used by every stage
|
# set_stage_toolchain: Decides the toolchain to be used by every stage
|
||||||
|
@ -64,8 +64,8 @@ OBJCOPY_$(1) := $(OBJCOPY_$(2))
|
||||||
OBJDUMP_$(1) := $(OBJDUMP_$(2))
|
OBJDUMP_$(1) := $(OBJDUMP_$(2))
|
||||||
STRIP_$(1) := $(STRIP_$(2))
|
STRIP_$(1) := $(STRIP_$(2))
|
||||||
READELF_$(1) := $(READELF_$(2))
|
READELF_$(1) := $(READELF_$(2))
|
||||||
INCLUDES_$(1) = -Isrc/arch/$(ARCHDIR-$(2))/include
|
CPPFLAGS_$(1) = -Isrc/arch/$(ARCHDIR-$(2))/include
|
||||||
CFLAGS_$(1) = $$(CFLAGS_common) $$(INCLUDES_$(1)) $(CFLAGS_$(2))
|
CFLAGS_$(1) = $$(CFLAGS_common) $$(CPPFLAGS_$(1)) $(CFLAGS_$(2))
|
||||||
LIBGCC_FILE_NAME_$(1) = $(shell [ -r `$(CC_$(2)) -print-libgcc-file-name` ] && \
|
LIBGCC_FILE_NAME_$(1) = $(shell [ -r `$(CC_$(2)) -print-libgcc-file-name` ] && \
|
||||||
$(CC_$(2)) -print-libgcc-file-name)
|
$(CC_$(2)) -print-libgcc-file-name)
|
||||||
endef
|
endef
|
||||||
|
|
Loading…
Reference in New Issue