From: Jean Delvare <khali@linux-fr.org>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] Looking for IT8720 datasheet.
Date: Tue, 07 Oct 2008 20:16:32 +0000 [thread overview]
Message-ID: <20081007221632.0e5a8679@hyperion.delvare> (raw)
In-Reply-To: <79c9d4530810020825r79de9817g6933f6cab1d414c1@mail.gmail.com>
On Fri, 03 Oct 2008 13:14:31 -0400, Frank Myhr wrote:
> Jean-Marc Spaggiari wrote:
> > When I force has_fan to 5 fans, I get:
> >
> > fan1: 3000 RPM (min = 0 RPM)
> > fan2: 0 RPM (min = 0 RPM)
> > fan3: 5973 RPM (min = 0 RPM)
> > fan4: -1 RPM (min = -1 RPM)
> > fan5: -1 RPM (min = -1 RPM)
> >
> > So can I just help him to read to pwm 4 and 5 correctly?
>
> The -1 values you're reporting for fan4 and fan5 come from
> #define FAN16_FROM_REG(val) ((val)=0?-1:(val)=0xffff?0:1350000/((val)*2))
>
> In other words, the 8720 reports "0", the above macro converts that to
> -1. (I'm not entirely clear why this is done.)
That's a legacy from the good ol' times. Old drivers tend to report
missing fans, fan failures or impossible register values with a speed
of -1. It never made any sense to me and many drivers report this
condition as RPM = 0, with an optional fanN_fault file getting value 1
for detected fan failures.
We have no clear standard for this yet so in practice every driver is
free to report -1 or 0 for uncommon fan conditions.
> > Regarding the resistors. should I publish the values? Or it's useless?
>
> I think it would help at least other users of your board. Perhaps add it
> to Documentation/hwmon/it87 ?
Best is to write a configuration file for the motherboard in question
and publish it on lm-sensors.org/wiki.
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
prev parent reply other threads:[~2008-10-07 20:16 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-02 15:25 [lm-sensors] Looking for IT8720 datasheet Jean-Marc Spaggiari
2008-10-02 15:39 ` Jean Delvare
2008-10-02 16:28 ` Jean-Marc Spaggiari
2008-10-02 16:41 ` Jean Delvare
2008-10-02 18:48 ` Frank Myhr
2008-10-03 13:23 ` Jean-Marc Spaggiari
2008-10-03 15:51 ` Frank Myhr
2008-10-03 16:32 ` Jean-Marc Spaggiari
2008-10-03 17:14 ` Frank Myhr
2008-10-03 20:29 ` Frank Myhr
2008-10-07 20:12 ` Jean Delvare
2008-10-07 20:16 ` Jean Delvare [this message]
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=20081007221632.0e5a8679@hyperion.delvare \
--to=khali@linux-fr.org \
--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.