[lm-sensors] RE: vt8231.c

[Roger Lucas]

Hi Knut,

I'm not sure exactly how the VT8251 generates its alarms.  I'll have to look
into the device spec and see exactly what it does then get back to you.

- Roger

-----Original Message-----
From: Knut Petersen [mailto:Knut_Petersen at t-online.de] 
Sent: 02 November 2005 18:48
To: Roger Lucas
Cc: 'Jean Delvare'; 'LM Sensors'
Subject: Re: [lm-sensors] RE: vt8231.c

Roger Lucas schrieb:

>Hi Jean,
>
>Updated patch attached.  I love it when you inherit loads of code that you
>end up fixing... :-)
>  
>
One problem: As you know, I use a Verax fan with 9 pulses/revolution.

If i use

    compute fan1 @/9 , @*9
    set fan1_min 1
    set fan1_div 1

in sensors.conf, the shortened sensors output is:

     fan1:     1324 RPM  (min =  571 RPM, div = 1) 

and /sys/bus/i2c/devices/9191-6000/fan1_min contains 5140.

But if I use

    compute fan1 @/9 , @*9
    set fan1_min 0
    set fan1_div 1

that results in a sensors output of

   fan1:     1324 RPM  (min =    0 RPM, div = 1)          ALARM

and /sys/bus/i2c/devices/9191-6000/fan1_min contains 0.

The previous patch also behaved wrong for a "set fan1_min 0".

cu,
 Knut