You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.)
The text was updated successfully, but these errors were encountered:
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
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.)
The text was updated successfully, but these errors were encountered: