Re: [PATCH 3/3] acerhdf: Lowered default temp fanon/fanoff values

[Borislav Petkov <bp@alien8.de>]

On Sun, Oct 16, 2011 at 11:44:23AM +0200, Peter Feuerer wrote:
> From: Peter Feuerer <peter@piie.net>
> 
> Due to new supported hardware, of which the actual temperature limits of
> processor, harddisk and other components are unknown, it feels safer
> with lower fanon / fanoff settings.
> 
> It won't change much for most people, already using acerhdf, as they use
> their own fanon/fanoff variable settings when loading the module.
> 
> Furthermore seems like kernel and userspace tools have been improved
> to work more efficient and netbooks don't get so hot anymore.
> 
> Signed-off-by: Peter Feuerer <peter@piie.net>
> Cc: Borislav Petkov <petkovbb@gmail.com>
> Cc: Matthew Garrett <mjg@redhat.com>

I'm wondering whether the bios_tbl in 2/3 is not growing out of
proportions currently and since a lot of the fields are repeating, it
might make sense to fold its functionality in a switch-case statement.
Maybe in another patch series...

But those patches look ok to me:

Acked-by: Borislav Petkov <bp@alien8.de>

Thanks.

-- 
Regards/Gruss,
Boris.