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

Allow fan control via character device (e.g. "echo 1 > /dev/acer-max-fan-0") #95

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

TheFeshy
Copy link

Creates a fan control device for any of the listed laptops with a fan quirk. This device can accept ascii or binary input (e.g. can be written to with echo, or via a program like python writing 0x00 and 0x01.)

echo 1 > /dev/acer-max-fan-0 to turn on, echo 0 > /dev/acer-max-fan-0 to turn off again.

I split the capabilities so that we can distinguish between laptops that have a turbo button, and laptops that don't but still have the fans controlled via the EC. I put my own laptop (the PH517-61) in this category, and also the PH517-51 (as it's the same laptop but with Intel/Nvidia.)

@JafarAkhondali
Copy link
Owner

JafarAkhondali commented May 12, 2023

Thanks @TheFeshy
Can you please add sample usage to readme?
Can someone test this functionality on your device please?

@hornc
Copy link
Contributor

hornc commented May 21, 2023

@JafarAkhondali I'll try to test this in the next few days.

@hornc
Copy link
Contributor

hornc commented May 25, 2023

Ok, I have checked out the PR and re-installed on my AN515-57. It took a reboot to have the device show up under

/dev/acer-fan-max-0

After a bit of messing around I realized the test commands above are wrong, it has max-fan instead of the correct device fan-max

it should be:

echo 1 > /dev/acer-fan-max-0 to turn on, echo 0 > /dev/acer-fan-max-0 to turn off again.

I was able to run both commands without sudo, but didn't notice any obvious change in fan speed or behavior however.

When I run sensors I don't see any fan info normally. I have not set up any other fan monitoring sw.

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

4 participants