[coreboot] [PATCH]modify "make menuconfig" output
uwe at hermann-uwe.de
Wed Aug 26 16:35:46 CEST 2009
On Tue, Aug 25, 2009 at 02:34:22PM -0600, Myles Watson wrote:
> On Tue, Aug 25, 2009 at 2:30 PM, Patrick Georgi<patrick at georgi-clan.de> 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.
Indeed, and I think we should avoid that. The small proposed patch
is perfectly fine, IMHO.
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
> 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.
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
More information about the coreboot