Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-12 Thread Mark M. Hoffman
Hi Jean: * Jean Delvare <[EMAIL PROTECTED]> [2007-08-11 13:57:05 +0200]: > Don't assume that the default bank is 0. For one thing, we don't even > set it to 0 when the driver is loaded, so the initial state might be > different. For another, something (say, the BIOS) might access the chip > and le

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-12 Thread Jean Delvare
Hi Mark, On Sun, 5 Aug 2007 12:21:41 -0400, Mark M. Hoffman wrote: > Author: Mark M. Hoffman <[EMAIL PROTECTED]> > Date: Sun Aug 5 12:19:01 2007 -0400 > > hwmon: read fan_div values during probe > > This patch forces the driver to read the fan divider values during early > init. >

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-12 Thread Stefan Richter
I wrote: > Jean Delvare wrote: >> One remaining mystery is why you did not observe the problem with the >> older kernel. Maybe the ACPI thermal driver was not loaded (or not >> working) back then? > > I found two differences: Gentoo's udev init script autoloads thermal on > boot under 2.6.23-rc2

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-12 Thread Stefan Richter
Jean Delvare wrote: > On Sat, 11 Aug 2007 17:41:39 +0200, Stefan Richter wrote: >> While I test-booted 2.6.22(-rc) yesterday I had a look into the BIOS >> setup. There is a fan speed control based on a temperature threshold, >> separately for CPU fan and case fan. The thresholds are currently 55°

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-12 Thread Jean Delvare
On Sat, 11 Aug 2007 17:48:41 +0200, Stefan Richter wrote: > Jean Delvare wrote: >> So my theory is that something else (BIOS, ACPI?) > > (ksensors?) ACPI if my guess is correct. >> is changing the bank, >> probably to read temperature values which are in banks 1 and 2, causing >> the w83627ehf t

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-12 Thread Jean Delvare
Hi Stefan, On Sat, 11 Aug 2007 17:41:39 +0200, Stefan Richter wrote: > While I test-booted 2.6.22(-rc) yesterday I had a look into the BIOS > setup. There is a fan speed control based on a temperature threshold, > separately for CPU fan and case fan. The thresholds are currently 55°C > and 50°C

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-11 Thread Stefan Richter
> Jean Delvare wrote: >> So my theory is that something else (BIOS, ACPI?) (ksensors?) >> is changing the bank, >> probably to read temperature values which are in banks 1 and 2, causing >> the w83627ehf to get a wrong value for the VBAT register. If I am >> right, then the attached patch should

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-11 Thread Stefan Richter
Jean Delvare wrote: > On Sat, 11 Aug 2007 00:29:36 +0200, Stefan Richter wrote: ... >> The motherboard controls the CPU fan and I believe also the case fan, >> probably based on temperatures. (The manual is buried somewhere and >> MSI's download site is down right in this moment.) > > I would lik

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-11 Thread Jean Delvare
Hi Stefan, On Sat, 11 Aug 2007 00:29:36 +0200, Stefan Richter wrote: > Jean Delvare wrote: > > I just tried 2.6.23-rc2 on a system where I use the w83627ehf hardware > > monitoring driver, and was not able to reproduce the problem you > > described. Fan speeds are reported properly for me. Which I

Re: [lm-sensors] 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-10 Thread David Hubbard
Hi Stefan, (Replying to everyone on the list, sorry!) On 8/10/07, Stefan Richter <[EMAIL PROTECTED]> wrote: > Should I hardwire correct dividers or pulse per rev in sensors.conf or > is the driver supposed to work the correct dividers out --- like it did > before 2.6.23-rc? The dividers are read-

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-10 Thread Stefan Richter
I wrote: > # sensors > w83627ehf-isa-0290 > Adapter: ISA adapter > VCore: +0.95 V (min = +0.00 V, max = +1.74 V) > in1: +12.30 V (min = +1.64 V, max = +3.22 V) ALARM > AVCC: +3.28 V (min = +1.89 V, max = +1.94 V) ALARM > 3VCC: +3.26 V (min = +0.18 V, max = +0.72 V) AL

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-10 Thread Stefan Richter
Jean Delvare wrote: > I just tried 2.6.23-rc2 on a system where I use the w83627ehf hardware > monitoring driver, and was not able to reproduce the problem you > described. Fan speeds are reported properly for me. Which I kind of > expected, as I tested all my w83627ehf patches on this system befor

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-10 Thread Jean Delvare
Hi Stefan, On Sun, 05 Aug 2007 19:13:34 +0200, Stefan Richter wrote: > Mark M. Hoffman wrote: > > Does this patch (against v2.6.23-rc2) fix it? > > > > commit f15c50e703c14ff7d72c3cb34e8e55417476a290 > > Author: Mark M. Hoffman <[EMAIL PROTECTED]> > > Date: Sun Aug 5 12:19:01 2007 -0400 > > >

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-05 Thread Stefan Richter
Mark M. Hoffman wrote: > Does this patch (against v2.6.23-rc2) fix it? > > commit f15c50e703c14ff7d72c3cb34e8e55417476a290 > Author: Mark M. Hoffman <[EMAIL PROTECTED]> > Date: Sun Aug 5 12:19:01 2007 -0400 > > hwmon: read fan_div values during probe > > This patch forces the drive

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-05 Thread Mark M. Hoffman
Hi Stefan: * Stefan Richter <[EMAIL PROTECTED]> [2007-08-05 13:20:48 +0200]: > Now that I booted from 2.6.22-rc5 to 2.6.23-rc2 I noticed that ksensors > displayed 1/16 of the actual speed of the CPU fan (correct is ca. 1400 > RPM under light load) and 0 for the case fan (correct is ca. 480 RPM > u

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-05 Thread Mark M. Hoffman
Hi Stefan: * Stefan Richter <[EMAIL PROTECTED]> [2007-08-05 16:51:07 +0200]: > Mark M. Hoffman wrote: > > It's more likely the patch before that: > > > > hwmon/w83627ehf: Preserve speed reading when changing fan min > > I reverted this patch alone and rebooted, still the same problem. > Afte

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-05 Thread Stefan Richter
Mark M. Hoffman wrote: > It's more likely the patch before that: > > hwmon/w83627ehf: Preserve speed reading when changing fan min I reverted this patch alone and rebooted, still the same problem. After reboot: 0 RPM for case fan, correct CPU fan speed, drops to 1/16 after kmsensors'

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-05 Thread Mark M. Hoffman
Hi Stefan: * Stefan Richter <[EMAIL PROTECTED]> [2007-08-05 13:20:48 +0200]: > Now that I booted from 2.6.22-rc5 to 2.6.23-rc2 I noticed that ksensors > displayed 1/16 of the actual speed of the CPU fan (correct is ca. 1400 > RPM under light load) and 0 for the case fan (correct is ca. 480 RPM > u

Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-05 Thread Stefan Richter
I wrote: > Now that I booted from 2.6.22-rc5 to 2.6.23-rc2 I noticed that ksensors > displayed 1/16 of the actual speed of the CPU fan (correct is ca. 1400 > RPM under light load) and 0 for the case fan (correct is ca. 480 RPM > under light load). > > I reverted patch > hwmon/w83627ehf: No n

2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed

2007-08-05 Thread Stefan Richter
Now that I booted from 2.6.22-rc5 to 2.6.23-rc2 I noticed that ksensors displayed 1/16 of the actual speed of the CPU fan (correct is ca. 1400 RPM under light load) and 0 for the case fan (correct is ca. 480 RPM under light load). I reverted patch hwmon/w83627ehf: No need to initialize fan