Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69662 )
Change subject: soc/amd/common: Don't set Wstack-usage option for clang builds ......................................................................
soc/amd/common: Don't set Wstack-usage option for clang builds
Clang doesn't understand this option. It's currently blowing up builds.
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: I7d8b9c26d3fc861615a8553332ed1070974b751b --- M src/soc/amd/common/psp_verstage/Makefile.inc 1 file changed, 14 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/62/69662/1
diff --git a/src/soc/amd/common/psp_verstage/Makefile.inc b/src/soc/amd/common/psp_verstage/Makefile.inc index fb82ef4..489ac51 100644 --- a/src/soc/amd/common/psp_verstage/Makefile.inc +++ b/src/soc/amd/common/psp_verstage/Makefile.inc @@ -4,8 +4,10 @@ verstage-generic-ccopts += -D__USER_SPACE__ CPPFLAGS_common += -I$(VBOOT_SOURCE)/firmware/2lib/include/
+ifeq ($(CONFIG_COMPILER_GCC),y) # This size should match the size in the linker script. CFLAGS_arm += -Wstack-usage=40960 +endif
verstage-y += boot_dev.c verstage-y += delay.c