Difference between revisions of "Talk:Patch for controlling fan speed"

From ThinkWiki
Jump to: navigation, search
(its a talk topic)
 
Line 6: Line 6:
 
a suggestion to improve the patch.
 
a suggestion to improve the patch.
 
--Jiang
 
--Jiang
 +
----
 +
 +
I'd say it's a bug in gkrellm. It should parse the line header rather than relying on line numbers. But feel free to change (and test) the patch if you wish.
 +
 +
--[[User:Thinker|Thinker]] 05:14, 26 Oct 2005 (CEST)
 +
----

Revision as of 04:14, 26 October 2005

I can confirm that it works on Thinkpad T43 here. However after applying the patch, the fan speed monitor of gkrellm 2.2.7 cannot read value correctly. Maybe we gkrellm is reading the second line for speed but instead find the line for level, so it got confused? Would it be possible to interchange the lines so that speed still appears in the second line and level appears in the third instead? I'm no coder, just a suggestion to improve the patch. --Jiang


I'd say it's a bug in gkrellm. It should parse the line header rather than relying on line numbers. But feel free to change (and test) the patch if you wish.

--Thinker 05:14, 26 Oct 2005 (CEST)