diff options
| author | Frank Vibrans <frank.vibrans@amd.com> | 2011-02-14 18:47:37 +0000 |
|---|---|---|
| committer | Marc Jones <marc.jones@amd.com> | 2011-02-14 18:47:37 +0000 |
| commit | 0822ad8b1916e2461ef9e3af83e97ad1fcdac2ab (patch) | |
| tree | 6d63f9a0252d45c7d6b600be9a5aecf7a06b64d4 | |
| parent | 420faca0d0152a70e9855b78e028efe5bd3e3448 (diff) | |
| download | coreboot-0822ad8b1916e2461ef9e3af83e97ad1fcdac2ab.tar.xz | |
This code fixes a number of build issues related to the AMD Agesa code. The particular issues are global variables existing in romstage and the use of GCC intrinsics in the build. The former issue will be addressed shortly, and the latter issue requires community assistance. This code is dependent on the AMD Family 14h mainboard code.
Signed-off-by: Frank Vibrans <frank.vibrans@amd.com>
Acked-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Acked-by: Marc Jones <marcj303@gmail.com>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@6348 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
| -rw-r--r-- | Makefile | 5 | ||||
| -rw-r--r-- | src/arch/x86/init/ldscript_fallback_cbfs.lb | 3 |
2 files changed, 6 insertions, 2 deletions
@@ -300,13 +300,16 @@ INCLUDES += -Isrc/devices/oprom/include # abspath is a workaround for romcc INCLUDES += -include $(abspath $(obj)/config.h) -CFLAGS = $(INCLUDES) -Os -nostdinc -pipe -g +CFLAGS = $(INCLUDES) -Os -pipe -g CFLAGS += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes CFLAGS += -Wwrite-strings -Wredundant-decls -Wno-trigraphs CFLAGS += -Wstrict-aliasing -Wshadow ifeq ($(CONFIG_WARNINGS_ARE_ERRORS),y) CFLAGS += -Werror endif +ifneq ($(CONFIG_AMD_AGESA),y) +CFLAGS += -nostdinc +endif CFLAGS += -fno-common -ffreestanding -fno-builtin -fomit-frame-pointer CBFS_COMPRESS_FLAG:=l diff --git a/src/arch/x86/init/ldscript_fallback_cbfs.lb b/src/arch/x86/init/ldscript_fallback_cbfs.lb index 480fd32d5d..082b21193c 100644 --- a/src/arch/x86/init/ldscript_fallback_cbfs.lb +++ b/src/arch/x86/init/ldscript_fallback_cbfs.lb @@ -49,5 +49,6 @@ SECTIONS *(.comment.*) *(.note.*) } - _bogus = ASSERT((SIZEOF(.bss) + SIZEOF(.data)) == 0, "Do not use global variables in romstage"); +/* _bogus = ASSERT((SIZEOF(.bss) + SIZEOF(.data)) == 0, "Do not use global variables in romstage"); */ + _bogus = ASSERT((SIZEOF(.bss)) == 0, "Do not use global variables in romstage"); } |