On Thu, May 5, 2011 at 8:43 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 > Another small patch to make sb800 code works without compile agesav5 > thanks > >
Acked-by: Marc Jones <[email protected]> -- http://se-eng.com -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
