[coreboot] [PATCH] Move v3 config variables to Kconfig style

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Thu Feb 14 21:53:44 CET 2008


On 14.02.2008 19:31, Stefan Reinauer wrote:
> Carl-Daniel Hailfinger wrote:
>> v3 still has many places where config variables are tested with #if in
>> v2 style. That fails silently in v3. Fix up the most obvious ones.
>>
>> Signed-off-by: Carl-Daniel Hailfinger
>> <c-d.hailfinger.devel.2006 at gmx.net>
>>   
> While the patch itself is discussable, it is cosmetics and not a fix.
> In fact you are removing the check of the CONFIG_ variable's value,
> and so if for some reason someone sets a defined value to 0, THEN the
> code would fail silently after applying your diff.
>
> Can you please elaborate how any of the lines you touch would cause
> silent failure in v3?

All of them. These values are Kconfig bools and if they are set, they
are set to "y" and not 1.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list