[lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support for

[lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support for

[Guenter Roeck]

The following patch series adds support for NCT6775F and NCT6776F to the
w83627ehf driver. It also includes some cleanup and adds support for
the fourth temperature sensor on W83677HG-B.

v4:
- Store rpm instead of raw fan speed readings
- For NCT6775F, increase fan divisor if the fan speed reads 0
  [ On NCT6775F, fan speed readings can return 0 instead of 0xff if the
    fan divisor value is too low ]
- Separate changes into more patches to simplify review

v3:
- Documentation: Remove references to datasheets which no longer exist
- Documentation: Add information about limits of SmartFan IV support.
- SmartFan III mode is not supported on NCT6776F, so remove related attributes
  and refuse to configure it.
- (additional patch) Improve support for chips with 16-bit fan count registers

v2:
- W83677HG-B does share the AUXTIN/VIN3 pin, so we can not skip this check.
- Cosmetic changes to fix a couple of checkpatch errors and to undo some
  unnecessary formatting changes in patch 5.
- Patch 1 introduced a bug in temp_to_reg which was fixed in a later patch.
  Modified code to not introduce the bug in the first place.
- Two dev_dbg messages were changed to dev_info for testing. Changed back
  to dev_dbg.

The patched driver can be downloaded as stand-alone driver from
	http://www.roeck-us.net/linux/drivers/w83627ehf/

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Guenter Roeck]

On Mon, 2011-02-14 at 14:35 -0500, Guenter Roeck wrote:
> The following patch series adds support for NCT6775F and NCT6776F to the
> w83627ehf driver. It also includes some cleanup and adds support for
> the fourth temperature sensor on W83677HG-B.
> 
> v4:
> - Store rpm instead of raw fan speed readings
> - For NCT6775F, increase fan divisor if the fan speed reads 0
>   [ On NCT6775F, fan speed readings can return 0 instead of 0xff if the
>     fan divisor value is too low ]
> - Separate changes into more patches to simplify review
> 
> v3:
> - Documentation: Remove references to datasheets which no longer exist
> - Documentation: Add information about limits of SmartFan IV support.
> - SmartFan III mode is not supported on NCT6776F, so remove related attributes
>   and refuse to configure it.
> - (additional patch) Improve support for chips with 16-bit fan count registers
> 
> v2:
> - W83677HG-B does share the AUXTIN/VIN3 pin, so we can not skip this check.
> - Cosmetic changes to fix a couple of checkpatch errors and to undo some
>   unnecessary formatting changes in patch 5.
> - Patch 1 introduced a bug in temp_to_reg which was fixed in a later patch.
>   Modified code to not introduce the bug in the first place.
> - Two dev_dbg messages were changed to dev_info for testing. Changed back
>   to dev_dbg.
> 
> The patched driver can be downloaded as stand-alone driver from
> 	http://www.roeck-us.net/linux/drivers/w83627ehf/

Feedback, anyone ?

Even if you don't have time to review the code, a "Tested-by" would be
appreciated as well.

Thanks,
Guenter



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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Ian Dobson]

--------------------------------------------------
From: "Guenter Roeck" <guenter.roeck@ericsson.com>
Sent: Tuesday, February 22, 2011 5:16 PM
To: "Jean Delvare" <khali@linux-fr.org>
Cc: "Ian Dobson" <i.dobson@planet-ian.com>; "Andy Lutomirski" 
<luto@mit.edu>; "Randy Dunlap" <rdunlap@xenotime.net>; 
<binximeng@gmail.com>; <lmsensors@tapanitarvainen.fi>; 
<andrea.rizzolo@gmail.com>; <jeff.sadowski@gmail.com>; 
<lm-sensors@lm-sensors.org>; <linux-doc@vger.kernel.org>
Subject: Re: [PATCH v4 00/10] hwmon: (w83627ehf) Add support for NCT6775F 
and NCT6776F

