All of lore.kernel.org
 help / color / mirror / Atom feed
* [lm-sensors] w83627hf.c pwm frequency patch
@ 2007-05-12  9:05 com.ea
  2007-05-14  9:58 ` Guillaume Millet
  2007-05-15 17:00 ` Jean Delvare
  0 siblings, 2 replies; 3+ messages in thread
From: com.ea @ 2007-05-12  9:05 UTC (permalink / raw)
  To: lm-sensors

Hello guys,

I came back home for a few days and could test the changes I made to the driver to support the pwm
frequency registers. Anyway I only have a w83697hf at hand to make the tests, and the registers in
627hf are a bit different. Does someone want to test it?

Btw, is it ok if I send a patch against 2.6.21.1?

Carlos

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

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

* Re: [lm-sensors] w83627hf.c pwm frequency patch
  2007-05-12  9:05 [lm-sensors] w83627hf.c pwm frequency patch com.ea
@ 2007-05-14  9:58 ` Guillaume Millet
  2007-05-15 17:00 ` Jean Delvare
  1 sibling, 0 replies; 3+ messages in thread
From: Guillaume Millet @ 2007-05-14  9:58 UTC (permalink / raw)
  To: lm-sensors

Hi

com.ea a écrit :
Does someone want to test it?

I tested with *sucess* your driver, compiled with 2.6.20-15-generic 
kernel headers (ubuntu feisty), for my w83697hf chip.

Some remarks.
Under Windows, MSI proposes CoreCenter utility which has two modes, 
either auto (maybe "thermal cruise" or "fan speed cruise" mode) either 
fix with 8 levels. With SpeedFan, I noted pwm#_freq that CoreCenter 
associates with a fix level and the range of fan rpm :

level pwm1_freq(kHz) fan rpm    pwm1
8/8   93.8           2678-2721  128-255
7/8   93.8           2596-2636  32-64
6/8   93.8           2481-2518  5-8
5/8   46.9           2250-2311  2
4/8   31.1           2163-2191  1
3/8   18.8           1939-2008  1
2/8   11.7           1776-1834  1
1/8   6.3                <1520  1

I wrote on the right the pwm on lm_sensors that gives about the same fan 
rpm.
Lastly, under 1520 rpm, fan1_input is wrong (the same under Windows).

Regards,
Guillaume


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

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

* Re: [lm-sensors] w83627hf.c pwm frequency patch
  2007-05-12  9:05 [lm-sensors] w83627hf.c pwm frequency patch com.ea
  2007-05-14  9:58 ` Guillaume Millet
@ 2007-05-15 17:00 ` Jean Delvare
  1 sibling, 0 replies; 3+ messages in thread
From: Jean Delvare @ 2007-05-15 17:00 UTC (permalink / raw)
  To: lm-sensors

Hi Carlos,

On Sat, 12 May 2007 11:05:39 +0200, com.ea wrote:
> Hello guys,
> 
> I came back home for a few days and could test the changes I made to the driver to support the pwm
> frequency registers. Anyway I only have a w83697hf at hand to make the tests, and the registers in
> 627hf are a bit different. Does someone want to test it?

Unfortunately I have a W83627THF variant, it uses DC for fan speed
control, so obviously your work won't have any effect on my chip.

> Btw, is it ok if I send a patch against 2.6.21.1?

No, please send a patch against 2.6.22-rc1 or later. The w83627hf
driver received a huge update between 2.6.21 and 2.6.22-rc1.

-- 
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] 3+ messages in thread

end of thread, other threads:[~2007-05-15 17:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-05-12  9:05 [lm-sensors] w83627hf.c pwm frequency patch com.ea
2007-05-14  9:58 ` Guillaume Millet
2007-05-15 17:00 ` Jean Delvare

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.