-
-
Notifications
You must be signed in to change notification settings - Fork 565
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 NZXT kraken X (X42, X52, X62 or X72) Support #1119
Conversation
Can this be integrated into KrakenV3? Just glancing over it, and it seems a lot of code is quite similar. |
It could, but the sensor reading is quite different and I thought that would add many conditions and obfuscate the code. And also I did not want to break anything as I cannot test on V3 hardware. |
stream.Read(_rawData); | ||
|
||
// if not 0x04, it is not temperature data | ||
if (_rawData[0] != 0x04) return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you reformat this document and follow code style (e.g. return on new line).
@@ -411,6 +411,8 @@ public static Model GetModel(string name) | |||
return Model.B660M_DS3H_AX_DDR4; | |||
case var _ when name.Equals("ROG STRIX Z790-I GAMING WIFI", StringComparison.OrdinalIgnoreCase): | |||
return Model.ROG_STRIX_Z790_I_GAMING_WIFI; | |||
case var _ when name.Equals("MPG X570 GAMING PLUS (MS-7C37)",StringComparison.OrdinalIgnoreCase): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move this added motherboard to a separate PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oups, sorry about that it was not supposed to be here
sorry did PR on wrong branch, done a new one : #1143 |
Support tested on Kraken X62 Firmware 6.2 should fix #1095