Hello Stefan
The sb800 cimx code originally has its own lib function code, But these files was removed in the current tree. As for now sb800 cimx code share the Agesa v5 lib function code, but this sharing cause it is not possible to use sb800 cimx wrapper without compile Agesa v5 lib files.
This patch implement the lib function needed by sb800 cimx, So we can use the sb800 cimx wrapper and Agesa v5 wrapper independently.
Thanks
-- kerry
-----Original Message----- From: Stefan Reinauer [mailto:stefan.reinauer@coreboot.org] Sent: Saturday, April 23, 2011 3:58 AM To: She, Kerry Cc: coreboot@coreboot.org Subject: Re: [coreboot] [Patch 2/4] compile sb800 code without agesav5
- She, Kerry Kerry.She@amd.com [110422 05:12]:
Signed-off-by: Kerry She Kerry.she@amd.com
SB800 CIMX code can share the AGESA V5 lib code, some platform only use sb800 cimx code, not use AGESA v5 code, compile the sb800 cimx and AGESA v5 lib code.
Signed-off-by: Kerry She Kerry.she@amd.com
Index: Makefile.inc
--- Makefile.inc (revision 6481) +++ Makefile.inc (working copy) @@ -110,7 +110,9 @@ CFLAGS += -Werror endif ifneq ($(CONFIG_AMD_AGESA),y) -CFLAGS += -nostdinc
- ifneq ($(CONFIG_AMD_CIMX_SB800),y)
- CFLAGS += -nostdinc
- endif
endif CFLAGS += -fno-common -ffreestanding -fno-builtin -fomit-frame-pointer
Why is that needed. Can we instead fix the code, please?