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 the Gigabyte B360 AORUS GAMING 3 WIFI-CF mainboard. #622

Conversation

cocus
Copy link
Contributor

@cocus cocus commented Jan 6, 2022

This mainboard uses the IT8686E controller, and it only needed to be
added to the list reusing most of the values from X570_AORUS_MASTER.

@cocus
Copy link
Contributor Author

cocus commented Jan 6, 2022

Before:
image
After:
image

And I must add that these values correspond to the ones shown on the Gigabyte/Aorus Hardware Monitor

@cocus
Copy link
Contributor Author

cocus commented Jan 7, 2022

Well, maybe the 3VSB and the VBat are not right, but the other numbers do agree with the Gigabyte app

@PhyxionNL
Copy link
Collaborator

Well, maybe the 3VSB and the VBat are not right, but the other numbers do agree with the Gigabyte app

Do you want to fix this?

@cocus cocus force-pushed the cocus/add-b360-aorus-gaming-3-wifi-cf-mainboard branch from 80223f0 to bd7176d Compare January 31, 2022 20:14
@cocus
Copy link
Contributor Author

cocus commented Jan 31, 2022

Ok, I've fixed the issue. The scaling was wrong, using a 1:1 worked (comparing against AIDA64 since the Gigabyte app doesn't show Vbat nor 3VSB).

@cocus
Copy link
Contributor Author

cocus commented Jan 31, 2022

Oh, seems like Git pushed the newlines of the SuperIO file wrong, and modified the entire file! Damn it. One moment.
EDIT: Ok, fixed!

This mainboard uses the IT8686E controller, and it only needed to be
added to the list reusing most of the values from X570_AORUS_MASTER.
@cocus cocus force-pushed the cocus/add-b360-aorus-gaming-3-wifi-cf-mainboard branch from bd7176d to 292f76f Compare January 31, 2022 20:18
@PhyxionNL PhyxionNL merged commit cb6729f into LibreHardwareMonitor:master Jan 31, 2022
@PhyxionNL
Copy link
Collaborator

Thanks @cocus

@mirh
Copy link

mirh commented Apr 20, 2024

Intel300SeriesVoltageInputPlacementManagement.InitVoltageInput4IT8686 inside of Gigabyte.Engine.EnvironmentControl.dll has this names:

Index (0-6)
CPU Vcore
+3.3V
+12V
+5V
CPU VAXG
CPU VCCSA
DRAM CH(A/B)

Did they just eventually renamed the labels in a more recent version, or am I missing something?
Also.. I found some interesting "values that multiply others" at IntelBaseVoltageInputPlacementManagement.ToVoltageGain4FirstSuperIO (or VoltageDetector.AdjustVoltageHardwareMonitorDatas).
Could they be the reference resistance? They don't look like the numbers used here.

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