On Tue, Aug 25, 2009 at 02:34:22PM -0600, Myles Watson wrote: > On Tue, Aug 25, 2009 at 2:30 PM, Patrick Georgi<[email protected]> wrote: > > Am Dienstag, den 25.08.2009, 14:13 -0600 schrieb Myles Watson: > >> How about this fix? > > I thought about that, but it has the disadvantage that it spreads the > > configuration of the board across two files. > Yes.
Indeed, and I think we should avoid that. The small proposed patch is perfectly fine, IMHO. Acked-by: Uwe Hermann <[email protected]> > I'd like to think that much of the second file will disappear as > CONFIG_USE_DCACHE_RAM and others are moved to their final resting > places. Then it might make sense to move the last few things like the > mainboard strings into the mainboard/<vendor>/Kconfig file. I doubt this will always work, there are many per-board options that will still be required, and they should really be in the board dir, IMHO. > > I'm not sure what the best fix is... > > I guess the third option is live with it being printed twice. Nah, why should we? The fix is easy. Uwe. -- http://www.hermann-uwe.de | http://www.holsham-traders.de http://www.crazy-hacks.org | http://www.unmaintained-free-software.org -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
