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 support for ASUS ROG CROSSHAIR X670E GENE #1092

Merged
merged 4 commits into from
Jun 13, 2023

Conversation

Crosenhain
Copy link
Contributor

Adding basic support for the ASUS ROG CROSSHAIR X670E GENE, so LibreHardwareMonitorLib can read the T SENSOR

@PhyxionNL PhyxionNL merged commit 9930c5f into LibreHardwareMonitor:master Jun 13, 2023
@threeohclock
Copy link

I believe this pull broke the library for this motherboard. When I use FanControl I am now getting this:

6/16/2023 8:50:38 AM: LibreHardwareMonitor.Hardware.Motherboard.Lpc.EC.EmbeddedController+MultipleBoardRecordsFoundException: Multiple board records refer to the same model 'ROG_CROSSHAIR_X670E_EXTREME'
at LibreHardwareMonitor.Hardware.Motherboard.Lpc.EC.EmbeddedController.Create(Model model, ISettings settings) in C:\Users\Remi\source\repos\FanControl\LibreHardwareMonitor\LibreHardwareMonitorLib\Hardware\Motherboard\Lpc\EC\EmbeddedController.cs:line 337
at LibreHardwareMonitor.Hardware.Motherboard.Motherboard..ctor(SMBios smBios, ISettings settings) in C:\Users\Remi\source\repos\FanControl\LibreHardwareMonitor\LibreHardwareMonitorLib\Hardware\Motherboard\Motherboard.cs:line 71
at LibreHardwareMonitor.Hardware.Motherboard.MotherboardGroup..ctor(SMBios smbios, ISettings settings) in C:\Users\Remi\source\repos\FanControl\LibreHardwareMonitor\LibreHardwareMonitorLib\Hardware\Motherboard\MotherboardGroup.cs:line 18
at LibreHardwareMonitor.Hardware.Computer.AddGroups() in C:\Users\Remi\source\repos\FanControl\LibreHardwareMonitor\LibreHardwareMonitorLib\Hardware\Computer.cs:line 501
at LibreHardwareMonitor.Hardware.Computer.Open() in C:\Users\Remi\source\repos\FanControl\LibreHardwareMonitor\LibreHardwareMonitorLib\Hardware\Computer.cs:line 495
at FanControl.Domain.BackendProviders.LHM.LHMBackendProvider.Open()

@PhyxionNL
Copy link
Collaborator

You're right @threeohclock, fixed:
7e841d7

@Crosenhain
Copy link
Contributor Author

Apologies, thanks for the fix

@threeohclock
Copy link

Thank you so much! Really appreciate the immediate fix.

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

3 participants