Krystian Hebel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/36976 )
Change subject: vendorcode/amd/pi/Makefile.inc: remove -fno-zero-initialized-in-bss ......................................................................
vendorcode/amd/pi/Makefile.inc: remove -fno-zero-initialized-in-bss
This fixes issue that became visible after implementing post-CAR stage on top of `340e4b80904f lib/cbmem_top: Add a common cbmem_top implementation`. Compilation error was:
Forbidden global variables in romstage: ffffff00 d top.2205
Signed-off-by: Krystian Hebel krystian.hebel@3mdeb.com Change-Id: I088ac824f9b66387843ae5810fd2c75a8b16d9db --- M src/vendorcode/amd/pi/Makefile.inc 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/76/36976/1
diff --git a/src/vendorcode/amd/pi/Makefile.inc b/src/vendorcode/amd/pi/Makefile.inc index 4e8787b..90714aa 100644 --- a/src/vendorcode/amd/pi/Makefile.inc +++ b/src/vendorcode/amd/pi/Makefile.inc @@ -68,7 +68,7 @@ AGESA_INC += -I$(src)/commonlib/include AGESA_INC += -I$(VBOOT_SOURCE)/firmware/include
-AGESA_CFLAGS += -march=amdfam10 -mno-3dnow -fno-zero-initialized-in-bss +AGESA_CFLAGS += -march=amdfam10 -mno-3dnow AGESA_CFLAGS += -fno-strict-aliasing -D__LIBAGESA__ CFLAGS_x86_32 += $(AGESA_CFLAGS) CFLAGS_x86_64 += $(AGESA_CFLAGS)