> On Mon, 2011-02-14 at 14:35 -0500, Guenter Roeck wrote:
>> The following patch series adds support for NCT6775F and NCT6776F to the
>> w83627ehf driver. It also includes some cleanup and adds support for
>> the fourth temperature sensor on W83677HG-B.
>>
>> v4:
>> - Store rpm instead of raw fan speed readings
>> - For NCT6775F, increase fan divisor if the fan speed reads 0
>>   [ On NCT6775F, fan speed readings can return 0 instead of 0xff if the
>>     fan divisor value is too low ]
>> - Separate changes into more patches to simplify review
>>
>> v3:
>> - Documentation: Remove references to datasheets which no longer exist
>> - Documentation: Add information about limits of SmartFan IV support.
>> - SmartFan III mode is not supported on NCT6776F, so remove related 
>> attributes
>>   and refuse to configure it.
>> - (additional patch) Improve support for chips with 16-bit fan count 
>> registers
>>
>> v2:
>> - W83677HG-B does share the AUXTIN/VIN3 pin, so we can not skip this 
>> check.
>> - Cosmetic changes to fix a couple of checkpatch errors and to undo some
>>   unnecessary formatting changes in patch 5.
>> - Patch 1 introduced a bug in temp_to_reg which was fixed in a later 
>> patch.
>>   Modified code to not introduce the bug in the first place.
>> - Two dev_dbg messages were changed to dev_info for testing. Changed back
>>   to dev_dbg.
>>
>> The patched driver can be downloaded as stand-alone driver from
>> http://www.roeck-us.net/linux/drivers/w83627ehf/
>
> Feedback, anyone ?
>
> Even if you don't have time to review the code, a "Tested-by" would be
> appreciated as well.
>
> Thanks,
> Guenter
>
>

Hi Guenter,

You can add a tested by ian dobson (i.dobson@planet-ian.com) for a nct6776f 
on a Asus p8p67 pro

I've also looked at the code, although I'm not a long term kernel hacker I 
understand/program in C and various other languages for my job and I've not 
seen anything major wrong with the code, so a Reviewed by might also be 
accepable. Note I'm still running a patched version with the 1.5 second 
caching disabled.

I had a look at the tacho signal from the CPU fan with my oscilloscope and 
sometimes I see a small dropout during high part of the signal. The signal 
doesn't drop to 0, just about 30-50%. Do you know the switching point for 
tacho signal? In the documentation they talk about a TTL signal so low is 
below 20% and high is over 50% and the fan is outputting as signal in the 
"undefined area" every so often.


Regards and Thanks for the great work
Ian Dobson 


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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Guenter Roeck]

Hi Ian,

On Tue, 2011-02-22 at 13:31 -0500, Ian Dobson wrote:
> 
> --------------------------------------------------
> From: "Guenter Roeck" <guenter.roeck@ericsson.com>
> Sent: Tuesday, February 22, 2011 5:16 PM
> To: "Jean Delvare" <khali@linux-fr.org>
> Cc: "Ian Dobson" <i.dobson@planet-ian.com>; "Andy Lutomirski" 
> <luto@mit.edu>; "Randy Dunlap" <rdunlap@xenotime.net>; 
> <binximeng@gmail.com>; <lmsensors@tapanitarvainen.fi>; 
> <andrea.rizzolo@gmail.com>; <jeff.sadowski@gmail.com>; 
> <lm-sensors@lm-sensors.org>; <linux-doc@vger.kernel.org>
> Subject: Re: [PATCH v4 00/10] hwmon: (w83627ehf) Add support for NCT6775F 
> and NCT6776F
> 
> > On Mon, 2011-02-14 at 14:35 -0500, Guenter Roeck wrote:
> >> The following patch series adds support for NCT6775F and NCT6776F to the
> >> w83627ehf driver. It also includes some cleanup and adds support for
> >> the fourth temperature sensor on W83677HG-B.
> >>
> >> v4:
> >> - Store rpm instead of raw fan speed readings
> >> - For NCT6775F, increase fan divisor if the fan speed reads 0
> >>   [ On NCT6775F, fan speed readings can return 0 instead of 0xff if the
> >>     fan divisor value is too low ]
> >> - Separate changes into more patches to simplify review
> >>
> >> v3:
> >> - Documentation: Remove references to datasheets which no longer exist
> >> - Documentation: Add information about limits of SmartFan IV support.
> >> - SmartFan III mode is not supported on NCT6776F, so remove related 
> >> attributes
> >>   and refuse to configure it.
> >> - (additional patch) Improve support for chips with 16-bit fan count 
> >> registers
> >>
> >> v2:
> >> - W83677HG-B does share the AUXTIN/VIN3 pin, so we can not skip this 
> >> check.
> >> - Cosmetic changes to fix a couple of checkpatch errors and to undo some
> >>   unnecessary formatting changes in patch 5.
> >> - Patch 1 introduced a bug in temp_to_reg which was fixed in a later 
> >> patch.
> >>   Modified code to not introduce the bug in the first place.
> >> - Two dev_dbg messages were changed to dev_info for testing. Changed back
> >>   to dev_dbg.
> >>
> >> The patched driver can be downloaded as stand-alone driver from
> >> http://www.roeck-us.net/linux/drivers/w83627ehf/
> >
> > Feedback, anyone ?
> >
> > Even if you don't have time to review the code, a "Tested-by" would be
> > appreciated as well.
> >
> > Thanks,
> > Guenter
> >
> >
> 
> Hi Guenter,
> 
> You can add a tested by ian dobson (i.dobson@planet-ian.com) for a nct6776f 
> on a Asus p8p67 pro
> 
Thanks!

