[coreboot] [patch] superiotool dumps environmental controller registers too

[Carl-Daniel Hailfinger]

On 22.02.2008 23:48, Torsten Duwe wrote:
> I have 2 versions to throw in.
>
> On Friday 22 February 2008, Carl-Daniel Hailfinger wrote:
>
>   
>> It can either mean "Environment Controller"
>> or "Embedded Controller". In the future, superiotool will be able to
>> dump settings of both types of these devices, so I fear the parameter
>> has to be replaced by something like "--environ-dump".
>>     
>
> --env-dump, because I consider that a common abbreviation ("man 1 env").
>   

Because of "man 1 env" and the completely different meaning of it, I'd 
call it "--envctl-dump".


>> Thoughts? We should wait with the commit until there is consensus about
>> the name of the long option.
>>     
>
> OTOH the whole thing is about voltages, fans and temperature sensors, right? 
> Then there's a package called lm-sensors, so why not --sensors 
> or --sensors-dump? "Environment" is also quite ambiguous, and the -s is still 
> available, so --sensors* would be my favourite.
>   

"--sensors-fan-voltage-controller-dump"? Older ITE datasheets call this 
thin a "Fan Controller".

Maybe we should wait for input on the name until monday before this is 
committed.

Regards,
Carl-Daniel

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