Detect AMD Family 0x0f one or two sensors per core

[cy]

Summary

amdtemp assumes all AMD family 0x0f have two sensors per core. Not all do. This patch, obtained from NetBSD, correctly determines one or two sensors per core.

In addition, this revision corrects temp = ((val >> 21) & 0x7ff) * 5 / 4 to use the macros instead of raw values.

The scope of this revision does not attempt to rename functions or macros which support family 0x0f. This should be considered in another revision. I will submit another revision for that at a later date.

Test Plan

This has been tested and is running on what was my three family 0x0f machines (now two family 0x0f machines and one family 0x10 machine).

[cem]

sys/dev/amdtemp/amdtemp.c

389

This seems to be the "CmpCap [13:12]" field specified for F3xE8 Northbridge Capabilities Register in the Family 10h BKDG; it might be good to define macro names for the magic number 12 and 0x3 mask.

A zero value indicates a single core on the node. 0x1 => 2 cores, 0x2 => 3 cores, 0x3 => 4 cores, FWIW.

709–710

I think this unrelated change should be addressed separately.

[cy]

$JOB is taking precedence this week. I'll update this revision Friday or later.

sys/dev/amdtemp/amdtemp.c

389

According to 0x0f BKDG (26094.PDF), 00b == single core , 01b == dual, 10b == reserved, 11b = reserved.

We should do as you suggest for 0x10 though. I will update the 0x10 code as well.

[cy]

NetBSD determines the number of sensors and cores differently than we do. I will need more time to compare our approach with theirs.