Messages in this thread | | | Date | Wed, 26 Sep 2012 00:24:17 +0200 | From | Borislav Petkov <> | Subject | Re: [PATCH 1/1] x86: Added support for Acer Aspire 5755G fan control. |
| |
Adding Peter.
On Tue, Sep 25, 2012 at 12:41:11PM +0300, Tero Keski-Valkama wrote: > 2012/9/25 Borislav Petkov <bp@alien8.de>: > > On Tue, Sep 25, 2012 at 10:34:13AM +0300, Tero Keski-Valkama wrote: > > > > But before we go with this any further: you mentioned some issues still > > with acerhdf - you don't want to turn off your fan but to turn it to > > full? > > > > I think in this case, you want to simply not load the driver and use the > > BIOS settings, no? > > > > -- > > Regards/Gruss, > > Boris. > > Technically I think my original issue is about thermal zones, and BIOS > control doesn't solve it as is. However, the patch should be valid, > even if it doesn't solve my original issue. > > The thing is that passive cooling thermal zones throttle all the cores > to stone age, to 800 MHz, before the fan even really starts powering > up from the low default level.
I don't understand: you want the thermal zones to not throttle your cpu to 800 Mhz and/or the fan to start sooner?
> So, my original problem would be solved > by: > a) Direct control of the fan, to be able to put it on full for example > through userspace control,
That's always a bad idea, especially if userspace dies on you.
> b) Changing the active cooling thermal zones, so that the fan speeds > up earlier, or
Ok, I see, here's what you want.
> c) Changing the passive cooling (CPU throttling) thermal zones, so > that it doesn't prevent fan from speeding up in the first place. > > At the moment, I don't see any way to change the thermal zones through > existing interfaces for this laptop, and I don't see any potential > thermal zone controls, or manual fan control option in the Embedded > Controller registry.
I'll let Peter have a look at the patch too.
@Peter, it is at: http://lkml.org/lkml/2012/9/25/92
Btw, the commit message starts with "x86:... " and it should be "acerhdf:.. " Peter, please change that when handling the patch further.
Also, you've added yourself to the copyright - this means that you're pretty much going to get all future email about acerhdf. Do you really want that?
-- Regards/Gruss, Boris.
| |