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

From: Borislav Petkov
Date: Mon Oct 17 2011 - 08:13:06 EST


On Sun, Oct 16, 2011 at 11:44:23AM +0200, Peter Feuerer wrote:
> From: Peter Feuerer <peter@xxxxxxxx>
>
> 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@xxxxxxxx>
> Cc: Borislav Petkov <petkovbb@xxxxxxxxx>
> Cc: Matthew Garrett <mjg@xxxxxxxxxx>

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@xxxxxxxxx>

Thanks.

--
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/