On Thu, May 5, 2011 at 8:33 PM, She, Kerry <[email protected]> wrote: > Hello, > > >> -----Original Message----- >> From: Marc Jones [mailto:[email protected]] >> Sent: Friday, May 06, 2011 8:52 AM >> To: Stefan Reinauer >> Cc: She, Kerry; [email protected] >> Subject: Re: [coreboot] [Patch 2/4] compile sb800 code without agesav5 >> >> On Fri, Apr 22, 2011 at 1:58 PM, Stefan Reinauer >> <[email protected]> wrote: >> > * She, Kerry <[email protected]> [110422 05:12]: >> >> Signed-off-by: Kerry She <[email protected]> >> >> >> >> 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 <[email protected]> >> >> >> >> 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 <[email protected]> -- http://se-eng.com -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