> I've also looked at the code, although I'm not a long term kernel hacker I 
> understand/program in C and various other languages for my job and I've not 
> seen anything major wrong with the code, so a Reviewed by might also be 
> accepable. Note I'm still running a patched version with the 1.5 second 
> caching disabled.

If you see something minor in the code which you think might be
improved, please let me know. It doesn't have to be a major issue.

> I had a look at the tacho signal from the CPU fan with my oscilloscope and 
> sometimes I see a small dropout during high part of the signal. The signal 
> doesn't drop to 0, just about 30-50%. Do you know the switching point for 
> tacho signal? In the documentation they talk about a TTL signal so low is 
> below 20% and high is over 50% and the fan is outputting as signal in the 
> "undefined area" every so often.
> 

min/typ/max:

Input Low Threshold Voltage Vt- 0.5 0.8 1.1 V
Input High Threshold Voltage Vt+ 1.6 2.0 2.4 V
Hystersis VTH 0.5 1.2 V

How long is the dropout time ? 

Maybe you can check control register F0h (FANIN de-bouncer Register) in
Logical Device B. Bit 1..5 reflect sysfanin, cpufanin, auxfanin0,
auxfanin1, and auxfanin2. Bit is 1 if the de-bouncer is enabled, 0
otherwise. If it is not enabled on your system, maybe you can enable it
and see if it makes a difference.

Thanks,
Guenter



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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Phillip Susi]

On 02/22/2011 01:31 PM, Ian Dobson wrote:
> You can add a tested by ian dobson (i.dobson@planet-ian.com) for a
> nct6776f on a Asus p8p67 pro

Same board here.  Driver seems to work well, though I'm still trying to 
figure out why pwm1 seems to affect 2 of the 3 case fans I have in 
addition to the cpu fan.  If I set pwm1 to zero, fan1 stops as expected, 
but so does does fan3 according to the inputs.  I visually can see 
though, that front and rear fans have stopped, but the top fan is still 
going.  All of the case fans are 3 pin.  Fans 2 and 3 still show 
movement according to the inputs when pwm1 is off, but I can only see 
one of the 4 fans moving.

> I had a look at the tacho signal from the CPU fan with my oscilloscope
> and sometimes I see a small dropout during high part of the signal. The
> signal doesn't drop to 0, just about 30-50%. Do you know the switching
> point for tacho signal? In the documentation they talk about a TTL
> signal so low is below 20% and high is over 50% and the fan is
> outputting as signal in the "undefined area" every so often.

That sounds like a bad fan.  I'll have to borrow a scope from work and 
check mine.  Will let you know the results when I do.

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Guenter Roeck]

Hi Phillip,

On Tue, Feb 22, 2011 at 09:05:45PM -0500, Phillip Susi wrote:
> On 02/22/2011 01:31 PM, Ian Dobson wrote:
> > You can add a tested by ian dobson (i.dobson@planet-ian.com) for a
> > nct6776f on a Asus p8p67 pro
> 
> Same board here.  Driver seems to work well, though I'm still trying to 
> figure out why pwm1 seems to affect 2 of the 3 case fans I have in 
> addition to the cpu fan.  If I set pwm1 to zero, fan1 stops as expected, 
> but so does does fan3 according to the inputs.  I visually can see 
> though, that front and rear fans have stopped, but the top fan is still 
> going.  All of the case fans are 3 pin.  Fans 2 and 3 still show 
> movement according to the inputs when pwm1 is off, but I can only see 
> one of the 4 fans moving.
> 
The chip has three pwm outputs, so if you have more than three fans I guess
that one pwm output could affect more than one fan.

