All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <guenter.roeck@ericsson.com>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] RFC: nct6776f support in the w83627ehf and fan
Date: Thu, 10 Mar 2011 22:15:07 +0000	[thread overview]
Message-ID: <20110310221507.GB29548@ericsson.com> (raw)
In-Reply-To: <BE6F61DB46364D4B839ED3F50CDFDFAB@portable2>

Hi Robert,

On Thu, Mar 10, 2011 at 03:05:04PM -0500, Robert Kaiser wrote:
> Guenter Roeck schrieb:
> > On Thu, Mar 10, 2011 at 06:53:28AM -0500, Guenter Roeck wrote:
> > [ ... ]
> >>>
> >>> So first question is what chip he has.
> >>>
> >> ... answer is NCT6775F, from Robert's log. It uses the "old" registers for the min
> >> speed settings, so presumably that should be correct. Yet, there must be a reason
> >> for the alarm. Maybe it is the "maximum RPM" register after all - the NCT6775F
> >> has that set of registers as well.
> >>
> > Doesn't look like the "maximum RPM" register is the culprit. I have a system with NCT6775F
> > (not using its fan controls, though). Looks like one can not write into that register, so it
> > must be a status register. Maybe it reports the maximum RPM the chip has seen.
> 
> OK, so here's a bit of shortened output of my tests:
> 
> 
> # echo 5000 > /sys/devices/platform/w83627ehf.656/fan2_min
> # sensors
> [...]
> fan2:                    0 RPM  (min = 5037 RPM, div = 4)
> [...]
> 
> # sensors
> [...]
> fan2:                 1102 RPM  (min = 5113 RPM, div = 8)  ALARM
> [...]
> 
> # sensors
> [...]
> fan2:                 1095 RPM  (min = 5113 RPM, div = 8)  ALARM
> [...]
> 
> # sensors
> [...]
> fan2:                 1061 RPM  (min = 5113 RPM, div = 8)  ALARM
> [...]
> 
> # echo 0 > /sys/devices/platform/w83627ehf.656/fan2_min
> # sensors
> [...]
> fan2:                 1054 RPM  (min =    0 RPM, div = 8)
> [...]
> 
> # sensors
> [...]
> fan2:                 1095 RPM  (min =    0 RPM, div = 8)
> [...]
> 
> # sensors
> [...]
> fan2:                 1061 RPM  (min =    0 RPM, div = 8)
> [...]
> 
> 
> So, in short, it looks like it's correct that this is a minimum, but it 
> might get triggered very early in the cycle or so and when re-setting 
> the alarm due to that re-writing of the minimum, everything is OK.
> 
Looks like it.

Thanks a lot for the testing!

Guenter

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

  parent reply	other threads:[~2011-03-10 22:15 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-06 12:38 [lm-sensors] RFC: nct6776f support in the w83627ehf and fan RPM Ian Dobson
2011-03-06 17:07 ` Guenter Roeck
2011-03-06 17:21 ` Ian Dobson
2011-03-06 18:47 ` Guenter Roeck
2011-03-06 19:40 ` Ian Dobson
2011-03-06 20:05 ` Guenter Roeck
2011-03-07  2:50 ` Robert Kaiser
2011-03-07  5:21 ` Guenter Roeck
2011-03-07 14:59 ` Robert Kaiser
2011-03-07 16:03 ` Guenter Roeck
2011-03-10  0:25 ` Robert Kaiser
2011-03-10  3:45 ` Guenter Roeck
2011-03-10  5:06 ` Guenter Roeck
2011-03-10  5:20 ` [lm-sensors] RFC: nct6776f support in the w83627ehf and fan Ian Dobson
2011-03-10 11:35 ` Guenter Roeck
2011-03-10 11:53 ` Guenter Roeck
2011-03-10 11:57 ` I.dobson
2011-03-10 13:10 ` I.dobson
2011-03-10 17:32 ` Guenter Roeck
2011-03-10 20:05 ` Robert Kaiser
2011-03-10 20:08 ` [lm-sensors] RFC: nct6776f support in the w83627ehf and fan RPM Robert Kaiser
2011-03-10 20:27 ` [lm-sensors] RFC: nct6776f support in the w83627ehf and fan Ian Dobson
2011-03-10 21:03 ` Robert Kaiser
2011-03-10 22:11 ` Guenter Roeck
2011-03-10 22:15 ` Guenter Roeck [this message]
2011-03-10 23:17 ` Robert Kaiser
2011-03-10 23:38 ` Guenter Roeck
2011-03-11  0:04 ` Robert Kaiser
2011-03-11  0:29 ` Guenter Roeck
2011-03-11  0:58 ` Robert Kaiser
2011-03-11  1:07 ` Guenter Roeck
2011-03-11 17:09 ` Robert Kaiser
2011-03-11 17:47 ` Guenter Roeck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110310221507.GB29548@ericsson.com \
    --to=guenter.roeck@ericsson.com \
    --cc=lm-sensors@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.