All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Hámorszky Balázs" <balihb@gmail.com>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] MINSTART has no effect in fancontrol
Date: Sun, 05 Jul 2009 12:01:23 +0000	[thread overview]
Message-ID: <4A509613.60206@gmail.com> (raw)
In-Reply-To: <4A4FFD5F.6060508@gmail.com>

Jean Delvare wrote:
> 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.

You're right. My MINSTOP was 0. I've misunderstood the function of it.

> 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.
> 

Thanks for the quick help!

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

      parent reply	other threads:[~2009-07-05 12:01 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
2009-07-05 12:01 ` Hámorszky Balázs [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=4A509613.60206@gmail.com \
    --to=balihb@gmail.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.