Messages in this thread | | | Date | Fri, 10 Dec 2021 00:16:43 -0800 | From | Zev Weiss <> | Subject | Re: [PATCH] hwmon: (nct6775) add support for TSI temperature registers |
| |
On Wed, Nov 10, 2021 at 03:14:39PM PST, Zev Weiss wrote: >These registers report CPU temperatures (and, depending on the system, >sometimes chipset temperatures) via the TSI interface on AMD systems. >They're distinct from most of the other Super-IO temperature readings >(CPUTIN, SYSTIN, etc.) in that they're not a selectable source for >monitoring and are in a different (higher resolution) format, but can >still provide useful temperature data. > >Signed-off-by: Zev Weiss <zev@bewilderbeest.net> >Tested-by: Renze Nicolai <renze@rnplus.nl> >--- > >This patch has been tested on NCT6779 and NCT6798[1] hardware on >(respectively) ASRock Rack ROMED8HM3 and X570D4U boards, and seems to >work as expected; the implementation for the other chips supported by >the driver is purely based on the datasheets and has not been tested >(for lack of available hardware). > >[1] Or at least, its chip ID registers identify it as an NCT6798 and >it seems to behave consistently with that, though it's actually >physically labeled as an NCT6796. > > drivers/hwmon/nct6775.c | 136 ++++++++++++++++++++++++++++++++++++++-- > 1 file changed, 130 insertions(+), 6 deletions(-) >
Ping...any thoughts/feedback on this patch?
Thanks, Zev
| |