[lm-sensors] RE: vt8231.c

Roger Lucas roger at planbit.co.uk
Wed Nov 2 10:28:11 CET 2005


Hi Knut,

It looks like it is working on your system too. :-)

Thanks for you help in testing this out.

I posted a revised patch update to the mailing list yesterday - I got a
whole load of change requests from Jean.  Most of these were cosmetic or
minor performance changes, but I would suggest that you update if possible.

The fan is handled correctly, but you have the nasty condition where if you
set a minimum speed that is too low, it is set to zero.  The Via speed
sensor works on period rather than frequency, so there is a 1/x conversion
done in the driver.  If you select a frequency that is too low, then the
conversion to a period creates a period that is too large for the counters
and the resulting illegal result is rejected.  If you want very slow fan
speeds then you must make sure the divider is set high enough so that the
resulting period is within the limits of the device.

Personally, I don't like this.  It isn't really a massive amount of work to
get the driver to automatically either:

1) Automatically select the appropriate divider for the minimum speed that
you have requested (i.e. pick the highest divider ratio that can give the
speed you want range)

Or
2) Select the lowest possible minimum speed if the one you selected is still
too low.

A minimum speed of 0 should always disable the minimum speed detection.
Resetting the minimum speed to zero when you select a non-zero minimum speed
is just plain wrong, however, as it is not obvious exactly what is happening
"behind the scenes".  I spent quite a bit of time debugging the system to
convince myself it was working "correctly".

What are your thoughts on this?

Anyone else have thoughts on this?

- Roger

-----Original Message-----
From: Knut Petersen [mailto:Knut_Petersen at t-online.de] 
Sent: 02 November 2005 08:53
To: Roger Lucas
Subject: Re: vt8231.c

Hi Roger,

>
>
>Attached is a patch file for the 2.6.14 kernel tree.  I have also attached
>my sensors.conf file.
>
>Please could you try this and let me know if it works for you?
>
>

linux:/etc # sensors
vt8231-isa-6000
Adapter: ISA adapter
+12v:     +11.96 V  (min = +11.37 V, max = +12.56 V)  
+5v:       +5.21 V  (min =  +4.73 V, max =  +5.24 V)  
+2.5v:     +2.46 V  (min =  +2.42 V, max =  +2.57 V)  
Vcore1 (1.2v):
           +1.22 V  (min =  +1.14 V, max =  +1.25 V)  
+3.3V:     +3.35 V  (min =  +3.13 V, max =  +3.45 V)  
fan1:     11808 RPM  (min =    0 RPM, div = 1)         
fan2:        0 RPM  (min =    0 RPM, div = 2)         
Proc Temp: +45.1°C  (high =  +268°C, hyst =   -59°C)  
vid:       +1.85 V

eeprom-i2c-0-51
Adapter: SMBus Via Pro adapter at 5000
Memory type:            SDR SDRAM DIMM
Memory size (MB):       512



Voltages:
=======
+5V is within 0.02V with my multimeter,
+12V is about 0.16V low according to my multimeter

Temperature:
=========
The displayed value is believable.

Fans:
====
I connected a fan for tests. It´s a Verax fan with 9 pulses / revolution.

In sensors.conf I used the four dividers 1,2,4,9 instead of the *:

    set fan1_div *
    set fan1_min 1000

Here are the results

1:    fan1:     12024 RPM  (min =    0 RPM, div = 1)
2:    fan1:     11702 RPM  (min =    0 RPM, div = 2)
4:    fan1:     11702 RPM  (min =    0 RPM, div = 4)
8:    fan1:     12603 RPM  (min =  999 RPM, div = 8)
9:    fan1:     11915 RPM  (min =    0 RPM, div = 2)

Obviously the fan1_min are ignored most of the time and dividers are not 
handled correctly.
The datasheet of the fan lists 1500 rpm for a temperature of 30° Celsius 
and about 1200 rpm
for 20° Celsius. So 12024/9 = 1336 rpm is believable.

cu,
 knut





More information about the lm-sensors mailing list