Re: [PATCH] acerhdf: Fix fan activation with new thermal governor

Zhang Rui
2013-03-06T07:30:36Z

On Wed, 2013-03-06 at 08:00 +0100, Borislav Petkov wrote:
> On Wed, Mar 06, 2013 at 11:47:20AM +0800, Zhang Rui wrote:
> > > So this now does not have any of the Schmitt trigger equivalent (?)
> > > governor changes that were nicely discussed in this activity -
> > > So what do we want to have committed, and what do we not,
> >
> > the problem should have been fixed in 3.9-rc1 by
> > commit b8bb6cb999858043489c1ddef08eed2127559169
> > Author: Zhang Rui <rui.zhang@intel.com>
> > Date: Thu Nov 22 15:45:02 2012 +0800
> >
> > step_wise: Unify the code for both throttle and dethrottle
> >
> > Signed-off-by: Zhang Rui <rui.zhang@intel.com>
> >
> > I'll send Alexander' patch for 3.8 stable.
>
> Hold on, do I understand it correctly that this is yet another patch
> that "fixes" it?

sorry I should be clearer.
the problem that alexander' patch fixes, does not exist in 3.9-rc1
because the problem is fixed by this commit
b8bb6cb999858043489c1ddef08eed2127559169.

so what we need is, either 3.8.x kernel + alexander's patch, or clean
3.9-rc1.

> If so, maybe we should test 3.9-rc1 which contains the
> said patch above? Yes, no?
>
> Also, I don't see Alexander's patch applied anywhere so if it isn't
> upstream, you can't send it to stable.
>
then I'll back port b8bb6cb999858043489c1ddef08eed2127559169 for 3.8
stable.

thanks,
rui