How many fan counters (fanX_input) do you see on your system ?

I am a bit confused about the correlation of pwm to inputs and fan behavior.
Are you saying that pwm1 affects three of the four fans ? That would be a bit odd.
What are the values repoted by pwm2 and pwm3, and do you see anything change 
if you modify those values ?

Would it be possible to run pwmconfig and let us know what it reports ?

Thanks,
Guenter

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Phillip Susi]

On 02/22/2011 09:46 PM, Guenter Roeck wrote:

> How many fan counters (fanX_input) do you see on your system ?

The device reports 5 fan inputs and 3 pwm outputs, but the motherboard 
only has 4 fan headers: 2 4 pin and 2 3 pin.  From that I would expect 
pwm1 and pwm2 to be routed to the two 4 pin fan headers ( first one is 
cpu ), and the other two fan headers input to be connected to fan inputs 
3 and 4, with no control.

> I am a bit confused about the correlation of pwm to inputs and fan behavior.
> Are you saying that pwm1 affects three of the four fans ? That would be a bit odd.
> What are the values repoted by pwm2 and pwm3, and do you see anything change
> if you modify those values ?

Indeed, it is odd.  If I set pwm1 to a value of zero, all but one of the 
fans in the system stop.  Specifically the cpu fan, the front case fan, 
and the rear case fan stop, while the top case fan continues to spin.  I 
will need to open it back up to make sure, but my guess is that the top 
fan is connected to the second 4 pin fan header on the motherboard, but 
since the fan is only a 3 pin fan, it does not respond to the pwm output 
that is likely connected to it from pwm2.  The other two fan headers are 
only 3 pin headers, but since they seem to respond to changes to pwm1, I 
guess that the motherboard has on board FETs that respond to PWM1 to 
regulate the voltage on these two headers so their speed corresponds to 
the cpu fan speed.

> Would it be possible to run pwmconfig and let us know what it reports ?

It appears to agree with my assessment in that fan1 responds to pwm1.

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Guenter Roeck]

On Wed, Feb 23, 2011 at 12:00:57AM -0500, Phillip Susi wrote:
> On 02/22/2011 09:46 PM, Guenter Roeck wrote:
> 
> > How many fan counters (fanX_input) do you see on your system ?
> 
> The device reports 5 fan inputs and 3 pwm outputs, but the motherboard 
> only has 4 fan headers: 2 4 pin and 2 3 pin.  From that I would expect 
> pwm1 and pwm2 to be routed to the two 4 pin fan headers ( first one is 
> cpu ), and the other two fan headers input to be connected to fan inputs 
> 3 and 4, with no control.
> 
pwm2 is the CPU fan, actually.

How does the "sensors" output look like before and after you change pwm1 ?

> > I am a bit confused about the correlation of pwm to inputs and fan behavior.
> > Are you saying that pwm1 affects three of the four fans ? That would be a bit odd.
> > What are the values repoted by pwm2 and pwm3, and do you see anything change
> > if you modify those values ?
> 
> Indeed, it is odd.  If I set pwm1 to a value of zero, all but one of the 
> fans in the system stop.  Specifically the cpu fan, the front case fan, 
> and the rear case fan stop, while the top case fan continues to spin.  I 
> will need to open it back up to make sure, but my guess is that the top 
> fan is connected to the second 4 pin fan header on the motherboard, but 
> since the fan is only a 3 pin fan, it does not respond to the pwm output 
> that is likely connected to it from pwm2.  The other two fan headers are 
> only 3 pin headers, but since they seem to respond to changes to pwm1, I 
> guess that the motherboard has on board FETs that respond to PWM1 to 
> regulate the voltage on these two headers so their speed corresponds to 
> the cpu fan speed.
> 
Actually, sysfanout (pwm1) can be configured for DC output. You should see
the setting with pwm1_mode. If its value is 0, output is DC, otherwise
it is pwm. pwm2 and pwm3 only support pwm mode.

