lkml.org 
[lkml]   [2014]   [Dec]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/3] i8k: Remove laptop specific config data (fan_mult, fan_max) from driver
From
Date
On Thu, 18 Dec 2014 12:08:58 +0100, Pali Rohár said:

> So your BIOS cannot report nominal_rpm and because your machine=20
> is not in dmi list, all 3 patches do nothing for your machine.
>
> But you need to set multiplier to 1, right?
>
> What about this patch? (on top of 3/3)
>
> --- a/drivers/char/i8k.c
> +++ b/drivers/char/i8k.c
> @@ -850,6 +850,10 @@ static int __init i8k_probe(void)
> */
> for (fan = 0; fan < I8K_FAN_COUNT; ++fan) {
> i8k_fan_mult[fan] = I8K_FAN_DEFAULT_MULT;
> + if (i8k_get_fan_rpm(fan) > I8K_FAN_MAX_RPM) {
> + i8k_fan_mult[fan] = 1;
> + continue;
> + }
> for (val = 0; val < 256; ++val) {


Dell Latitude E6530, linux-next 20141208 plus the 3 patches you
posted on Nov 30 (support labels), plus the 4 of 3 patches in
this go around, and sensors reports:

i8k-virtual-0
Adapter: Virtual device
fan2: 2774 RPM

and under CPU load, that number rises to 3200 or so, so it seems to be
working....

So you probably should fold this into the current set of patches, and
feel free to put this in there:

Tested-By: Valdis Kletnieks <valdis.kletnieks@vt.edu>
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2014-12-18 16:21    [W:0.139 / U:0.836 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site