[coreboot] [PATCH]modify "make menuconfig" output

Uwe Hermann 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.
> Yes.

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.


Uwe.
-- 
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 mailing list