Messages in this thread | | | Subject | Re: [PATCH] thermal: intel_pch_thermal: Add PCI ids for Lewisburg PCH. | From | Daniel Lezcano <> | Date | Thu, 12 Nov 2020 12:29:19 +0100 |
| |
On 28/10/2020 21:21, Andres Freund wrote: > Hi, > > On 2020-01-16 11:41:34 -0800, Srinivas Pandruvada wrote: >> On Thu, 2020-01-16 at 10:42 -0800, Andres Freund wrote: >>> Hi, >>> >>> On 2020-01-16 05:53:13 -0800, Srinivas Pandruvada wrote: >>>> On Wed, 2020-01-15 at 10:44 -0800, Andres Freund wrote: >>>>> I noticed that I couldn't read the PCH temperature on my >>>>> workstation >>>>> (C620 series chipset, w/ 2x Xeon Gold 5215 CPUs) directly, but >>>>> had to >>>>> go >>>>> through IPMI. Looking at the data sheet, it looks to me like the >>>>> existing intel PCH thermal driver should work without changes for >>>>> Lewisburg. >>>> Does the temperature reading match with what you read via IPMI? >>> >>> It does: >>> >>> root@awork3:~# ipmitool sdr|grep ^PCH >>> PCH Temp | 58 degrees C | ok >>> >>> andres@awork3:~$ cat /sys/class/thermal/thermal_zone0/type >>> pch_lewisburg >>> andres@awork3:~$ cat /sys/class/thermal/thermal_zone0/temp >>> 58000 >>> >>> And if I generate some load, it rises for both: >>> root@awork3:~# ipmitool sdr|grep ^PCH >>> PCH Temp | 60 degrees C | ok >>> andres@awork3:~$ cat /sys/class/thermal/thermal_zone0/temp >>> 60000 >>> >> Thanks for the test. >> >> Rui can add his ACK. > > Ping? Looks like this got lost somewhere?
Waiting for Rui's ack :)
-- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
| |