[PATCH 2.6.12-rc2-mm3] i2c: modify lm87 to use auto fan_div
grant_lkml at dodo.com.au
Sun Apr 17 15:34:15 CEST 2005
On Sun, 17 Apr 2005 15:17:13 +0200, Jean Delvare <khali at linux-fr.org> wrote:
>> Something has to take a hit, and fan limit resolution is it, I've
>> already dropped the chop point from 192 down to 128, then check
>> if fan > 192 and do one adjustment in the set_fan_min.
>What exactly are you trying to achieve? 192 seemed to be a good
>equilibrium between range margin and resolution.
Exploring, it did nothing good for us, back up at 192 :)
>> Rest of adjustment in measurement side, not bidirectional until we
>> see a need?
>Absolutely. Note that this guarantees a "stable" divider, while
>bidirectional could result in endless divider change, which I would like
>to avoid, because divider change is somewhat expensive.
Okay, what I have so far is fan_min set to zero, disable fan alarm,
no touch divider.
fan_min set too low, set to lowest value (div = 8, fan_min = 254),
this indicates to the user the lowest limit value for adm9240: 664.
Since their fan is (mine was) running alarm not asserted, alarm will
be asserted if fan speed goes below min operating point --> correct
fan_min set too high? 50000 -> fan_min, displayed back as something
like 42000 and alarm asserted :) Takes a few measurement cycles to
recover fan speed display... that's what you wanted?
Not quite right yet...
The rules seem to be don't auto-adjust fan clock divider unless
fan_min > 192, speed == 255 and div < max_div...
Then my brain stopped :) And I just shut down the other machines.
More information about the lm-sensors