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 config for HP Laptop 17-by1xxx.xml #691

Merged
merged 1 commit into from
May 27, 2019
Merged

Add config for HP Laptop 17-by1xxx.xml #691

merged 1 commit into from
May 27, 2019

Conversation

Zwergesel
Copy link
Contributor

This config was manually created by trial and error based on the "HP Laptop 14-cm0xxx.xml" and tested on an HP 17-by1102ng.

Note: It may be necessary to disable the BIOS option "System Configuration" > "Fan Always On" in order to make the fan turn off completely.

Quick overview:

  • Writing to register 0xDB controls the fan speed, but the scale does not seem to be exactly linear. Writing values above 90 may crash the PC. The range 45 to 80 mapped roughly to the 10% - 100% speed with a special value of 30 to turn off the fan entirely (although any value < 35 will do that as far as I could tell).
  • Reading from register 0x71 returns the current fan speed as values between 8 and 17 if the fan is spinning, so configuring 7 to 17 maps the fan speed nicely between 10% and 100%. Once again a special value of 0 is used if the fan is off.

Without NBFC the fans will only stop spinning at roughly < 35°C and otherwise be pretty much constantly spinning at 10%-20% which is fairly annoying. This config sets the temperature for the fan starting to spin at 55°C and ramps it up until it reaches 100% at 78°C. The NBFC config from which this was copied allowed even higher temperatures before starting the fan, but I wanted to keep it somewhat conservative. Maximum temperature of the CPU is listed as 100°C. (source)

* Tested on an HP 17-by1102ng
@hirschmann
Copy link
Owner

Thank you very much for the config and your great description of how you created it!
Looks really good 👍

@hirschmann hirschmann merged commit cb0105e into hirschmann:master May 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants