-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upstreaming this to Linux kernel #52
Comments
Hi sorry for long delay to reply because I have also many private enterprise repositories flooding my mailbox 😂 |
Hey @Fred78290, did you have a chance to contact anyone about upstreaming this? |
A chance maybe… |
Which one? |
https://www.phoronix.com/news/Linux-6.5-HWMON Newest HWMON patches to kernel 6.5 bring support for NCT6687D. |
The upstream patch is the one from this repository or different code? |
Looks like it is false alert. |
It was not a false alert...
|
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:
The output from
|
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:
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:
I think it's worth pursuing getting these adjustments into the upstream kernel @Fred78290. |
Any updates on this? |
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 linux@roeck-us.net as per the hwmon documentation in Linux 6.0.
What do you think?
The text was updated successfully, but these errors were encountered: