Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/80288?usp=email )
Change subject: soc/amd/phoenix/Makefile: only include FSP folder conditionally ......................................................................
soc/amd/phoenix/Makefile: only include FSP folder conditionally
Only add the vendorcode/amd/fsp/phoenix and vendorcode/amd/fsp/common folders to the include search path when the SOC_AMD_PHOENIX_FSP Kconfig option is selected.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I18668ab8578b297c328fdc647c8a95f540ac6272 Reviewed-on: https://review.coreboot.org/c/coreboot/+/80288 Reviewed-by: Varshit Pandya pandyavarshit@gmail.com Reviewed-by: Matt DeVillier matt.devillier@amd.corp-partner.google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/amd/phoenix/Makefile.mk 1 file changed, 3 insertions(+), 0 deletions(-)
Approvals: Matt DeVillier: Looks good to me, approved Varshit Pandya: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/soc/amd/phoenix/Makefile.mk b/src/soc/amd/phoenix/Makefile.mk index feb9b1a..9c775cc 100644 --- a/src/soc/amd/phoenix/Makefile.mk +++ b/src/soc/amd/phoenix/Makefile.mk @@ -42,8 +42,11 @@
CPPFLAGS_common += -I$(src)/soc/amd/phoenix/include CPPFLAGS_common += -I$(src)/soc/amd/phoenix/acpi + +ifeq ($(CONFIG_SOC_AMD_PHOENIX_FSP),y) CPPFLAGS_common += -I$(src)/vendorcode/amd/fsp/phoenix CPPFLAGS_common += -I$(src)/vendorcode/amd/fsp/common +endif
# Building the cbfs image will fail if the offset, aligned to 64 bytes, isn't large enough ifeq ($(CONFIG_CBFS_VERIFICATION),y)