[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1361287675.2323.26.camel@rzhang1-mobl4>
Date: Tue, 19 Feb 2013 23:27:55 +0800
From: Zhang Rui <rui.zhang@...el.com>
To: Borislav Petkov <bp@...en8.de>
Cc: Peter Feuerer <peter@...e.net>, Alexander Lam <azl@...rew.cmu.edu>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Andreas Mohr <andi@...as.de>
Subject: Re: thermal governor: does it actually work??
On Tue, 2013-02-19 at 16:05 +0100, Borislav Petkov wrote:
> On Tue, Feb 19, 2013 at 10:51:04PM +0800, Zhang Rui wrote:
> > Okay, here I see the problem, say you want to turn on the fan at 60C and
> > turn it off at 55C, is this what you want?
> > hmmm, is it possible to do some tricks in acerhdf driver?
> > say,
> > only one active trip point as before. but change acerhdf_get_trip_temp
> > to:
> > acerhdf_get_trip_temp()
> > {
> > if (fan is on)
> > return 55;
> > else
> > return 60;
> > }
> >
> > In this way, when the fan is off and the temperature is raising, the fan
> > will be turned on at 60C because we have an active trip point of 60C.
> > And when the fan is turned on and the temperature starts to drop, the
> > fan will be turned off at 55C.
>
> Makes a sense to me. I was questioning the need for 4 trip points too.
I do not understand why you need 4 trip points for the issue above,
unless I missed another gap here.
> We probably would need a second, critical trip point though, just in
> case.
>
yeah, it is okay to have a critical trip point.
thanks,
rui
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists