[coreboot] [Patch 2/4] compile sb800 code without agesav5

[She, Kerry]

> Hello,
> 
> 
> > -----Original Message-----
> > From: Marc Jones [mailto:marcj303 at gmail.com]
> > Sent: Friday, May 06, 2011 8:52 AM
> > To: Stefan Reinauer
> > Cc: She, Kerry; coreboot at 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 at coreboot.org> wrote:
> > > * She, Kerry <Kerry.She at amd.com> [110422 05:12]:
> > >> Signed-off-by: Kerry She <Kerry.she at 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 at 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
Another small patch to make sb800 code works without compile agesav5
thanks

-------------- next part --------------
A non-text attachment was scrubbed...
Name: agesa_wrapper_const.patch
Type: application/octet-stream
Size: 734 bytes
Desc: agesa_wrapper_const.patch
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20110506/f63b383b/attachment.obj>