From: khali@linux-fr.org (Jean Delvare)
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] Random thoughts on fan speed control
Date: Tue, 05 Sep 2006 08:50:23 +0000 [thread overview]
Message-ID: <20060905105023.46f7d404.khali@linux-fr.org> (raw)
In-Reply-To: <20060904231231.c7dc7e1f.khali@linux-fr.org>
Hi Rudolf,
> > * Reading the F71805F documentation, I learned that the external
> > circuitery needed for PWM fan speed control is completely different
> > from the one needed for DC fan speed control. So I wonder if it is a
> > good idea to let the user change between the two modes. Clearly the
> > BIOS should set it right. I tried to switch to DC mode and it had no
> > effect at all on my fan (not even on/off). I wonder if it is possible
> > to confuse or even damage the hardware by using the wrong mode?
>
> Yes DC/PWM settings were already removed from W83793 and imho not implemented on
> EHF too.
Well, I have no problem with chip makers implementing both and letting
the integrators choose what they want to do. But in that case the BIOS
should set the proper operation mode.
> You need to know what circuits are onboard -> BIOS should do that for you. If
> you want to implement this I would suggest just a kernel param for BIOS fixups.
I usually avoid adding BIOS fixups unless really needed. I don't want
to incite motherboard manufacturers to ship broken BIOSes. As long as
nobody reports a problem with this, I won't implement it. And even
then, I'd rather ask the user to request a fixed BIOS, as it makes much
more sense.
So my current plan is:
* Provide pwmN_mode, but read-only.
* Only provide pwmN_freq in PWM mode.
Thanks,
--
Jean Delvare
next prev parent reply other threads:[~2006-09-05 8:50 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-09-04 21:12 [lm-sensors] Random thoughts on fan speed control Jean Delvare
2006-09-05 7:50 ` Rudolf Marek
2006-09-05 8:50 ` Jean Delvare [this message]
2006-09-05 15:01 ` David Hubbard
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=20060905105023.46f7d404.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.