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

[Marc Jones]

On Thu, May 5, 2011 at 8:43 PM, She, Kerry <Kerry.She at amd.com> wrote:
>> 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
>
>

Acked-by: Marc Jones <marcj303 at gmail.com>

-- 
http://se-eng.com