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

Cosmetic update for config of Gigabyte AERO 14KV8 #697

Merged

Conversation

christiansteinert
Copy link
Contributor

Thank you for the quick merge and the kind words.

This is a small cosmetic update for the Gigabyte AERO 14KV8 configuration that you just pulled:

  • one of the RegisterWriteConfigurations unnecessarily had the write mode set to "OnWriteFanSpeed" even though it is enough to set the respective register once in order to enable the required fan mode. This is now fixed by setting the write mode to "OnInitialization" so that the register is not written all the time without need for it.
  • for one of the fans the temperature ranges were not listed in ascending order. This is not actually a problem but ascending order is nicer to read so I fixed this as well :-)

Thanks again for your great work and sorry for not catching these things right away
Christian

Thank you for the quick merge and the kind words.

This is a small cosmetic update for the Gigabyte AERO 14KV8 configuration that you just pulled:
- one of the RegisterWriteConfigurations unnecessarily had the write mode set to "OnWriteFanSpeed" even though it is enough to set the respective register once in order to enable the required fan mode. This is now fixed by setting the write mode to "OnInitialization" so that the register is not written all the time without need for it. 
- for one of the fans the temperature ranges were not listed in ascending order. This is not actually a problem but ascending order is nicer to read so I fixed this as well :-)
@hirschmann
Copy link
Owner

Thanks for the update :)

@hirschmann hirschmann merged commit 3b5c0c8 into hirschmann:master Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants