nct6687d
nct6687d copied to clipboard
Fred78290
Upstreaming this to Linux kernel
Hi,
I've successfully built the module with my MSI MAG Tomahawk Z790 Wifi DDR4 motherboard and seems to work fine, although there are a few differences with the coretemp-isa-0000 values reported by lm-sensor (my running kernel is 6.0.16). What I'd like to ask is if you have tried to upstream this driver to the official Linux kernel project. The author of the nct6683 module is Guenter Roeck [email protected] as per the hwmon documentation in Linux 6.0.
What do you think?
Hi sorry for long delay to reply because I have also many private enterprise repositories flooding my mailbox 😂 So, I have never contacted Gunther. May be I must do it….
Hey @Fred78290, did you have a chance to contact anyone about upstreaming this?
A chance maybe… Some official linux distribution already include my work
Which one?
https://www.phoronix.com/news/Linux-6.5-HWMON
Newest HWMON patches to kernel 6.5 bring support for NCT6687D.
Looks like it is false alert. I dug into the kernel sources https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git/tree/drivers/hwmon/nct6683.c and found that driver already has support for NCT6687D but with different ID. Kernel driver sources: #define SIO_NCT6687_ID 0xd590 Fred78290's sources: #define SIO_NCT6687D_ID 0xd592 In-kernel support was added at 2020-12-02.
It was not a false alert... Kernel 6.5.2, MSI MAG B560M BAZOOKA, in-kernel driver:
nct6687-isa-0a20
Adapter: ISA adapter
VIN0: 1.02 V (min = +0.00 V, max = +0.00 V)
VIN1: 1.34 V (min = +0.00 V, max = +0.00 V)
VIN2: 624.00 mV (min = +0.00 V, max = +0.00 V)
VIN3: 624.00 mV (min = +0.00 V, max = +0.00 V)
VIN4: 672.00 mV (min = +0.00 V, max = +0.00 V)
VIN5: 1.06 V (min = +0.00 V, max = +0.00 V)
VIN6: 1.02 V (min = +0.00 V, max = +0.00 V)
VIN7: 1.54 V (min = +0.00 V, max = +0.00 V)
VCC: 3.36 V (min = +0.00 V, max = +0.00 V)
VSB: 3.36 V (min = +0.00 V, max = +0.00 V)
AVSB: 3.36 V (min = +0.00 V, max = +0.00 V)
VTT: 1.06 V (min = +0.00 V, max = +0.00 V)
VBAT: 3.18 V (min = +0.00 V, max = +0.00 V)
VREF: 1.82 V (min = +0.00 V, max = +0.00 V)
fan1: 767 RPM (min = 0 RPM)
fan2: 2752 RPM (min = 0 RPM)
fan3: 1089 RPM (min = 0 RPM)
fan4: 1090 RPM (min = 0 RPM)
fan5: 0 RPM (min = 0 RPM)
fan6: 0 RPM (min = 0 RPM)
fan7: 0 RPM (min = 0 RPM)
fan8: 0 RPM (min = 0 RPM)
fan9: 0 RPM (min = 0 RPM)
fan10: 0 RPM (min = 0 RPM)
PECI 0.0: +30.5°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = Intel PECI
Diode 0 (curr): +35.0°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = thermal diode
Thermistor 15: +31.5°C (low = +0.0°C)
(high = +127.0°C, hyst = +27.0°C)
(crit = +0.0°C) sensor = thermistor
PCH CHIP: +43.0°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C)
Thermistor 16: +31.0°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = thermistor
Thermistor 0: +23.0°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = thermistor
Thermistor 1: +24.5°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = thermistor
intrusion0: OK
beep_enable: disabled
If you compare it to Fred78290' sources, does it output the same (or maybe) similar values?
Ok, running with latest 6.5.5 kernel, I found that enabling nct6683D and nct6675F and compatibles like in the screenshot:
it detects the sensor chip:
nct6683: Found NCT6687D or compatible chip at 0x4e:0xa20
nct6683 nct6683.2592: NCT6687D EC firmware version 1.0 build 08/16/22
The output from sensors:
nct6687-isa-0a20
Adapter: ISA adapter
VIN0: 992.00 mV (min = +0.00 V, max = +0.00 V)
VIN1: 992.00 mV (min = +0.00 V, max = +0.00 V)
VIN2: 608.00 mV (min = +0.00 V, max = +0.00 V)
VIN3: 1.39 V (min = +0.00 V, max = +0.00 V)
VIN4: 672.00 mV (min = +0.00 V, max = +0.00 V)
VIN5: 1.33 V (min = +0.00 V, max = +0.00 V)
VIN6: 880.00 mV (min = +0.00 V, max = +0.00 V)
VIN7: 1.52 V (min = +0.00 V, max = +0.00 V)
VCC: 3.33 V (min = +0.00 V, max = +0.00 V)
CPU Fan: 0 RPM (min = 0 RPM)
fan2: 0 RPM (min = 0 RPM)
Back Fan: 1477 RPM (min = 0 RPM)
fan4: 0 RPM (min = 0 RPM)
fan5: 0 RPM (min = 0 RPM)
fan6: 0 RPM (min = 0 RPM)
Front Upper Fan: 1498 RPM (min = 0 RPM)
Front Lower Fan: 1220 RPM (min = 0 RPM)
fan9: 0 RPM (min = 0 RPM)
fan10: 0 RPM (min = 0 RPM)
PECI 0.0: +41.5°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = Intel PECI
Diode 0 (curr): +36.5°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = thermal diode
Thermistor 15: +38.0°C (low = +0.0°C)
(high = +127.0°C, hyst = +27.0°C)
(crit = +0.0°C) sensor = thermistor
PCH CHIP: +49.0°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C)
Thermistor 16: +36.5°C (low = +0.0°C)
(high = +0.0°C, hyst = +0.0°C)
(crit = +0.0°C) sensor = thermistor
intrusion0: OK
beep_enable: disabled
What’s interesting is that your fans are named, mine is not, it's just fan1-fan10, both connected and not.
What’s interesting is that your fans are named, mine is not, it's just fan1-fan10, both connected and not.
Yes, you can change that. Inside /etc/sensors.d you create a file with content similar to this:
chip "nct6687-isa-0a20"
label fan1 "CPU Fan"
label fan3 "Back Fan"
label fan7 "Front Upper Fan"
label fan8 "Front Lower Fan"
Just make sure which fanX corresponds to each actual fan in your case.
Problems I have with the upstream module that are fixed by this module:
- On my machine, I need to force load the module
- Sensor and fan names aren't very descriptive by default
- You can't set fan speeds
I think it's worth pursuing getting these adjustments into the upstream kernel @Fred78290.
