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 HP Laptop 15-db1xxx config #835

Closed
wants to merge 40 commits into from
Closed

Add HP Laptop 15-db1xxx config #835

wants to merge 40 commits into from

Conversation

MegaMarian12350
Copy link

@MegaMarian12350 MegaMarian12350 commented Feb 26, 2020

This config was manually created and based on the "HP Notebook 15-ba010ng.xml". It may be necessary to disable the BIOS option "System Configuration: Fan Always On" in order to make the fan turn off completely.

The minimum fan speed from the based config was still using the fan, so I dissasembled my PC in order to identify when the fan was spinning or not. I also added the temperature thresholds and they're as close as the default (HP) ones.

Thanks to the Fan Speed Overrides, I managed to fix the incorrect fan speed detection. As a result, the min and max fan speeds was changed in order to reduce the complexity of the config (8 > 48 instead of 14 > 54 as the read operations). Unfortunately, there may be cases when the fan read speed is higher or lower than expected (102,5 or 97,5 instead of 100% when at full speed) or it may stop working if the temperature is low (less than 50° C).

Because I'm new on GitHub, I didn't managed to upload the config without changing the file extension from XML to TXT. Sorry about that.

Config:
HP Laptop 15-db1xxx.txt <- Change to XML

wberdowski and others added 30 commits April 28, 2019 22:22
This config file for NFC, F@tih project's product Vestel SmartBoard's motherboard (Phase 1-2) has same with a notebook motherboard, so allows to control fan speed, maybe helpful overheating.
This notebook's fan schema is same with 500S-14ISK
based on thinkpad edge E520 config
based on HP ZBook Studio 15 G3
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.
@JoEdLoquino
Copy link

Add HP Pavilion G6-1207tx Config

@hirschmann
Copy link
Owner

hirschmann commented Mar 29, 2020

Thank you very much for your config!
Unfortunately I can't merge this pull request, because you (accidentally) try to merge the master branch into the stable branch which is intended for stable releases only.

I also don't understand why you've put so many FanSpeedPercentageOverrides in the config.
Wouldn't it be possible to fix wrong readings by changing MinSpeedValueRead and MaxSpeedValueRead?

And what exactly do you mean by:

may stop working if the temperature is low (less than 50° C)

Does the fan stop, does it spin at full speed or does the firmware take over control?

@MegaMarian12350
Copy link
Author

MegaMarian12350 commented Mar 29, 2020

Unfortunately I can't merge this pull request, because you (accidentally) try to merge the master branch into the stable branch which is intended for stable releases only.

Yes, That was my fault

I also don't understand why you've put so many FanSpeedPercentageOverrides in the config.
Wouldn't it be possible to fix wrong readings by changing MinSpeedValueRead and MaxSpeedValueRead?

I made too many fan overrides because the fan reading was often incorrect, even after changing the read values.

And what exactly do you mean by:

may stop working if the temperature is low (less than 50° C)

Does the fan stop, does it spin at full speed or does the firmware take over control?

The fan keeps working as intended, only the read value ("current speed" in the program) stops working because the temps are too low, and it's hardcoded/taken control by the firmware.

@MegaMarian12350
Copy link
Author

Closed pull request due to the wrong branch

@Gleisson-dev
Copy link

Add Lenovo G480 config

@xCuri0
Copy link

xCuri0 commented Jun 5, 2023

I can confirm it works on HP Notebook 15-ac111tu too. However the reading seems a bit off

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.