Skip to content
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

Add preliminary support for Nuvoton NCT6799D #846

Merged
merged 1 commit into from
Nov 10, 2022

Conversation

Alexx999
Copy link
Contributor

@Alexx999 Alexx999 commented Oct 16, 2022

Partially addresses #843
On X670E Hero it allows controlling fans (my primary goal) but the temps seem wrong so this is definitely not 100% complete just yet.

@Rem0o
Copy link
Contributor

Rem0o commented Oct 19, 2022

Search arround Github a bit, found this file:
https://github.com/NetBSD/src/blob/8320b6cc18fa066411b30896566869b05eb29ed7/sys/dev/ic/nslm7x.c

seems to contains most register addresses you need.

@jstar81
Copy link

jstar81 commented Oct 20, 2022

Good to see progress about this, I am about to move from current AMD AM4 platform to new AM5 with Hero X670E board, (just waiting PSU to arrive), and FanControl is a must,, I have been using it since I found it, also donated to the project :)

@Rem0o
Copy link
Contributor

Rem0o commented Oct 20, 2022

@jstar81 consider donating to people contributing to this repo too, like @PhyxionNL and @Alexx999, the backend work is not to be neglected.

@jstar81
Copy link

jstar81 commented Oct 20, 2022

@Rem0o thanks for your reply, I donated $10 to @PhyxionNL :) Alex is missing Sponsor option for some reason

@Alexx999
Copy link
Contributor Author

@jstar81 Living in Ukraine has some downsides sometimes, lack of GitHub sponsorship support is one of them. Although, it's not the most pressing problem at the time being 🙂

@Rem0o Yeah, I saw the NetBSD code, it has some neat bits (i.e. 13 bit ID checks) but the sensors list used is way back from NCT6779D which is not any better than what we have here.

@Kodoz
Copy link

Kodoz commented Oct 25, 2022

Thank you! Built libs for FanControl and now it works with my X670 mobo

@jstar81
Copy link

jstar81 commented Oct 28, 2022

Thank you! Built libs for FanControl and now it works with my X670 mobo

Are you able to help noobies like me, how to build libs?
I would really like to start using FanControl & LibreHardwareMonitor with my new X670E setup :( But so far doesn't recognize any fans except my GPU. Asus own software isn't good.

@Kodoz
Copy link

Kodoz commented Oct 31, 2022

Thank you! Built libs for FanControl and now it works with my X670 mobo

Are you able to help noobies like me, how to build libs? I would really like to start using FanControl & LibreHardwareMonitor with my new X670E setup :( But so far doesn't recognize any fans except my GPU. Asus own software isn't good.

e-mail me please

@PhyxionNL
Copy link
Collaborator

You can also get the binaries here (you must be signed into GitHub):
https://github.com/LibreHardwareMonitor/LibreHardwareMonitor/actions/runs/3261148596

@PhyxionNL
Copy link
Collaborator

@Alexx999 Do you want to do anything still with this PR?

@Alexx999
Copy link
Contributor Author

@PhyxionNL I tried looking into weird temp readings, but without any info/datasheets it's non-trivial, to say the least. Also, CPUID hwmon has exactly the same readings :)
So I'd say it should be merged as-is as there's no readily available info to fix it anyway.

@PhyxionNL
Copy link
Collaborator

@PhyxionNL I tried looking into weird temp readings, but without any info/datasheets it's non-trivial, to say the least. Also, CPUID hwmon has exactly the same readings :) So I'd say it should be merged as-is as there's no readily available info to fix it anyway.

Yeah, too bad these datasheets are always missing, so this sounds good, thanks!

@PhyxionNL PhyxionNL merged commit f9cfde5 into LibreHardwareMonitor:master Nov 10, 2022
@Alexx999 Alexx999 deleted the feature/NCT6799D branch November 10, 2022 13:35
@chunkeey
Copy link

Uh, Question. I've stumbled on this while looking for linux-upstream support of my ASRock X670E PG Lighting (I know different vendor). In my case, the SuperIO identifies itself with the same 0xd802 ID, but mine has the label "NCT6796D-S" on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants