Attention is currently required from: Martin L Roth.
Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74150 )
Change subject: Makefile.inc: introduce all_x86 target ......................................................................
Makefile.inc: introduce all_x86 target
For compilation units that should be built for all stages that run on the x86 cores in a newer AMD SoC, but can't be built for verstage on PSP which is an ARM core, the all target can't be used, since that would result in the compilation unit also being added to the verstage target in the verstage on PSP case. In order to not need to add a compilation unit to bootblock, verstage_x8, romstage and ramstage in separate lines in the makefile, introduce the all_x86 target that adds a file to bootblock, verstage_x86, romstage, postcar, ramstage. The compilation units also need to be added to the postcar stage which is only present on the pre-Zen SoCs to be able to also use the all_x86 target in common AMD code that is also used in those per-Zen SoCs.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I9d0184182b931185990094d0874b49c0b5cb9f7e --- M Makefile.inc 1 file changed, 25 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/74150/1
diff --git a/Makefile.inc b/Makefile.inc index d2d3f88..3b34a99 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -114,6 +114,9 @@ verstage_x86-handler = endif
+$(call add-special-class,all_x86) +all_x86-handler = $(foreach class,bootblock verstage_x86 romstage postcar ramstage,$(eval $(class)-y += $(2))) + # Add dynamic classes for rmodules $(foreach supported_arch,$(ARCH_SUPPORTED), \ $(eval $(call define_class,rmodules_$(supported_arch),$(supported_arch))))