Skip to content

Fix 'HP ProBook 650 G2' not reaching 0%#775

Merged
hirschmann merged 1 commit intohirschmann:masterfrom
rico-chet:fix-hp-probook-650-g2-zero-percent-fan-speed-failure
Oct 29, 2019
Merged

Fix 'HP ProBook 650 G2' not reaching 0%#775
hirschmann merged 1 commit intohirschmann:masterfrom
rico-chet:fix-hp-probook-650-g2-zero-percent-fan-speed-failure

Conversation

@rico-chet
Copy link
Contributor

When simply mapping fan register value of 255 to 0%, the fan is not able
to reach 0% directly when starting at a rather high speed, running
constantly at around 25% instead. This can be worked around manually by
first setting the speed to a value between 0% and 20%, then setting it
to 0%.

Mapping the fan register value of 254 to 0% fixes the issue permanently.
Mapping read value of 255 to 0% fixes the percentage indicator which
shows negative speed values otherwise.

When simply mapping fan register value of 255 to 0%, the fan is not able
to reach 0% directly when starting at a rather high speed, running
constantly at around 25% instead.  This can be worked around manually by
first setting the speed to a value between 0% and 20%, then setting it
to 0%.

Mapping the fan register value of 254 to 0% fixes the issue permanently.
Mapping read value of 255 to 0% fixes the percentage indicator which
shows negative speed values otherwise.
@hirschmann
Copy link
Owner

Thanks a lot for the fix! Looks good to me.

@hirschmann hirschmann merged commit 537e9b2 into hirschmann:master Oct 29, 2019
vdurante pushed a commit to vdurante/nbfc that referenced this pull request Feb 29, 2024
When simply mapping fan register value of 255 to 0%, the fan is not able
to reach 0% directly when starting at a rather high speed, running
constantly at around 25% instead.  This can be worked around manually by
first setting the speed to a value between 0% and 20%, then setting it
to 0%.

Mapping the fan register value of 254 to 0% fixes the issue permanently.
Mapping read value of 255 to 0% fixes the percentage indicator which
shows negative speed values otherwise.
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