Re: AMD Ryzen temp sensor

[Paul Goyette <paul%whooppee.com@localhost>]

Hi, Ian!

Thanks for the driver.  I took a preliminary look at the code, and it 
looks reasonable.

I'd probably re-word the HISTORY sections of the man pages a bit (for 
consistency with other code), but otherwise I see no reason not to 
import this for -current (or HEAD).  I'm not so sure about whether we 
should pull-up a new driver to -8 at this late stage, but I can ask, 
once we get a little bit of stable time on -current.



On Wed, 24 Jan 2018, Ian Clark wrote:

> Hi All,
>
> Due to what can only be described as incompetence on my part I'm
> currently running an AMD Ryzen with inadequate cooling. Due to this
> it'd be really useful to be able to get the CPU temp so I know when
> I'm going to need new hardware. ;)
>
> To that end I've ported the driver recently added to FreeBSD, and
> thought I'd share it here if it's any use to anyone. Configure with:
>
> amdsmn* at pci?
> amdzentemp* at amdsmn?
>
> src changes for 8_BETA are here:
>
> http://tsunami.ohmyno.co.uk/~ian/ryzen_temp_changes.tbz
>
> As I've never done anything like this before I've absolutely no idea
> how to submit it, but am happy to do so if someone points me in the
> right direction? I did have a gander round the site but probably
> missed the correct page. Do I need to submit a feature request pr?
>
> The driver is a port of the amd sys management network driver from
> freebsd, and the changes to amdtemp. I've split these into a new
> device to avoid a hard dependency on amdsmn within amdtemp.
>
> Cheers,
>
> Ian
>
> !DSPAM:5a68de93214055407574582!

+------------------+--------------------------+----------------------------+
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:          |
| (Retired)        | FA29 0E3B 35AF E8AE 6651 | paul at whooppee dot com   |
| Kernel Developer | 0786 F758 55DE 53BA 7731 | pgoyette at netbsd dot org |
+------------------+--------------------------+----------------------------+