From: Guenter Roeck <linux@roeck-us.net> To: linux-hwmon@vger.kernel.org Cc: linux-kernel@vger.kernel.org, "Clemens Ladisch" <clemens@ladisch.de>, "Jean Delvare" <jdelvare@suse.com>, "Darren Salt" <devspam@moreofthesa.me.uk>, "Bernhard Gebetsberger" <bernhard.gebetsberger@gmx.at>, "Ken Moffat" <zarniwhoop73@googlemail.com>, "Ondrej Čerman" <ocerman@sda1.eu>, "Holger Kiehl" <Holger.Kiehl@dwd.de>, "Sebastian Reichel" <sebastian.reichel@collabora.com>, "Brad Campbell" <lists2009@fnarfbargle.com>, "Guenter Roeck" <linux@roeck-us.net> Subject: [PATCH v2 0/5] hwmon: k10temp driver improvements Date: Sat, 18 Jan 2020 09:26:10 -0800 Message-ID: <20200118172615.26329-1-linux@roeck-us.net> (raw) This patch series implements various improvements for the k10temp driver. Patch 1/5 introduces the use of bit operations. Patch 2/5 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/5 adds support for reporting Core Complex Die (CCD) temperatures on Ryzen 3 (Zen2) CPUs. Patch 4/5 adds support for reporting core and SoC current and voltage information on Ryzen CPUs. Patch 5/5 removes the maximum temperature from Tdie for Ryzen CPUs. It is inaccurate, misleading, and it just doesn't make sense to report wrong information. With all patches in place, output on Ryzen 3900X 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 Tctl: +86.8°C Tccd1: +80.0°C Tccd2: +81.8°C Icore: +44.14 A Isoc: +13.83 A The voltage and current information is limited to Ryzen CPUs. Voltage and current reporting on Threadripper and EPYC CPUs is different, and the reported information is either incomplete or wrong. Exclude it for the time being; it can always be added if/when more information becomes available. Tested with the following Ryzen CPUs: 1300X A user with this CPU in the system reported somewhat unexpected values for Vcore; it isn't entirely if at all clear why that is the case. Overall this does not warrant holding up the series. 1600 1800X 2200G 2400G 3800X 3900X 3950X v2: Added tested-by: tags as received. Don't display voltage and current information for Threadripper and EPYC. Stop displaying the fixed (and wrong) maximum temperature of 70 degrees C for Tdie on model 17h/18h CPUs.
next reply index Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-01-18 17:26 Guenter Roeck [this message] 2020-01-18 17:26 ` [PATCH v2 1/5] hwmon: (k10temp) Use bitops Guenter Roeck 2020-01-18 17:26 ` [PATCH v2 2/5] hmon: (k10temp) Convert to use devm_hwmon_device_register_with_info Guenter Roeck 2020-01-18 17:26 ` [PATCH v2 3/5] hwmon: (k10temp) Report temperatures per CPU die Guenter Roeck 2020-01-18 17:26 ` [PATCH v2 4/5] hwmon: (k10temp) Show core and SoC current and voltages on Ryzen CPUs Guenter Roeck 2020-01-18 17:26 ` [PATCH v2 5/5] hwmon: (k10temp) Don't show temperature limits on Ryzen (Zen) CPUs Guenter Roeck 2020-01-19 0:33 ` [PATCH v2 0/5] hwmon: k10temp driver improvements Ken Moffat 2020-01-19 0:48 ` Guenter Roeck 2020-01-19 1:08 ` Ken Moffat 2020-01-19 3:13 ` Brad Campbell 2020-01-19 10:18 ` Jonathan McDowell 2020-01-19 15:46 ` Guenter Roeck 2020-01-19 19:38 ` Jonathan McDowell 2020-01-19 13:38 ` Holger Kiehl 2020-01-19 15:49 ` Guenter Roeck
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200118172615.26329-1-linux@roeck-us.net \ --to=linux@roeck-us.net \ --cc=Holger.Kiehl@dwd.de \ --cc=bernhard.gebetsberger@gmx.at \ --cc=clemens@ladisch.de \ --cc=devspam@moreofthesa.me.uk \ --cc=jdelvare@suse.com \ --cc=linux-hwmon@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=lists2009@fnarfbargle.com \ --cc=ocerman@sda1.eu \ --cc=sebastian.reichel@collabora.com \ --cc=zarniwhoop73@googlemail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Linux-Hwmon Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/linux-hwmon/0 linux-hwmon/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 linux-hwmon linux-hwmon/ https://lore.kernel.org/linux-hwmon \ linux-hwmon@vger.kernel.org public-inbox-index linux-hwmon Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kernel.vger.linux-hwmon AGPL code for this site: git clone https://public-inbox.org/public-inbox.git