[coreboot] [PATCH]es: Various changes from the kconfig branch

Patrick Georgi patrick at georgi-clan.de
Wed Aug 5 14:25:19 CEST 2009


Carl-Daniel Hailfinger schrieb:
> Hi Patrick,
>
> On 05.08.2009 13:04, Patrick Georgi wrote:
>   
>> as preparation for the kconfig merge, here are some changes from the
>> kconfig tree that are unrelated to kconfig itself.
>>
>> 20090805-1-ttys0-rename
>> Takes care of the different CONFIG_* names of the variable containing
>> the serial I/O port
>>   
>>     
>
> There should only be one config variable for this. Wouldn't a global
> search/replace be able to standardize on either CONFIG_TTYS0_BASE or
> CONFIG_SERIAL_IOBASE?
>   
I _think_ the CONFIG_SERIAL_IOBASE was introduced in the kconfig stuff
to be more similar to v3. Once we decide on one or the other, I can
replace the respective occurrences.
I don't care much and would go for CONFIG_TTYS0_BASE as that is what
exists in the codebase. I'll prepare a patch.

>> 20090805-2-config-prefixes
>> Some more CONFIG_* prefixes that were missing
>>   
>>     
>
> I don't particularly like the CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL
> #define, but since that code already looks like this and you only added
> a prefix, it is
> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>   
Maybe we can get rid of this eventually, but for now this gets in. One
battle at a time.
r4497

>> 20090805-3-empty-files
>> These changes exist mostly to make diff happy, which ignores empty files
>> (or rather: Make Patrick happy when he's using diff). Maybe the right
>> way would be to delete those files.
>>   
>>     
>
> If deleting these files doesn't break abuild, please delete them. For
> the deletion, you can use my ack.
>   
It will require some more work, as these empty files are still included
at some places I think. I'll keep this open for discussion.


Thanks,
Patrick




More information about the coreboot mailing list