Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/77420?usp=email )
Change subject: vc/amd: Only pull in Makefiles & dirs that are needed ......................................................................
vc/amd: Only pull in Makefiles & dirs that are needed
This keeps the vc/amd/pi & pi/00670F00 Makefiles from getting pulled into the build when they aren't needed.
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: If600c78c2ba74dd03cf493586dae037b96b7d623 Reviewed-on: https://review.coreboot.org/c/coreboot/+/77420 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Elyes Haouas ehaouas@noos.fr Reviewed-by: Frans Hendriks fhendriks@eltan.com Reviewed-by: Felix Singer service+coreboot-gerrit@felixsinger.de --- M src/vendorcode/amd/Makefile.inc M src/vendorcode/amd/pi/Makefile.inc 2 files changed, 6 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Elyes Haouas: Looks good to me, approved Felix Singer: Looks good to me, approved Frans Hendriks: Looks good to me, approved
diff --git a/src/vendorcode/amd/Makefile.inc b/src/vendorcode/amd/Makefile.inc index ddeb2a4..3210134 100644 --- a/src/vendorcode/amd/Makefile.inc +++ b/src/vendorcode/amd/Makefile.inc @@ -1 +1,2 @@ -subdirs-y += pi +subdirs-$(CONFIG_SOC_AMD_STONEYRIDGE) += pi +subdirs-$(CONFIG_CPU_AMD_PI_00730F01) += pi diff --git a/src/vendorcode/amd/pi/Makefile.inc b/src/vendorcode/amd/pi/Makefile.inc index b00e38b..a5768ea 100644 --- a/src/vendorcode/amd/pi/Makefile.inc +++ b/src/vendorcode/amd/pi/Makefile.inc @@ -28,9 +28,10 @@ # #*****************************************************************************
+ifeq ($(CONFIG_SOC_AMD_STONEYRIDGE),y) + subdirs-y += 00670F00
-ifeq ($(CONFIG_SOC_AMD_STONEYRIDGE),y) # AGESA V5 Files
AGESA_ROOT = $(top)/$(call strip_quotes,$(CONFIG_AGESA_BINARY_PI_VENDORCODE_PATH)) @@ -60,6 +61,8 @@ ifeq ($(CONFIG_CPU_AMD_PI_00730F01),y) # AGESA V5 Files
+subdirs-y += 00730F01 + AGESA_ROOT = $(call strip_quotes,$(CONFIG_AGESA_BINARY_PI_VENDORCODE_PATH))
AGESA_INC = -I$(obj) -I$(src)/vendorcode/amd/include