[coreboot] [PATCH] Make hidden configuration flags visible (i945)
Myles Watson
mylesgw at gmail.com
Fri Oct 1 00:35:06 CEST 2010
On Thu, Sep 30, 2010 at 4:31 PM, Patrick Georgi <patrick at georgi-clan.de> wrote:
> Am 01.10.2010 00:24, schrieb Myles Watson:
>> +choice
>> + prompt "Chipset variant"
>> + default I945GM
>> + depends on NORTHBRIDGE_INTEL_I945
>> + help
>> + Different i945 variants require slightly different setup.
>> +
>> +config I945GM
>> + bool "i945GM (Mobile) chipset"
>> +
>> +config I945GC
>> + bool "i945GC chipset"
>> +
>> +endchoice
>>
>> I don't think this should be exposed to the user. A board Kconfig
>> should select it, but there shouldn't be a menu prompt for it.
> How to make it invisible, simply by removing the "prompt"? My primary
> interest was in using choice to make sure Kconfig allows only either of
> the values.
If that works, it's fine with me. Otherwise, if there's only these
two variants, it could be collapsed to a single Kconfig option. I
just think it's good to avoid options that are never correct in
user-visible Kconfig.
Thanks,
Myles
More information about the coreboot
mailing list