Messages in this thread | | | Date | Mon, 17 Oct 2011 14:13:00 +0200 | From | Borislav Petkov <> | Subject | Re: [PATCH 3/3] acerhdf: Lowered default temp fanon/fanoff values |
| |
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.
| |