:) Excellent. Glad to be of help. -Anton On Mon, 21 Nov 2005, Jean Delvare wrote: > > 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 >