If configured for DC, the upper 6 bit of pwmX translate to output voltage.

Motherboard pinout (per its datasheet) is a bit odd. Looks like only the CPU fan
(pwm2) supports pwm mode. The other fan connectors don't have a pwm pin.
Fan connectors are labeled PWR_FAN1, CPU_FAN, CHA_FAN1 and CHA_FAN2. CHA_FAN1
has a 4-pin header, but the 4th pin is labeled as "+5V", not pwm.

> > Would it be possible to run pwmconfig and let us know what it reports ?
> 
> It appears to agree with my assessment in that fan1 responds to pwm1.

And pwm2 and pwm3 don't have an effect ?

Also - just trying to make sure - if you change pwm1, does it have an effect on 
pwm2 and/or pwm3 ? It should not, but who knows. And what are the values reported
by pwm2 and pwm3 ?

Thanks,
Guenter

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Phillip Susi]

On 2/23/2011 12:53 AM, Guenter Roeck wrote:
> Actually, sysfanout (pwm1) can be configured for DC output. You should see
> the setting with pwm1_mode. If its value is 0, output is DC, otherwise
> it is pwm. pwm2 and pwm3 only support pwm mode.
> 
> If configured for DC, the upper 6 bit of pwmX translate to output voltage.
> 
> Motherboard pinout (per its datasheet) is a bit odd. Looks like only the CPU fan
> (pwm2) supports pwm mode. The other fan connectors don't have a pwm pin.
> Fan connectors are labeled PWR_FAN1, CPU_FAN, CHA_FAN1 and CHA_FAN2. CHA_FAN1
> has a 4-pin header, but the 4th pin is labeled as "+5V", not pwm.

That is odd.  The 4th pin should either be pwm or no connect, not +5V.

>>> Would it be possible to run pwmconfig and let us know what it reports ?
>>
>> It appears to agree with my assessment in that fan1 responds to pwm1.
> 
> And pwm2 and pwm3 don't have an effect ?
> 
> Also - just trying to make sure - if you change pwm1, does it have an effect on 
> pwm2 and/or pwm3 ? It should not, but who knows. And what are the values reported
> by pwm2 and pwm3 ?

I was just playing directly with the knobs in /sys rather than running
sensors or pwmconfig.  Setting pwm2 and 3 to zero had no effect.
Setting pwm1 to zero caused fan1 and I think it was fan4 to drop to 0.
fan2 and fan3 were still reading as spinning, though I am pretty sure
that only one fan was actually still spinning.  I'll open the case
tonight and figure out which fan was connected to which header.

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Guenter Roeck]

Hi Phillip,

On Wed, 2011-02-23 at 10:38 -0500, Phillip Susi wrote:
> On 2/23/2011 12:53 AM, Guenter Roeck wrote:
> > Actually, sysfanout (pwm1) can be configured for DC output. You should see
> > the setting with pwm1_mode. If its value is 0, output is DC, otherwise
> > it is pwm. pwm2 and pwm3 only support pwm mode.
> > 
> > If configured for DC, the upper 6 bit of pwmX translate to output voltage.
> > 
> > Motherboard pinout (per its datasheet) is a bit odd. Looks like only the CPU fan
> > (pwm2) supports pwm mode. The other fan connectors don't have a pwm pin.
> > Fan connectors are labeled PWR_FAN1, CPU_FAN, CHA_FAN1 and CHA_FAN2. CHA_FAN1
> > has a 4-pin header, but the 4th pin is labeled as "+5V", not pwm.
> 
> That is odd.  The 4th pin should either be pwm or no connect, not +5V.
> 
The information is from the P8P67 Pro Manual. You can download it from
the ASUS web site. Interesting is that the fan connectors on the P8P67-M
are different - there are three 4-pin fan connectors, and the PWM pin on
all of them is labeled as "PWM".

> >>> Would it be possible to run pwmconfig and let us know what it reports ?
> >>
> >> It appears to agree with my assessment in that fan1 responds to pwm1.
> > 
> > And pwm2 and pwm3 don't have an effect ?
> > 
> > Also - just trying to make sure - if you change pwm1, does it have an effect on 
> > pwm2 and/or pwm3 ? It should not, but who knows. And what are the values reported
> > by pwm2 and pwm3 ?
> 
> I was just playing directly with the knobs in /sys rather than running
> sensors or pwmconfig.  Setting pwm2 and 3 to zero had no effect.
> Setting pwm1 to zero caused fan1 and I think it was fan4 to drop to 0.
> fan2 and fan3 were still reading as spinning, though I am pretty sure
> that only one fan was actually still spinning.  I'll open the case
> tonight and figure out which fan was connected to which header.

