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

From: Peter Feuerer
Date: Sun Feb 24 2013 - 06:44:00 EST


Hi Boris,

thanks for your best wishes in the last mail, I'm feeling little better now.

Borislav Petkov writes:

On Sat, Feb 23, 2013 at 08:20:10PM +0100, Borislav Petkov wrote:
From: Borislav Petkov <bp@xxxxxxx>

The new step_wise thermal governor wasn't able to handle the one-trip
point design of acerhdf where we want to turn off the fan if we go under
the 'fanoff' temperature and to turn it on only after exceeding the
'fanon' temperature.

Do that by looking at the current fan state and return the temperature
accordingly.

Suggested-by: Zhang Rui <rui.zhang@xxxxxxxxx>
Cc: Peter Feuerer <peter@xxxxxxxx>
Cc: Andreas Mohr <andi@xxxxxxxx>
Cc: Alexander Lam <azl@xxxxxxxxxxxxxx>
Signed-off-by: Borislav Petkov <bp@xxxxxxx>

Ok, ignore this one for now - the suspend/resume path doesn't pan out
somehow, need to do some more handholding here.

Please test my last patch with the 4 trip points ;) - even if you don't really like it, it is working great! - And to be honest, I still prefer this solution!

Anyhow, did you test what Rui asked us to test?

--
kind regards,
--peter;
--
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/