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

Fixing-ITE-IT8628E to show up to 6 fans instead of 4 #850

Merged
merged 4 commits into from
Oct 23, 2022

Conversation

sp991
Copy link
Contributor

@sp991 sp991 commented Oct 20, 2022

This is an attempt to fixe issue #849

@sp991 sp991 changed the title Fixing-ITE-IT8628E to show 5 fans Fixing-ITE-IT8628E to show 5 fans instead of 4 Oct 20, 2022
@Rem0o
Copy link
Contributor

Rem0o commented Oct 20, 2022

Does the 5th address register that gets used actually maps properly to the 5th fan channel?

@sp991
Copy link
Contributor Author

sp991 commented Oct 20, 2022

Does the 5th address register that gets used actually maps properly to the 5th fan channel?

I have no idea. All this is pretty new to me and I was just playing arround yesterday. I'm pretty knowledgeable in tech but I have no development background and doesn't know any code language except some basic VBA after fiddling in excel macro. I'm an accountant by training.

I guess I would need to compile from my fork in order to test but I have no idea how to do this and I ended up here after installing Fan Control which is a little utility that uses LHM in the background.

@Rem0o
Copy link
Contributor

Rem0o commented Oct 20, 2022

@sp991 I know, I'm FanControl developper.

Well yes you would need to compile and test your changes. You can't just throw code at the wall and hope for it to stick.

@sp991

This comment was marked as resolved.

@sp991

This comment was marked as resolved.

@sp991
Copy link
Contributor Author

sp991 commented Oct 21, 2022

After playing around in MS Visual Studio far too long, I found out that the control field in LHM only shows something when Fan Control is controlling them or maybe another 3rd party application but if they are managed by BIOS or motherboard software, the field is blank instead of showing the actual % of load.
So the 2 line of code I changed are working perfectly and I put the LHM library in the Fan Control folder and managed to control the Fan#5
image

@PhyxionNL
Copy link
Collaborator

I checked it87 driver and they state that this is a custom design that supports up to 6, so please change 5 to 6 for both and I'll merge :)

@sp991 sp991 changed the title Fixing-ITE-IT8628E to show 5 fans instead of 4 Fixing-ITE-IT8628E to show up to 6 fans instead of 4 Oct 23, 2022
@sp991
Copy link
Contributor Author

sp991 commented Oct 23, 2022

It's done, thank you so much!

@PhyxionNL PhyxionNL merged commit 53b0305 into LibreHardwareMonitor:master Oct 23, 2022
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