Hi Marc, I am working on porting coreboot to qemu-arm (versatilepb) mainboard, and after I finish it, we can commit it together with this patch. But now, the most important thing is changing the CBFS structure.
On Tue, May 24, 2011 at 3:54 AM, Marc Jones <[email protected]> wrote: > Hi Hamo, > > Thanks for the patch. I know you need this for your ARM development, > but we should wait for an ARM target before committing this. > > This fundamentally changes how .xcompile is used and included by > coreboot. I think that I would prefer that the xcompile script accept > a flag for using the ARM toolchain over the x86 tools and still set > the compiler variables. > > Marc > > > > Be careful with whitespace changes. Try not to put to many in with > code changes. Thanks. I will care about it next time. > > > I think that these makefile changes shouldn't be needed as long as > coreboot is including the correct settings from the .xcompile file. > makefile uses CC(AS, LD and others) as the default compiler, but now, xcompile generated CC_ARCH. So we need to bind the right CC_ARCH variable(and others of cause) to CC. > Marc > -- > http://se-eng.com > -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
