From: khali@linux-fr.org (Jean Delvare)
To: lm-sensors@vger.kernel.org
Subject: Fix fan speeds on ADM1026
Date: Thu, 19 May 2005 06:23:59 +0000 [thread overview]
Message-ID: <20030612104457.509e86a9.khali@linux-fr.org> (raw)
In-Reply-To: <3EE7EEF7.1040803@penguincomputing.com>
> Jerome Hsiao @ Arima pointed out to me that the fan speeds from the
> ADM1026 driver were not consistent with other drivers.
>
> Seems that most (all?) drivers assume two pulses per rev from the
> fans.
> I expected that users would need to use a "compute" directive to
> modify the fan speed for the number of pulses-per-rev of their fans.
> (All the fans I use are 4 ppr so I'm always having to "compute" a
> correct value.)
>
> But it's not consistent with the other drives...
>
> So here is a patch to fix the ADM1026 so it reports fan speeds
> consistent with the other drivers...
>
> This was done against CVS...
Well, that's what fanX_div is meant for. As far as I know, the number of
pulse/rotation depends mostly on the fan itself, not on the monitoring
chipset. That's why we let the users change the fanX_div value through
the config file.
Maybe there's actually something to be fixed in the adm1026 driver, but
maybe this is just a matter of setting the right fanX_div for unregular
fans.
--
Jean Delvare
http://www.ensicaen.ismra.fr/~delvare/
next prev parent reply other threads:[~2005-05-19 6:23 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-19 6:23 Fix fan speeds on ADM1026 Philip Pokorny
2005-05-19 6:23 ` Mark Studebaker
2005-05-19 6:23 ` Jean Delvare [this message]
2005-05-19 6:23 ` Philip Pokorny
2005-05-19 6:24 ` Mark D. Studebaker
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=20030612104457.509e86a9.khali@linux-fr.org \
--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.