it87.c fan_div patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux