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

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 <mhoffman@xxxxxxxxxxxxx>
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.
Otherwise, a call to store_fan_min() could access uninitialized variables.

Alas not; there is no change.
(I applied on vanilla 2.6.23-rc2 and rebooted.)

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 before
submitting them.

Please try using sensors instead of ksensors, and confirm that the
behavior is the same. I'd like to rule out a problem in ksensors
itself. sensors will also report the fan divs, this is a useful
information given the problem you have.

Your original post suggests that the fan speed is supposed to change
depending on the system load? Or temperature? Please describe the
mechanism used to achieve this. Could it be that this mechanism isn't
working properly, and the reported (low) speeds are actually true?

What fan inputs are used by your CPU and system fans? "sensors
-c /dev/null" will tell.

Other than that, I can only ask for the same things Mark already
suggested: compile with HWMON debugging and provide the logs (this will
show what fan div the driver is trying to select), and try bisecting
using git to find out which patch exactly caused the problem.

Thanks,
--
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Relevant Pages

  • Re: [PATCH] I2C fixes for 2.6.10-rc2
    ... I2C: add adm1026 chip driver ... Here is the revised adm1026 driver port for kernel 2.6.10-rc2. ... No way is currently provided to turn on DAC-mediated automatic fan control. ... +static ssize_t show_in(struct device *dev, char *buf, int nr) ...
    (Linux-Kernel)
  • Re: 2.6.23-rc1 regression: hwmon/w83627ehf: wrong fan speed
    ... Fan speeds are reported properly for me. ... driver didn't agree on what the actual divider is. ... So my theory is that something else is changing the bank, ...
    (Linux-Kernel)
  • Re: The #24 Team
    ... they have nothing to hang their heads over, ... As a JG Fan & JJ Fan, ... Congrats to JJ who may become the 4th Greatest Driver of All-Time ... Now you are saying Jeff at 36 is nearing the end of a career. ...
    (rec.autos.sport.nascar)
  • Re: Unification is Here! We are Brothers Again!
    ... IRL car on a road course. ... established driver, in a car you've never driven, at a track that you've ... I will be more than happy to blast Gordon on, but lack of talent isn't ... not mean you can't be a fan of both Nascar and open wheel. ...
    (rec.autos.sport.cart)
  • Re: P5GDC-V Deluxe build
    ... If your install disk is original WinXP, ... fan --that was the easy one! ... should I enter the bios immediately and configure, configure, ... The driver CD is for after the OS install. ...
    (alt.comp.periphs.mainboard.asus)