[coreboot] [PATCH] continue cleaning up log levels

[Joseph Smith]

On 04/02/2010 06:19 PM, Stefan Reinauer wrote:
> On 4/2/10 7:55 PM, Myles Watson wrote:
>> loglevel.diff:
>> Remove BIOS_EMERG, BIOS_ALERT, BIOS_CRIT, BIOS_NOTICE, BIOS_NEVER
> I am not convinced that's a good idea. But let's hear what other people
> say...
>> I didn't change their numerical values. They were rarely used.
>> BIOS_EMERG, BIOS_ALERT, BIOS_CRIT -> BIOS_ERR
>> BIOS_NOTICE -> BIOS_INFO
> I don't think we want to just remove log levels like that. Let's rather
> fix the messages and choose the right log levels for them.
>
>> BIOS_NEVER -> BIOS_SPEW+1
> The whole intent was to make clear to the reader that this is something
> that's never seen. ;-)
> That doesn't become clear from BIOS_SPEW+1 in my opinion.
>
>
>> loglevel2.diff:
>> BIOS_ERR -> ERROR
>> BIOS_WARNING -> WARNING
>> BIOS_INFO -> INFO
>> BIOS_DEBUG -> DEBUG
>> BIOS_SPEW -> SPEW
>> For 80 character line aficionados, this puts most of the lines back,
>> since printk_debug(...) is about the same length as printk(DEBUG, ...)
> I kind of like the fact that they are prefixed, while I see the point
> that BIOS_ is a bad prefix. What about LOG_? Because, that's what it is,
> log levels.
> I like the idea of not abbreviating ERROR to ERR as the only exception
> to the rule.
>
> Stefan
>
Yes I agree Stefan, lets get that _BIOS_ out of coreboot.

-- 
Thanks,
Joseph Smith
Set-Top-Linux
www.settoplinux.org