[lm-sensors] it87.c fan_div patch
Jean Delvare
khali at linux-fr.org
Mon Nov 21 11:05:35 CET 2005
Hi Anton,
On 2005-11-21, Anton V. Staaf wrote:
> Hi, I just got lm_sensors (v2.9.2) up and running on my Shuttle
> (SN41G) and am loving it. In setting things up I found what I think is a
> bug in the it87_fan_div setting code. I've included a patch vs. TOT
> CVS. What was happening was that the check to make sure the third fan's
> divisor was either 2 or 8 was using the wrong member of the it87 data
> structure. This was causing the if to always fail and thus always force
> the divisor to 2.
Good catch! You're all correct. We'll commit your patch to lm_sensors
CVS shortly. Amazing this bug was left unnoticed while the driver is
widely used and more than 4 year old.
At least the Linux 2.6 driver isn't affected by this problem.
Thanks,
--
Jean Delvare
More information about the lm-sensors
mailing list