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
Update EmbeddedController.cs #740
Conversation
Correction of the motherboard model (CROSSHAIR III FORMULA > ROG CROSSHAIR VIII FORMULA). It has the same sensors so I put it with the CROSSHAIR VIII HERO
@@ -158,7 +158,7 @@ public BoardInfo(Model model, BoardFamily family, params ECSensor[] sensors) | |||
ECSensor.TempChipset, ECSensor.TempCPU, ECSensor.TempMB, | |||
ECSensor.TempVrm, ECSensor.FanChipset, ECSensor.CurrCPU, ECSensor.VoltageCPU | |||
), | |||
new(new Model[] {Model.ROG_CROSSHAIR_VIII_HERO, Model.ROG_CROSSHAIR_VIII_HERO_WIFI }, BoardFamily.Amd500, | |||
new(new Model[] {Model.ROG_CROSSHAIR_VIII_HERO, Model.ROG_CROSSHAIR_VIII_HERO_WIFI, Model.ROG_CROSSHAIR_VIII_FORMULA }, BoardFamily.Amd500, |
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.
But Formula, unlike Hero, does not have water sensors, does it?
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.
If this is the correct model, it seems it does have water headers and connectors:
https://rog.asus.com/motherboards/rog-crosshair/rog-crosshair-viii-formula-model/spec
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.
Ah, OK. Thanks for checking this for me!
The ROG CROSSHAIR VIII FORMULA board has the same sensors as the CROSSHAIR VIII HERO [1] thus let's join their definitions which adds missing sensors for Formula. [1] LibreHardwareMonitor/LibreHardwareMonitor#740 Signed-off-by: Eugene Shalygin <eugene.shalygin@gmail.com>
The ROG CROSSHAIR VIII FORMULA board has the same sensors as the CROSSHAIR VIII HERO [1] thus let's join their definitions which adds missing sensors for Formula. [1] LibreHardwareMonitor/LibreHardwareMonitor#740 Signed-off-by: Eugene Shalygin <eugene.shalygin@gmail.com>
The ROG CROSSHAIR VIII FORMULA board has the same sensors as the CROSSHAIR VIII HERO [1] thus let's join their definitions which adds missing sensors for Formula. [1] LibreHardwareMonitor/LibreHardwareMonitor#740 Signed-off-by: Eugene Shalygin <eugene.shalygin@gmail.com> Link: https://lore.kernel.org/r/20220517080508.1910953-1-eugene.shalygin@gmail.com Signed-off-by: Guenter Roeck <linux@roeck-us.net>
The ROG CROSSHAIR VIII FORMULA board has the same sensors as the CROSSHAIR VIII HERO [1] thus let's join their definitions which adds missing sensors for Formula. [1] LibreHardwareMonitor/LibreHardwareMonitor#740 Signed-off-by: Eugene Shalygin <eugene.shalygin@gmail.com> Link: https://lore.kernel.org/r/20220517080508.1910953-1-eugene.shalygin@gmail.com Signed-off-by: Guenter Roeck <linux@roeck-us.net>
The ROG CROSSHAIR VIII FORMULA board has the same sensors as the CROSSHAIR VIII HERO [1] thus let's join their definitions which adds missing sensors for Formula. [1] LibreHardwareMonitor/LibreHardwareMonitor#740 Signed-off-by: Eugene Shalygin <eugene.shalygin@gmail.com> Link: https://lore.kernel.org/r/20220517080508.1910953-1-eugene.shalygin@gmail.com Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Correction of the motherboard model (CROSSHAIR III FORMULA > ROG CROSSHAIR VIII FORMULA). It has the same sensors so I put it with the CROSSHAIR VIII HERO.