Re: [PATCH RFT] hwmon: (nct6755) Add support for NCT6799D

[Christian Lamparter <chunkeey@gmail.com>]

Hi Guenter,

On 12/30/22 18:50, Guenter Roeck wrote:
 > On Fri, Dec 30, 2022 at 06:32:23PM +0100, Christian Lamparter wrote:
 >> On 12/28/22 14:57, Guenter Roeck wrote:
 >>> NCT6799D is mostly compatible to NCT6798D, with minor variations.
 >>>
 >>> Note that NCT6798D and NCT6799D have a new means to select temperature
 >>> sources, and to report temperatures from those sources. This is not
 >>> currently implemented, meaning that most likely not all temperatures
 >>> are reported.
 >>>
 >>> Initial support. Compile tested only.
 >>
 >> FYI: This ID is compatible with the NCT6796D-S on the ASRock X670E PG Lighting Mainboard.
 >> (Previous discussion: "[PATCH v1] hwmon: (nct6775) Add chip ID for NCT6796D-S"
 >> <https://www.spinics.net/lists/linux-hwmon/msg17785.html>)
 >>
 >> With this patch applied, the nct6775 module loads and produces the following output when loaded.
 >>
 >> | [28152.009197] nct6775: Found NCT6799D or compatible chip at 0x2e:0x290
 >>
 >> and lm-sensors also enumerates the sensors and finally all six fans.
 >> So this is definitively an improvement over my patch.
 >>
 >
 > Thanks a lot for testing. I'd suspect that NCT6796D-S was mislabeled
 > in the user manual. That would not be the first time.
The chips sits behind the GPU now. But I made a blurry photo before
I added the GPU though :-). The IC's label shows "Nuvoton NCT6796D-S"
<https://gist.github.com/chunkeey/89aeb85d6f71e7a2da9344615303c203>
(the chip is soldered at an 45° angle, hence I rotated the picture)

I got that MB because of the phoronix review:
<https://www.phoronix.com/review/asrock-x670e-pg>
(NB: The review also noted the missing sensor support...
But not for much longer ;-) )

Michael Larabel also made (better) pictures:
<https://www.phoronix.com/image-viewer.php?id=asrock-x670e-pg&image=asrock_x670pglightning_3_lrg>

(While not the same motherboard, this Japanese review site has a
really sharp picture from the ASRock X670E Taichi NCT6796D-S SuperIO
<https://www.gdm.or.jp/x670etaichi_57_1024x768>)

That all being said: Other users in the github thread for the
lm-sensors project:
<https://github.com/lm-sensors/lm-sensors/issues/416#issuecomment-1276620580>
clearly stated that their ASUS X670E TUF came with a "NCT6799D-R".

Obviously, I'm totally fine with what the module calls it:
"NCT6799D or compatible".

 >> |# sensors
 >> |
 >> |nct6799-isa-0290
 >> |Adapter: ISA adapter
 >> |in0:                   720.00 mV (min =  +0.00 V, max =  +1.74 V)
 >> |in1:                     1.82 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in2:                     3.39 V  (min =  +2.98 V, max =  +3.63 V)
 >> |in3:                     3.28 V  (min =  +2.98 V, max =  +3.63 V)
 >> |in4:                     1.65 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in5:                     1.05 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in6:                     1.41 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in7:                     3.39 V  (min =  +2.98 V, max =  +3.63 V)
 >> |in8:                     3.23 V  (min =  +2.70 V, max =  +3.63 V)
 >> |in9:                   904.00 mV (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in10:                  600.00 mV (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in11:                  608.00 mV (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in12:                    1.04 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in13:                  896.00 mV (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |in14:                    1.26 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
 >> |fan1:                   891 RPM  (min =    0 RPM)
 >> |fan2:                     0 RPM  (min =    0 RPM)
 >> |fan3:                  1345 RPM  (min =    0 RPM)
 >> |fan4:                   774 RPM  (min =    0 RPM)
 >> |fan5:                     0 RPM  (min =    0 RPM)
 >> |fan7:                     0 RPM  (min =    0 RPM)
 >
 > Interesting. Do you have three fans connected, and the motherboard has
 > seven fans ?

There are six 4-pin fan ports on the motherboard (the list above has fan7
but not fan6. So there are still in total: six fans).

Three fans (well - one AIO with a pump and two extra fans) are permanently installed
into the case. But I have connected an old, external CPU-Fan just to check the function
of fan2,fan5 and fan7 tachometers. So this worked :).

 >
 >> |SYSTIN:                 +29.5°C    sensor = thermistor
 >> |CPUTIN:                 +29.0°C  (high = +80.0°C, hyst = +75.0°C)  sensor = thermistor
 >> |AUXTIN0:                 +4.0°C  (high = +67.0°C, hyst = +110.0°C)  sensor = thermistor
 >
 > Hmm, that hysteresis value seems wrong. I'll need to check if the
 > register addresses are correct.
Do you have the datasheet? I'm looking for it (I've also asked Nuvoton).
But so far no response. If you need a register range dump,
please let me know.

(AFAIK, all AMD5 X670E broads have dual chipset "chips". I think CPUTIN is
one of the two "PROM21" chips. Don't know what AUXTIN0 is...)
 >> |AUXTIN1:                +15.0°C    sensor = thermistor
 >> |AUXTIN2:                +15.0°C    sensor = thermistor
 >> |AUXTIN3:                +13.0°C    sensor = thermistor
 >
 > Looks like either those are not connected, or there is a temperature
 > offset which is not applied, or it is set to wrong values (the chip
 > does support temperature offset registers).
yes, I too think these are not connected. The values don't move, even after
a prolonged stresstest. Furthermore, neither the UEFI BIOS nor the Windows
utility has additional temperature gauges.

Regards,
Christian