Re: [PATCH 1/1] x86: Added support for Acer Aspire 5755G fan control.

[Borislav Petkov]

On Tue, Sep 25, 2012 at 10:34:13AM +0300, Tero Keski-Valkama wrote:
> This patch is a patch to add support for Acer Aspire 5755G fan control
> to acerhdf module for the latest BIOS version available in the Acer
> web site, V1.20.
> 
> <DSDT disassembly dump>
> ...
> Offset (0xA7),
> OSTT,   8,
> OSST,   8,
> THLT,   8,
> TCNL,   8,
> MODE,   1,
>     ,   2,
> INIT,   1,
> FAN1,   1,
> FAN2,   1,
> FANT,   1,
> SKNM,   1,
> SDTM,   8,
> FSSN,   4,
> FANU,   4,
> ...
> </DSDT disassembly dump>
> 
> By experiment it was found that setting the register address 0xAB to
> 0x00 (FAN1 -> 0) caused the fan to stop, and setting it to the
> original value of 0x08 (FAN1 -> 1) caused the fan to operate
> automatically in BIOS control.
> 
> By observation the value in the Embedded Controller table which
> correlated best with the temperature, and changes up and down along
> with real temperature was the register address 0xB4 (SKTC).
> 
> This patch and related evidence and dumps were sent to the original
> authors of the module, peter and bp.
> 
> The original author of the patch: Tero Keski-Valkama

No need for that - the Signed-off-by and patch authorship annotation in
git says that already.

> 
> Signed-off-by: Tero Keski-Valkama <tero.keski-valkama@xxxxxxxx>

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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/