Difficult to imagine that two fans would show spinning if only one does.
I would think that if you drop pwm2 down to 0 (and of course set its
mode to manual) should result in fan2 speed dropping, and CPU
temperature to go up.

This actually leads to the next question: What temperature changes do
you see after playing with pwm settings ? If the CPU fan is affected, I
would think that CPU temperature should go up pretty fast.

Thanks,
Guenter



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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Phillip Susi]

On 2/23/2011 11:22 AM, Guenter Roeck wrote:
> Difficult to imagine that two fans would show spinning if only one does.
> I would think that if you drop pwm2 down to 0 (and of course set its
> mode to manual) should result in fan2 speed dropping, and CPU
> temperature to go up.
> 
> This actually leads to the next question: What temperature changes do
> you see after playing with pwm settings ? If the CPU fan is affected, I
> would think that CPU temperature should go up pretty fast.

It turns out that the cpu fan refuses to stop.  It seems to have a
minimum speed of around 1000 rpm ( max: 2000 ) and is controlled by
pwm2.  pwm1 controls both sysfans, and pwm3 does nothing.  The pwrfan
can not be controlled.

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Guenter Roeck]

On Wed, 2011-02-23 at 13:33 -0500, Phillip Susi wrote:
> On 2/23/2011 11:22 AM, Guenter Roeck wrote:
> > Difficult to imagine that two fans would show spinning if only one does.
> > I would think that if you drop pwm2 down to 0 (and of course set its
> > mode to manual) should result in fan2 speed dropping, and CPU
> > temperature to go up.
> > 
> > This actually leads to the next question: What temperature changes do
> > you see after playing with pwm settings ? If the CPU fan is affected, I
> > would think that CPU temperature should go up pretty fast.
> 
> It turns out that the cpu fan refuses to stop.  It seems to have a
> minimum speed of around 1000 rpm ( max: 2000 ) and is controlled by
> pwm2.  pwm1 controls both sysfans, and pwm3 does nothing.  The pwrfan
> can not be controlled.

Ok, that pretty much matches the information in the board manual.

Thanks,
Guenter



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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Jean Delvare]

On Wed, 23 Feb 2011 13:33:14 -0500, Phillip Susi wrote:
> On 2/23/2011 11:22 AM, Guenter Roeck wrote:
> > Difficult to imagine that two fans would show spinning if only one does.
> > I would think that if you drop pwm2 down to 0 (and of course set its
> > mode to manual) should result in fan2 speed dropping, and CPU
> > temperature to go up.
> > 
> > This actually leads to the next question: What temperature changes do
> > you see after playing with pwm settings ? If the CPU fan is affected, I
> > would think that CPU temperature should go up pretty fast.
> 
> It turns out that the cpu fan refuses to stop.  It seems to have a
> minimum speed of around 1000 rpm

This is very common for 4-pin fans. In fact, I don't think I was ever
able to stop a 4-pin CPU fan.

-- 
Jean Delvare

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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Phillip Susi]

On 2/23/2011 2:03 PM, Jean Delvare wrote:
> This is very common for 4-pin fans. In fact, I don't think I was ever
> able to stop a 4-pin CPU fan.

My last few AMD systems had no trouble stopping the fan.  Maybe it's an
Intel thing?


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

Re: [lm-sensors] [PATCH v4 00/10] hwmon: (w83627ehf) Add support

[Jean Delvare]

On Wed, 23 Feb 2011 14:09:08 -0500, Phillip Susi wrote:
> On 2/23/2011 2:03 PM, Jean Delvare wrote:
> > This is very common for 4-pin fans. In fact, I don't think I was ever
> > able to stop a 4-pin CPU fan.
> 
> My last few AMD systems had no trouble stopping the fan.  Maybe it's an
> Intel thing?

Possible, yes. All my systems with 4-pin CPU fans are Intel-based.

-- 
Jean Delvare

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