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

[Marc Jones]

On Thu, May 5, 2011 at 8:33 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

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



-- 
http://se-eng.com