On Tue, Feb 22, 2011 at 4:24 AM, Bao, Zheng <[email protected]> wrote: > The text sections in *.romstage.o are called .text instead of .rom.text. > > The .text can be built in, but the _erom cannot be calculated correctly > > without this patch. Nobody uses _erom currently, so nobody seems cares it. > > > > Signed-off-by: Zheng Bao <[email protected]> > > > > Index: src/arch/x86/init/ldscript_fallback_cbfs.lb > > =================================================================== > > --- src/arch/x86/init/ldscript_fallback_cbfs.lb (revision 6374) > > +++ src/arch/x86/init/ldscript_fallback_cbfs.lb (working copy) > > @@ -35,6 +35,7 @@ > > .rom . : { > > _rom = .; > > *(.rom.text); > > + *(.text); > > *(.rom.data); > > *(.rodata); > > *(.rodata.*);
Zheng, I think that .text should be like .data. and should only be in ramstage. Is something being built into romstage that shouldn't be? Marc -- http://se-eng.com -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
