Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Arthur Heymans, Fred Reitberger, Felix Held. Hello Arthur Heymans,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/63054
to review the following change.
Change subject: soc/amd/pi: Use -Wno-pragma-pack ......................................................................
soc/amd/pi: Use -Wno-pragma-pack
Agesa headers extensively use and override pragma pack which fails to compile with clang.
Change-Id: Ib234be536388f41d63c2d26cac4c35881af25930 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/soc/amd/common/pi/Makefile.inc 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/63054/1
diff --git a/src/soc/amd/common/pi/Makefile.inc b/src/soc/amd/common/pi/Makefile.inc index 55fbd95..8646b0b 100644 --- a/src/soc/amd/common/pi/Makefile.inc +++ b/src/soc/amd/common/pi/Makefile.inc @@ -16,4 +16,7 @@ ramstage-y += refcode_loader.c ramstage-y += s3_resume.c
+CFLAGS_x86_32 += -Wno-pragma-pack +CFLAGS_x86_64 += -Wno-pragma-pack + endif # CONFIG_SOC_AMD_PI