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

Support for MSI Z690 Unify / NCT6687D (missing T_Sen_x issue related) (dumps and required changes included) #1306

Open
einsnulleins opened this issue Apr 5, 2024 · 0 comments

Comments

@einsnulleins
Copy link

einsnulleins commented Apr 5, 2024

Hi,
I have no clue about pull requests and all that jazz, so here is all the required info to add proper thermal sensor support for this specific mainboard.

Issue: Some wrong names, wrong values and missing values for the thermal sensors on this board.
(Also voltages)
HWiNFO and AIDA64 properly report all values; Output from AIDA64 attached:
dump.txt
LHWM report with full hex data of controller:
report.txt

For this mainboard we need an array of 10 instead of 7 sensors with the following IDs:
CPU, System, VRM MOS, PCH, CPU Socket, T_Sen_1, T_Sen_2, PCIE_1, M.2, PCIE_2
(line 272 in Nct677X.cs)

corrected addresses for those values:
(lines ~288 in Nct677X.cs)
_temperaturesSource = new TemperatureSourceData[] {
new(null, 0x100),
new(null, 0x102),
new(null, 0x104),
new(null, 0x106),
new(null, 0x108),
new(null, 0x10A),
new(null, 0x10C),
new(null, 0x10E),
new(null, 0x110),
new(null, 0x112)

};

A friend of mine has compiled a version with these values as default for the controller and I am getting correct readouts via LHWMlib.

I suspect that the several open issues about missing T_Sen_x thermal sensors are related to the thermal sensor arrays being off for some mainboards.

I hope somebody will include this in one of their next pull requests.
Thank you

(I was struggling to understand the hex conversions at first but 0,1°C are equal to 256 in decimal. So a hex of "43 00" becomes 16640 in decimal and 60°C after full conversion.)

@einsnulleins einsnulleins changed the title Support for MSI Z690 Unify / NCT6687D (missing T_Sen_x issue related) (dumps included) Support for MSI Z690 Unify / NCT6687D (missing T_Sen_x issue related) (dumps and required changes included) Apr 5, 2024
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

No branches or pull requests

1 participant