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] W83667HG-B testing
Date: Wed, 21 Jul 2010 01:40:17 +0000	[thread overview]
Message-ID: <20100721014017.GA29955@ericsson.com> (raw)
In-Reply-To: <4C42ED6C.9030903@preclik.de>

On Tue, Jul 20, 2010 at 04:51:04PM -0400, Jean Delvare wrote:
> On Tue, 20 Jul 2010 13:41:42 -0700, Guenter Roeck wrote:
> > On Tue, 2010-07-20 at 02:54 -0400, Jean Delvare wrote:
> > > On Mon, 19 Jul 2010 18:24:16 -0700, Guenter Roeck wrote:
> > > > On Sun, Jul 18, 2010 at 08:02:52AM -0400, Tobias Preclik wrote:
> > > > > Hi,
> > > > > 
> > > > > my Zotac board H55-ITX features the W83667HG-B chip. I downloaded and
> > > > > compiled the standalone kernel module today. Here's the sensors output:
> > > > > 
> > > > > The module loads fine:
> > > > > 
> > > > > # modprobe w83627ehf
> > > > > # tail -n 1 /var/log/messages
> > > > > Jul 18 13:42:58 leela kernel: [50378.028634] w83627ehf: Found W83667HG-B
> > > > > chip at 0xa10
> > > > > # sensors
> > > > > w83667hg-isa-0a10
> > > > > Adapter: ISA adapter
> > > > > in0:         +0.98 V  (min =  +0.00 V, max =  +1.74 V)  
> > > > > in1:         +1.11 V  (min =  +0.10 V, max =  +0.20 V)   ALARM
> > > > > in2:         +3.36 V  (min =  +3.95 V, max =  +3.28 V)   ALARM
> > > > > in3:         +3.36 V  (min =  +1.57 V, max =  +2.10 V)   ALARM
> > > > > in4:         +1.10 V  (min =  +1.74 V, max =  +1.87 V)   ALARM
> > > > > in5:         +1.18 V  (min =  +1.02 V, max =  +1.30 V)  
> > > > > in7:         +3.28 V  (min =  +3.23 V, max =  +2.62 V)   ALARM
> > > > > in8:         +3.34 V  (min =  +3.18 V, max =  +0.05 V)   ALARM
> > > > > fan1:          0 RPM  (min = 3515 RPM, div = 128)  ALARM
> > > > > fan2:       1022 RPM  (min = 1638 RPM, div = 8)  ALARM
> > > > > fan3:          0 RPM  (min =  958 RPM, div = 128)  ALARM
> > > > > fan4:          0 RPM  (min = 10546 RPM, div = 128)  ALARM
> > > > > fan5:          0 RPM  (min = 3515 RPM, div = 128)  ALARM
> > > > > temp1:       +30.0°C  (high = -56.0°C, hyst = -17.0°C)  ALARM  sensor > > > > > thermistor
> > > > > temp2:       +41.5°C  (high = +80.0°C, hyst = +75.0°C)  sensor = thermistor
> > > > > temp3:        -3.0°C  (high = +80.0°C, hyst = +75.0°C)  sensor = thermistor
> > > > > cpu0_vid:   +0.000 V
> > > > > 
> > > > Actual values look mostly good, but the limits are really off.
> > > > Difficult to say, though, if those are wrong readings or just badly configured.
> > > > 
> > > > Jean, any idea/thoughts ?
> > > 
> > > Only a register dump would say from sure. But given that the alarm
> > > flags all match out-of-limit readings, I would be inclined to say these
> > > are only misonconfigured limits. Tobias could set proper limits and
> > > check that all the alarms go away.
> > > 
> > > The only inconsistency is with temp3... the reading is out of limit but
> > > there is no alarm flag. You may want to double check the datasheet to
> > > ensure that the driver reads both the temp3 value and its alarm flag
> > > from the right register/bit.
> > > 
> > I checked the specification. Registers are the same for both chips,
> > unless I am missing something. No idea what is going on.
> 
> I seem to remember that on some of these chips, in6 and temp3 are
> mutually exclusive? So you may have to check all the configuration
> registers which determine which of in6 or temp3 is enabled. Maybe in6
> is enabled and the driver wrongfully think temp3 is instead?
> 
> Just a random idea, I may be completely off the track.
> 
I did some research on the web. Looks like temp3 almost always shows
very low temperatures for the W83667, and also for w83627ehf. Also, since
the reading is low, but there is no low limit, there isn't really a reason
for raising an alarm. 

Also, min/max values for voltages seem to be off almost everywhere.
And values for temp1_max and temp1_hyst are often negative or very low.

So, in summry, looks like -B support is just as good or as bad as support
for the other chips. Not sure if that is good or bad, though, since the 
off-track readings are so common.

Guenter


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

  parent reply	other threads:[~2010-07-21  1:40 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-18 12:02 [lm-sensors] W83667HG-B testing Tobias Preclik
2010-07-20  1:24 ` Guenter Roeck
2010-07-20  6:54 ` Jean Delvare
2010-07-20 16:37 ` Guenter Roeck
2010-07-20 20:41 ` Guenter Roeck
2010-07-20 20:51 ` Jean Delvare
2010-07-20 20:58 ` Guenter Roeck
2010-07-20 21:26 ` Tobias Preclik
2010-07-21  1:40 ` Guenter Roeck [this message]
2010-08-14 19:21 ` Jean Delvare
2010-08-14 19:36 ` Jean Delvare

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=20100721014017.GA29955@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.