All of lore.kernel.org
 help / color / mirror / Atom feed
* [lm-sensors] pwmconfig can only control fan speed at beginning and
@ 2009-11-21 15:19 Allen Edwards
  2009-11-21 15:49 ` [lm-sensors] pwmconfig can only control fan speed at beginning Jean Delvare
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: Allen Edwards @ 2009-11-21 15:19 UTC (permalink / raw)
  To: lm-sensors


[-- Attachment #1.1: Type: text/plain, Size: 648 bytes --]

I searched back several months of your archives and can't find anything
relevant to my problem.  When I run pwmconfig it finds my fans and controls
the speeds. I can run a characterization of fan speeds vs settings and find
that 180 is a good speed noise wise.  It is definitely controlling the fan
speed.  But when it gets to the part about finding the min and max stall
speeds, it will not control the fan speed, the fan just stays at full speed.
 And, of course, fancontrol will not work.  This seems very strange to me.
 The fan is 3 wire and I have tried it in both a 3 and 4 wire controllable
header.

I would appreciate help on this.

Allen

[-- Attachment #1.2: Type: text/html, Size: 720 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
@ 2009-11-21 15:49 ` Jean Delvare
  2009-11-22  4:34 ` Allen Edwards
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Jean Delvare @ 2009-11-21 15:49 UTC (permalink / raw)
  To: lm-sensors

On Sat, 21 Nov 2009 07:19:01 -0800, Allen Edwards wrote:
> I searched back several months of your archives and can't find anything
> relevant to my problem.  When I run pwmconfig it finds my fans and controls
> the speeds. I can run a characterization of fan speeds vs settings and find
> that 180 is a good speed noise wise.  It is definitely controlling the fan
> speed.  But when it gets to the part about finding the min and max stall
> speeds, it will not control the fan speed, the fan just stays at full speed.
>  And, of course, fancontrol will not work.  This seems very strange to me.
>  The fan is 3 wire and I have tried it in both a 3 and 4 wire controllable
> header.

And you are using lm-sensors version...? Kernel version...? Hwmon
drivers...?

-- 
Jean Delvare

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
  2009-11-21 15:49 ` [lm-sensors] pwmconfig can only control fan speed at beginning Jean Delvare
@ 2009-11-22  4:34 ` Allen Edwards
  2009-11-22  8:51 ` Jean Delvare
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Allen Edwards @ 2009-11-22  4:34 UTC (permalink / raw)
  To: lm-sensors


