hwmon: (k10temp) Don't show temperature limits on Ryzen (Zen) CPUs
authorGuenter Roeck <linux@roeck-us.net>
Fri, 17 Jan 2020 14:43:20 +0000 (06:43 -0800)
committerGuenter Roeck <linux@roeck-us.net>
Thu, 23 Jan 2020 21:15:11 +0000 (13:15 -0800)
The maximum Tdie or Tctl is not published for Ryzen CPUs. What is
known, however, is that the traditional value of 70 degrees C is no
longer correct. On top of that, the limit applies to Tctl, not to Tdie.
Displaying it in either context is meaningless, confusing, and wrong.
Stop doing it.

Tested-by: Brad Campbell <lists2009@fnarfbargle.com>
Tested-by: Holger Kiehl <holger.kiehl@dwd.de>
Tested-by: Michael Larabel <michael@phoronix.com>
Tested-by: Jonathan McDowell <noodles@earth.li>
Tested-by: Ken Moffat <zarniwhoop73@googlemail.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/k10temp.c

index b961e12..4a470b5 100644 (file)
@@ -355,7 +355,7 @@ static umode_t k10temp_is_visible(const void *_data,
                        }
                        break;
                case hwmon_temp_max:
-                       if (channel)
+                       if (channel || data->show_tdie)
                                return 0;
                        break;
                case hwmon_temp_crit: