From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ@public.gmane.org Subject: [Bug 80901] [NVCF] PWM fan speed too high Date: Wed, 09 Jul 2014 16:35:04 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1077725669==" Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: nouveau-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "Nouveau" To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org List-Id: nouveau.vger.kernel.org --===============1077725669== Content-Type: multipart/alternative; boundary="1404923704.1Dd31.11650"; charset="us-ascii" --1404923704.1Dd31.11650 Date: Wed, 9 Jul 2014 16:35:04 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" https://bugs.freedesktop.org/show_bug.cgi?id=80901 --- Comment #15 from Martin Peres --- (In reply to comment #14) > The problem I see with trip-points is that those allow to set a fixed PWM > value when the sensors detect a certain temperature. The nouveau driver > instead raises the fan speed "continuously" after a certain temperature is > reached (would we need infinite trip-points for that?). > Please correct any error in my understanding here :) > > Maybe we can use the trip points as a start/end point and then move > autonomously disregarding the point[0-*]_pwm value? > > > For now, if you agree, I will move the sysfs values as suggested (and change > the documentation), hopefully before/during next weekend. > So this bug will still have an almost "clean" patch for anyone (included me) > who want it. > Then I will ask around in the lm-sensors mailing list (is it the correct > one?). Hopefully someone can point me to a better solution. > > Thanks for the review! :) Yes, you understood my review and have a perfect todo list! Good luck with it :) -- You are receiving this mail because: You are the assignee for the bug. --1404923704.1Dd31.11650 Date: Wed, 9 Jul 2014 16:35:04 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8"

Comment # 15 on bug 80901 from 
(In reply to comment #14)
> The problem I see with trip-points is that those allow to set a fixed PWM
> value when the sensors detect a certain temperature. The nouveau driver
> instead raises the fan speed "continuously" after a certain temperature is
> reached (would we need infinite trip-points for that?). 
> Please correct any error in my understanding here :)
> 
> Maybe we can use the trip points as a start/end point and then move
> autonomously disregarding the point[0-*]_pwm value?
> 
> 
> For now, if you agree, I will move the sysfs values as suggested (and change
> the documentation), hopefully before/during next weekend. 
> So this bug will still have an almost "clean" patch for anyone (included me)
> who want it.
> Then I will ask around in the lm-sensors mailing list (is it the correct
> one?). Hopefully someone can point me to a better solution.
> 
> Thanks for the review! :)

Yes, you understood my review and have a perfect todo list! Good luck with it
:)

You are receiving this mail because:
  • You are the assignee for the bug.
--1404923704.1Dd31.11650-- --===============1077725669== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Nouveau mailing list Nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org http://lists.freedesktop.org/mailman/listinfo/nouveau --===============1077725669==--