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:[email protected]] > Sent: Saturday, April 23, 2011 3:58 AM > To: She, Kerry > Cc: [email protected] > Subject: Re: [coreboot] [Patch 2/4] compile sb800 code without agesav5 > > * 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? >
sb800_amdlib32.diff
Description: sb800_amdlib32.diff
-- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
