On Thu, May 5, 2011 at 8:33 PM, She, Kerry Kerry.She@amd.com wrote:
Hello,
-----Original Message----- From: Marc Jones [mailto:marcj303@gmail.com] Sent: Friday, May 06, 2011 8:52 AM To: Stefan Reinauer Cc: She, Kerry; coreboot@coreboot.org Subject: Re: [coreboot] [Patch 2/4] compile sb800 code without agesav5
On Fri, Apr 22, 2011 at 1:58 PM, Stefan Reinauer stefan.reinauer@coreboot.org wrote:
- 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?
Kerry,
I think that this is fixed with the last patch from Frank. Can you update this patch and resend.
I have update the patch, sb800 code not use stdinc and sse option. Thanks -- kerry
Acked-by: Marc Jones marcj303@gmail.com