Re: [PATCH] i8k: Add support for temperature sensor labels

[Steven Honeyman]

On 29 November 2014 at 16:09, Pali RohÃr <pali.rohar@xxxxxxxxx> wrote:
> On Saturday 29 November 2014 17:04:07 Pali RohÃr wrote:
>> This patch adds labels for temperature sensors if SMM function
>> with EAX register 0x11a3 reports it. These informations was
>> taken from DOS binary NBSVC.MDM.
>>
>> Signed-off-by: Pali RohÃr <pali.rohar@xxxxxxxxx>
>> ---
>>  drivers/char/i8k.c |  110
>> +++++++++++++++++++++++++++++++++++++++++----------- 1 file
>> changed, 88 insertions(+), 22 deletions(-)
>
> I tested patch on Latitude E6440 and i8k CPU & GPU temps match
> intel coretemp & amd radeion temps.
>
> But I would like if somebody with other Dell laptop can test if
> temperature labels are correct...

Tested and working on E6540:
(I manually masked the serial/service tag)

-----Before-----

*[steven@e6540 linux]$ cat /proc/i8k
1.0 A12 6xxxxxx 44 -22 1 -22 3223 -1 -22
*[steven@e6540 linux]$ sensors
i8k-virtual-0
Adapter: Virtual device
fan2:        3223 RPM
temp1:        +46.0ÂC
temp2:        +44.0ÂC
temp3:        +38.0ÂC

coretemp-isa-0000
Adapter: ISA adapter
Physical id 0:  +49.0ÂC  (high = +84.0ÂC, crit = +100.0ÂC)
Core 0:         +45.0ÂC  (high = +84.0ÂC, crit = +100.0ÂC)
Core 1:         +49.0ÂC  (high = +84.0ÂC, crit = +100.0ÂC)

-----After-----

*[steven@e6540 ~]$ cat /proc/i8k
1.0 A12 6xxxxxx 47 -22 1 -22 3176 -1 -22
*[steven@e6540 ~]$ sensors
i8k-virtual-0
Adapter: Virtual device
fan2:        3176 RPM
CPU:          +47.0ÂC
Ambient:      +45.0ÂC
SODIMM:       +38.0ÂC

coretemp-isa-0000
Adapter: ISA adapter
Physical id 0:  +47.0ÂC  (high = +84.0ÂC, crit = +100.0ÂC)
Core 0:         +45.0ÂC  (high = +84.0ÂC, crit = +100.0ÂC)
Core 1:         +46.0ÂC  (high = +84.0ÂC, crit = +100.0ÂC)
--
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/