[-- Attachment #1.1: Type: text/plain, Size: 1020 bytes --]

lm-sensors 3.0.0, kernel 2.6.24.23-generic  Pardon my stupidity but I do not
know hot to find a version of hwmon.

Allen


On Sat, Nov 21, 2009 at 7:49 AM, Jean Delvare <khali@linux-fr.org> wrote:

> On Sat, 21 Nov 2009 07:19:01 -0800, Allen Edwards wrote:
> > I searched back several months of your archives and can't find anything
> > relevant to my problem.  When I run pwmconfig it finds my fans and
> controls
> > the speeds. I can run a characterization of fan speeds vs settings and
> find
> > that 180 is a good speed noise wise.  It is definitely controlling the
> fan
> > speed.  But when it gets to the part about finding the min and max stall
> > speeds, it will not control the fan speed, the fan just stays at full
> speed.
> >  And, of course, fancontrol will not work.  This seems very strange to
> me.
> >  The fan is 3 wire and I have tried it in both a 3 and 4 wire
> controllable
> > header.
>
> And you are using lm-sensors version...? Kernel version...? Hwmon
> drivers...?
>
> --
> Jean Delvare
>

[-- Attachment #1.2: Type: text/html, Size: 1428 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
  2009-11-21 15:49 ` [lm-sensors] pwmconfig can only control fan speed at beginning Jean Delvare
  2009-11-22  4:34 ` Allen Edwards
@ 2009-11-22  8:51 ` Jean Delvare
  2009-11-22 16:05 ` Allen Edwards
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Jean Delvare @ 2009-11-22  8:51 UTC (permalink / raw)
  To: lm-sensors

Hi Allen,

Please do not top-post.

On Sat, 21 Nov 2009 20:34:35 -0800, Allen Edwards wrote:
> On Sat, Nov 21, 2009 at 7:49 AM, Jean Delvare <khali@linux-fr.org> wrote:
> > On Sat, 21 Nov 2009 07:19:01 -0800, Allen Edwards wrote:
> > > I searched back several months of your archives and can't find anything
> > > relevant to my problem.  When I run pwmconfig it finds my fans and
> > > controls the speeds. I can run a characterization of fan speeds vs
> > > settings and find that 180 is a good speed noise wise.  It is definitely
> > > controlling the fan speed.  But when it gets to the part about finding
> > > the min and max stall speeds, it will not control the fan speed, the fan
> > > just stays at full speed.
> > >  And, of course, fancontrol will not work.  This seems very strange to
> > > me. The fan is 3 wire and I have tried it in both a 3 and 4 wire
> > > controllable header.
> >
> > And you are using lm-sensors version...? Kernel version...? Hwmon
> > drivers...?

> lm-sensors 3.0.0, kernel 2.6.24.23-generic  Pardon my stupidity but I do not
> know hot to find a version of hwmon.

I never asked for a version of hwmon; I asked which hwmon drivers you
were using on your system.

It probably doesn't matter anyway. Looking at lm-sensors' changelog
[1], I see the following:

3.0.3 (2008-09-28)
  (...)
  pwmconfig: (...)
             Fix MINSTOP and MINSTART test functions (#2340)
             Test MINSTOP before MINSTART
             Use better step values for MINSTOP test
             (...)
             Determine MINSTOP automatically
             Skip MINSTOP and MINSTART tests if fan can't stop

This makes it clear to me that the problem you reported is known and
already fixed in lm-sensors 3.0.3 and later. So all you have to do is
upgrade to lm-sensors 3.0.3 or later. If you don't want to for any
reason, note that pwmconfig is a standalone script, so you can just
copy the one from lm-sensors 3.0.3 [2] and run it from wherever you
like.

[1] http://www.lm-sensors.org/browser/lm-sensors/tags/V3-1-1/CHANGES
[2] http://www.lm-sensors.org/export/5798/lm-sensors/tags/V3-0-3/prog/pwm/pwmconfig

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
                   ` (2 preceding siblings ...)
  2009-11-22  8:51 ` Jean Delvare
@ 2009-11-22 16:05 ` Allen Edwards
  2009-11-22 16:35 ` Jean Delvare
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Allen Edwards @ 2009-11-22 16:05 UTC (permalink / raw)
  To: lm-sensors

On Sun, Nov 22, 2009 at 12:51 AM, Jean Delvare <khali@linux-fr.org> wrote:
> Hi Allen,
>
> Please do not top-post.
>
> On Sat, 21 Nov 2009 20:34:35 -0800, Allen Edwards wrote:
>> On Sat, Nov 21, 2009 at 7:49 AM, Jean Delvare <khali@linux-fr.org> wrote:
>> > On Sat, 21 Nov 2009 07:19:01 -0800, Allen Edwards wrote:
>> > > I searched back several months of your archives and can't find anything
>> > > relevant to my problem.  When I run pwmconfig it finds my fans and
>> > > controls the speeds. I can run a characterization of fan speeds vs
>> > > settings and find that 180 is a good speed noise wise.  It is definitely
>> > > controlling the fan speed.  But when it gets to the part about finding
>> > > the min and max stall speeds, it will not control the fan speed, the fan
>> > > just stays at full speed.
>> > >  And, of course, fancontrol will not work.  This seems very strange to
>> > > me. The fan is 3 wire and I have tried it in both a 3 and 4 wire
>> > > controllable header.
>> >
>> > And you are using lm-sensors version...? Kernel version...? Hwmon
>> > drivers...?
>
>> lm-sensors 3.0.0, kernel 2.6.24.23-generic  Pardon my stupidity but I do not
>> know hot to find a version of hwmon.
>
> I never asked for a version of hwmon; I asked which hwmon drivers you
> were using on your system.
>
> It probably doesn't matter anyway. Looking at lm-sensors' changelog
> [1], I see the following:
>
> 3.0.3 (2008-09-28)
>  (...)
>  pwmconfig: (...)
>             Fix MINSTOP and MINSTART test functions (#2340)
>             Test MINSTOP before MINSTART
>             Use better step values for MINSTOP test
>             (...)
>             Determine MINSTOP automatically
>             Skip MINSTOP and MINSTART tests if fan can't stop
>
> This makes it clear to me that the problem you reported is known and
> already fixed in lm-sensors 3.0.3 and later. So all you have to do is
> upgrade to lm-sensors 3.0.3 or later. If you don't want to for any
> reason, note that pwmconfig is a standalone script, so you can just
> copy the one from lm-sensors 3.0.3 [2] and run it from wherever you
> like.
>
> [1] http://www.lm-sensors.org/browser/lm-sensors/tags/V3-1-1/CHANGES
> [2] http://www.lm-sensors.org/export/5798/lm-sensors/tags/V3-0-3/prog/pwm/pwmconfig
>
> --
> Jean Delvare
> http://khali.linux-fr.org/wishlist.html
>

Thanks for the reply.

I thought I had the latest version having just done apt-get install so
I didn't think of this but I should have.

I now have version 3.1.1 unzipped but can't install it.  I get an
ERROR 1 exit on make.  See below.

dad@myth_desktop:~/lm_sensors-3.1.1$ sudo make all
gcc -Wl,-rpath,/usr/local/lib -o prog/sensors/sensors
prog/sensors/main.ro prog/sensors/chips.ro  -Llib -lsensors
lib/libsensors.so: undefined reference to `sensors_yylex'
lib/libsensors.so: undefined reference to `sensors_lex_error'
lib/libsensors.so: undefined reference to `sensors_yylineno'
lib/libsensors.so: undefined reference to `sensors_scanner_exit'
lib/libsensors.so: undefined reference to `sensors_scanner_init'
lib/libsensors.so: undefined reference to `sensors_yyfilename'
collect2: ld returned 1 exit status
make: *** [prog/sensors/sensors] Error 1


Not sure what I did wrong...

Allen

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
                   ` (3 preceding siblings ...)
  2009-11-22 16:05 ` Allen Edwards
@ 2009-11-22 16:35 ` Jean Delvare
  2009-11-22 16:57 ` Allen Edwards
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Jean Delvare @ 2009-11-22 16:35 UTC (permalink / raw)
  To: lm-sensors

On Sun, 22 Nov 2009 08:05:07 -0800, Allen Edwards wrote:
> I thought I had the latest version having just done apt-get install so
> I didn't think of this but I should have.
> 
> I now have version 3.1.1 unzipped but can't install it.  I get an
> ERROR 1 exit on make.  See below.
> 
> dad@myth_desktop:~/lm_sensors-3.1.1$ sudo make all
> gcc -Wl,-rpath,/usr/local/lib -o prog/sensors/sensors
> prog/sensors/main.ro prog/sensors/chips.ro  -Llib -lsensors
> lib/libsensors.so: undefined reference to `sensors_yylex'
> lib/libsensors.so: undefined reference to `sensors_lex_error'
> lib/libsensors.so: undefined reference to `sensors_yylineno'
> lib/libsensors.so: undefined reference to `sensors_scanner_exit'
> lib/libsensors.so: undefined reference to `sensors_scanner_init'
> lib/libsensors.so: undefined reference to `sensors_yyfilename'
> collect2: ld returned 1 exit status
> make: *** [prog/sensors/sensors] Error 1
> 
> 
> Not sure what I did wrong...

You forgot to install flex and bison, and our build system isn't
exactly user-friendly. After you do, run "make clean" then "make" again.

That being said... pwmconfig is a shell script, it doesn't need to be
built.

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
                   ` (4 preceding siblings ...)
  2009-11-22 16:35 ` Jean Delvare
@ 2009-11-22 16:57 ` Allen Edwards
  2009-11-22 18:52 ` Jean Delvare
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Allen Edwards @ 2009-11-22 16:57 UTC (permalink / raw)
  To: lm-sensors

On Sun, Nov 22, 2009 at 8:35 AM, Jean Delvare <khali@linux-fr.org> wrote:
> On Sun, 22 Nov 2009 08:05:07 -0800, Allen Edwards wrote:
>> I thought I had the latest version having just done apt-get install so
>> I didn't think of this but I should have.
>>
>> I now have version 3.1.1 unzipped but can't install it.  I get an
>> ERROR 1 exit on make.  See below.
>>
>> dad@myth_desktop:~/lm_sensors-3.1.1$ sudo make all
>> gcc -Wl,-rpath,/usr/local/lib -o prog/sensors/sensors
>> prog/sensors/main.ro prog/sensors/chips.ro  -Llib -lsensors
>> lib/libsensors.so: undefined reference to `sensors_yylex'
>> lib/libsensors.so: undefined reference to `sensors_lex_error'
>> lib/libsensors.so: undefined reference to `sensors_yylineno'
>> lib/libsensors.so: undefined reference to `sensors_scanner_exit'
>> lib/libsensors.so: undefined reference to `sensors_scanner_init'
>> lib/libsensors.so: undefined reference to `sensors_yyfilename'
>> collect2: ld returned 1 exit status
>> make: *** [prog/sensors/sensors] Error 1
>>
>>
>> Not sure what I did wrong...
>
> You forgot to install flex and bison, and our build system isn't
> exactly user-friendly. After you do, run "make clean" then "make" again.
>
> That being said... pwmconfig is a shell script, it doesn't need to be
> built.
>
> --
> Jean Delvare
> http://khali.linux-fr.org/wishlist.html
>

Actually, I did install both of these programs as the documentation is
pretty clear on that.

dad@myth_desktop:~/lm_sensors-3.1.1$ bison --version
bison (GNU Bison) 2.3
Written by Robert Corbett and Richard Stallman.

Copyright (C) 2006 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

dad@myth_desktop:~/lm_sensors-3.1.1$ flex  --version
flex 2.5.34


Allen

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
                   ` (5 preceding siblings ...)
  2009-11-22 16:57 ` Allen Edwards
@ 2009-11-22 18:52 ` Jean Delvare
  2009-11-22 21:47 ` Allen Edwards
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Jean Delvare @ 2009-11-22 18:52 UTC (permalink / raw)
  To: lm-sensors

On Sun, 22 Nov 2009 08:57:18 -0800, Allen Edwards wrote:
> On Sun, Nov 22, 2009 at 8:35 AM, Jean Delvare <khali@linux-fr.org> wrote:
> > You forgot to install flex and bison, and our build system isn't
> > exactly user-friendly. After you do, run "make clean" then "make" again.
> >
> > That being said... pwmconfig is a shell script, it doesn't need to be
> > built.
>
> Actually, I did install both of these programs as the documentation is
> pretty clear on that.
> 
> dad@myth_desktop:~/lm_sensors-3.1.1$ bison --version
> bison (GNU Bison) 2.3
> Written by Robert Corbett and Richard Stallman.
> 
> Copyright (C) 2006 Free Software Foundation, Inc.
> This is free software; see the source for copying conditions.  There is NO
> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> 
> dad@myth_desktop:~/lm_sensors-3.1.1$ flex  --version
> flex 2.5.34

Did you run "make clean" before running "make" again after installing
flex and bison? This is required.

If you did and it did not help, please provide a full build log and
I'll look at it.

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
                   ` (6 preceding siblings ...)
  2009-11-22 18:52 ` Jean Delvare
@ 2009-11-22 21:47 ` Allen Edwards
  2009-11-23  9:00 ` Jean Delvare
  2009-11-23 16:06 ` Allen Edwards
  9 siblings, 0 replies; 11+ messages in thread
From: Allen Edwards @ 2009-11-22 21:47 UTC (permalink / raw)
  To: lm-sensors


[-- Attachment #1.1: Type: text/plain, Size: 1760 bytes --]

On Sun, Nov 22, 2009 at 10:52 AM, Jean Delvare <khali@linux-fr.org> wrote:

> On Sun, 22 Nov 2009 08:57:18 -0800, Allen Edwards wrote:
> > On Sun, Nov 22, 2009 at 8:35 AM, Jean Delvare <khali@linux-fr.org>
> wrote:
> > > You forgot to install flex and bison, and our build system isn't
> > > exactly user-friendly. After you do, run "make clean" then "make"
> again.
> > >
> > > That being said... pwmconfig is a shell script, it doesn't need to be
> > > built.
> >
> > Actually, I did install both of these programs as the documentation is
> > pretty clear on that.
> >
> > dad@myth_desktop:~/lm_sensors-3.1.1$ bison --version
> > bison (GNU Bison) 2.3
> > Written by Robert Corbett and Richard Stallman.
> >
> > Copyright (C) 2006 Free Software Foundation, Inc.
> > This is free software; see the source for copying conditions.  There is
> NO
> > warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR
> PURPOSE.
> >
> > dad@myth_desktop:~/lm_sensors-3.1.1$ flex  --version
> > flex 2.5.34
>
> Did you run "make clean" before running "make" again after installing
> flex and bison? This is required.
>
> If you did and it did not help, please provide a full build log and
> I'll look at it.
>
>
With your help all is working and the fan noise cannot be heard from across
the room.  The "make clean" did the trick.  Thanks so much for your help.

I have a small follow-up question.  I searched for a long time looking for
how to start fancontrol automatically.  All I found was a rather elaborate
script here: http://ubuntuforums.org/showthread.php?t=42737 but it looked
old (2005) and the paths didn't look correct so I didn't use it.

I put a line in my rc.local file "/usr/local/sbin/fancontrol" and all seems
well.

Is this acceptable?

Allen

[-- Attachment #1.2: Type: text/html, Size: 2624 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
                   ` (7 preceding siblings ...)
  2009-11-22 21:47 ` Allen Edwards
@ 2009-11-23  9:00 ` Jean Delvare
  2009-11-23 16:06 ` Allen Edwards
  9 siblings, 0 replies; 11+ messages in thread
From: Jean Delvare @ 2009-11-23  9:00 UTC (permalink / raw)
  To: lm-sensors

Hi Allen,

On Sun, 22 Nov 2009 13:47:24 -0800, Allen Edwards wrote:
> With your help all is working and the fan noise cannot be heard from across
> the room.  The "make clean" did the trick.  Thanks so much for your help.
> 
> I have a small follow-up question.  I searched for a long time looking for
> how to start fancontrol automatically.  All I found was a rather elaborate
> script here: http://ubuntuforums.org/showthread.php?tB737 but it looked
> old (2005) and the paths didn't look correct so I didn't use it.

It is your distribution's responsibility to provide an initialization
script for the fancontrol service. On openSUSE this is included in the
lm_sensors service. But maybe some distributions think fancontrol is
not good enough to deserve it (and I can't blame them.)

> I put a line in my rc.local file "/usr/local/sbin/fancontrol" and all seems
> well.
> 
> Is this acceptable?

Yes it is, even though I would be a little more cautious:

if [ -x /usr/local/sbin/fancontrol -a -r /etc/fancontrol ]
then
	/usr/local/sbin/fancontrol
fi

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [lm-sensors] pwmconfig can only control fan speed at beginning
  2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
                   ` (8 preceding siblings ...)
  2009-11-23  9:00 ` Jean Delvare
@ 2009-11-23 16:06 ` Allen Edwards
  9 siblings, 0 replies; 11+ messages in thread
From: Allen Edwards @ 2009-11-23 16:06 UTC (permalink / raw)
  To: lm-sensors


[-- Attachment #1.1: Type: text/plain, Size: 642 bytes --]

>
> > Is this acceptable?
>
> Yes it is, even though I would be a little more cautious:
>
> if [ -x /usr/local/sbin/fancontrol -a -r /etc/fancontrol ]
> then
>        /usr/local/sbin/fancontrol
> fi
>
>
OK, thanks.  You have been a huge help and my wife is very happy with the
result.  That makes me happy as well!

As I understand it, /usr/local/sbin/fancontrol will not run without being
there or having its control file there so I am thinking I am OK without the
added protection you offer and I think I will leave things there unless I am
missing something other than clean programming style.

Again, thanks so much for your help.

Allen

[-- Attachment #1.2: Type: text/html, Size: 969 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2009-11-23 16:06 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-21 15:19 [lm-sensors] pwmconfig can only control fan speed at beginning and Allen Edwards
2009-11-21 15:49 ` [lm-sensors] pwmconfig can only control fan speed at beginning Jean Delvare
2009-11-22  4:34 ` Allen Edwards
2009-11-22  8:51 ` Jean Delvare
2009-11-22 16:05 ` Allen Edwards
2009-11-22 16:35 ` Jean Delvare
2009-11-22 16:57 ` Allen Edwards
2009-11-22 18:52 ` Jean Delvare
2009-11-22 21:47 ` Allen Edwards
2009-11-23  9:00 ` Jean Delvare
2009-11-23 16:06 ` Allen Edwards

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.