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

She, Kerry Kerry.She at amd.com
Mon Apr 25 13:57:58 CEST 2011


Hello Stefan

The sb800 cimx code originally has its own lib function code,
But these files was removed in the current tree. 
As for now sb800 cimx code share the Agesa v5 lib function code,
but this sharing cause it is not possible to use sb800 cimx wrapper without compile Agesa v5 lib files.

This patch implement the lib function needed by sb800 cimx,
So we can use the sb800 cimx wrapper and Agesa v5 wrapper independently.  

Thanks

--
kerry

> -----Original Message-----
> From: Stefan Reinauer [mailto:stefan.reinauer at coreboot.org]
> Sent: Saturday, April 23, 2011 3:58 AM
> To: She, Kerry
> Cc: coreboot at coreboot.org
> Subject: Re: [coreboot] [Patch 2/4] compile sb800 code without agesav5
> 
> * 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?
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: sb800_amdlib32.diff
Type: application/octet-stream
Size: 3625 bytes
Desc: sb800_amdlib32.diff
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20110425/c88f5211/attachment.obj>


More information about the coreboot mailing list