Re: [lm-sensors] fancontrol EeePC,

["Mildred Ki'Lya" <mildred593@online.fr>]

[-- Attachment #1.1: Type: text/plain, Size: 1920 bytes --]

On 04/25/2010 01:21 PM, Jean Delvare wrote:
> Hi Mildred,
>
> On Sun, 25 Apr 2010 09:35:58 +0200, Mildred Ki'Lya wrote:
>   
>> Hi,
>>
>> I am using fancontrol for my new EeePC, and when fancontrol exit (when I
>> reboot for example) the fans are set to manual mode at full speed.
>>
>> Aparently, the source code of fancontrol first tries to set the fan to safe
>> mode, and if it doesn't work, set it to manual at full speed. Apparently,
>> the EeePC safe mode is the same as automatic mode. So when we write "0" to
>> pwm1_enable, if we read it again, we'll see "2".
>>     
> This is a bug in the eeepc driver. If it doesn't support mode 0, it
> should return -EINVAL when one tries to set it.
>   

Well, I didn't think this was a bug, from what I read, mode 0
corresponds to safe mode which is the automatic mode on the EeePC, and
of course since this is the same as the mode 2 (automatic), when we read
the file back we get 2.

But you certainly know better about this, so I'm not pressing the issue
further.

If I understand how fancontrol works, when the mode 0 doesn't work, it
set the pwm to mnual mode (1) and maximum speed. Do you think that is a
good choice given that for some laptops where the default mode is the
automatic mode (2) ? That would be a perfectly acceptable fallback when
fancontrol isn't in charge.

Do you think there is possibility in the future that fancontrol reset
the fan state to automatic when it quits (when specified in the
configuration) ? I'd be willing to work on a patch if that were the case.


Mildred

-- 
Mildred Ki'Lya
╭───────── ｍｉｌｄｒｅｄ５９３＠ｏｎｌｉｎｅ．ｆｒ ──────────
│ Jabber, GoogleTalk: <mildred@jabber.fr>
│ Website: <http://ki.lya.online.fr>           GPG ID: 9A7D 2E2B
│ Fingerprint: 197C A7E6 645B 4299 6D37 684B 6F9D A8D6 9A7D 2E2B



[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors