Re: [RFT PATCH 0/4] hwmon: k10temp driver improvements

[Holger Kiehl]

On Thu, 16 Jan 2020, Guenter Roeck wrote:

> This patch series implements various improvements for the k10temp driver.
> 
> Patch 1/4 introduces the use of bit operations.
> 
> Patch 2/4 converts the driver to use the devm_hwmon_device_register_with_info
> API. This not only simplifies the code and reduces its size, it also
> makes the code easier to maintain and enhance. 
> 
> Patch 3/4 adds support for reporting Core Complex Die (CCD) temperatures
> on Ryzen 3 (Zen2) CPUs.
> 
> Patch 4/4 adds support for reporting core and SoC current and voltage
> information on Ryzen CPUs.
> 
> With all patches in place, output on Ryzen 3900 CPUs looks as follows
> (with the system under load).
> 
> k10temp-pci-00c3
> Adapter: PCI adapter
> Vcore:        +1.36 V
> Vsoc:         +1.18 V
> Tdie:         +86.8ÂC  (high = +70.0ÂC)
> Tctl:         +86.8ÂC
> Tccd1:        +80.0ÂC
> Tccd2:        +81.8ÂC
> Icore:       +44.14 A
> Isoc:        +13.83 A
> 
> The patch series has only been tested with Ryzen 3900 CPUs. Further test
> coverage will be necessary before the changes can be applied to the Linux
> kernel.
> 
Here from my little Asrock A300 with a Ryzen 2400G:

   sensors
   k10temp-pci-00c3
   Adapter: PCI adapter
   Vcore:        +0.78 V  
   Vsoc:         +1.11 V  
   Tdie:         +44.8ÂC  (high = +70.0ÂC)
   Tctl:         +44.8ÂC  
   Icore:        +5.20 A  
   Isoc:         +2.17 A  

   nvme-pci-0100
   Adapter: PCI adapter
   Composite:    +41.9ÂC  (low  = -273.1ÂC, high = +80.8ÂC)
                          (crit = +80.8ÂC)
   Sensor 1:     +41.9ÂC  (low  = -273.1ÂC, high = +65261.8ÂC)
   Sensor 2:     +44.9ÂC  (low  = -273.1ÂC, high = +65261.8ÂC)

   nct6793-isa-0290
   Adapter: ISA adapter
   in0:                    +0.34 V  (min =  +0.00 V, max =  +1.74 V)
   in1:                    +1.84 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in2:                    +3.41 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in3:                    +3.39 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in4:                    +0.26 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in5:                    +0.14 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in6:                    +0.67 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in7:                    +3.39 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in8:                    +3.26 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in9:                    +1.84 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in10:                   +0.19 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in11:                   +0.14 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in12:                   +1.85 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in13:                   +1.72 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   in14:                   +0.20 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
   fan1:                     0 RPM  (min =    0 RPM)
   fan2:                   317 RPM  (min =    0 RPM)
   fan3:                     0 RPM  (min =    0 RPM)
   fan4:                     0 RPM  (min =    0 RPM)
   fan5:                     0 RPM  (min =    0 RPM)
   SYSTIN:                +113.0ÂC  (high =  +0.0ÂC, hyst =  +0.0ÂC)  sensor = thermistor
   CPUTIN:                 +59.5ÂC  (high = +80.0ÂC, hyst = +75.0ÂC)  sensor = thermistor
   AUXTIN0:                +45.0ÂC  (high =  +0.0ÂC, hyst =  +0.0ÂC)  ALARM  sensor = thermistor
   AUXTIN1:               +107.0ÂC    sensor = thermistor
   AUXTIN2:               +106.0ÂC    sensor = thermistor
   AUXTIN3:               +103.0ÂC    sensor = thermistor
   SMBUSMASTER 0:          +44.5ÂC  
   PCH_CHIP_CPU_MAX_TEMP:   +0.0ÂC  
   PCH_CHIP_TEMP:           +0.0ÂC  
   PCH_CPU_TEMP:            +0.0ÂC  
   intrusion0:            OK
   intrusion1:            ALARM
   beep_enable:           disabled

   amdgpu-pci-0300
   Adapter: PCI adapter
   vddgfx:           N/A  
   vddnb:            N/A  
   edge:         +44.0ÂC  (crit = +80.0ÂC, hyst =  +0.0ÂC)

Patches applied without any problem against Linus git tree.

Many thanks for this work!

Regards,
Holger