[coreboot] [patch] superiotool dumps environmental controller registers too
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Sat Feb 23 00:21:51 CET 2008
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/
More information about the coreboot
mailing list