From: Jean Delvare <khali@linux-fr.org>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] MINSTART has no effect in fancontrol
Date: Sun, 05 Jul 2009 11:41:44 +0000 [thread overview]
Message-ID: <20090705134144.3db41b27@hyperion.delvare> (raw)
In-Reply-To: <4A4FFD5F.6060508@gmail.com>
Hi Balázs,
On Sun, 05 Jul 2009 03:09:51 +0200, Hámorszky Balázs wrote:
> In fancontrol MINSTART has no effect (it has, but only for an INTERVAL
> period of time).
> The line:
> pwmval="(${tval}-${mint})*(${maxpwm}-${minso})/(${maxt}-${mint})+${minso}"
> should be:
> pwmval="(${tval}-${mint})*(${maxpwm}-${minsa})/(${maxt}-${mint})+${minsa}"
> if I understand the configuration options of fancontrol right.
I guess you don't. The formula above is perfectly correct. With your
proposed change, MINSTART would be used instead of MINSTOP for the
speed computations, which is definitely not what we want.
> At the current state, pwmval can be lower than MINSTART
Of course it can, by design. But it can't be lower than MINPWM. I
suggest you read the fancontrol documentation:
http://www.lm-sensors.org/browser/lm-sensors/trunk/doc/fancontrol.txt
and in particular the graph. You'll see that MINSTART is _not_ used to
compute the fan speed. MINSTART is only used for the initial pulse to
get a fan spinning.
> (so on some temperatures, greater than MINTEMP, the fan won't start).
It should, if your settings are correct. Maybe you have set either
MINSTART or MINSTOP too low for your fan.
That being said, it could be that the MINSTART pulse of 1 second is a
tad too short for some fans. If this is your case then you could
increase the sleep value from 1 to 2 or 3 and see if it helps.
--
Jean Delvare
http://khali.linux-fr.org/wishlist.html
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2009-07-05 11:41 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-05 1:09 [lm-sensors] MINSTART has no effect in fancontrol Hámorszky Balázs
2009-07-05 11:41 ` Jean Delvare [this message]
2009-07-05 12:01 ` Hámorszky Balázs
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=20090705134144.3db41b27@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.