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

Sync fans based on temp instead of duty #390

Open
crawfxrd opened this issue Jul 29, 2023 · 2 comments
Open

Sync fans based on temp instead of duty #390

crawfxrd opened this issue Jul 29, 2023 · 2 comments

Comments

@crawfxrd
Copy link
Member

The current implementation syncs all fans to the highest duty.

Replace this with syncing to the highest temperature reported by the sensors.

This will allow independent curves for FAN0 and FAN1 that work as expected based on their actual points.

@qnxdev
Copy link

qnxdev commented Oct 16, 2023

I highly support this, my darp9 is super hot sometimes, I have to turn on fan manually with Fn + 1 but why can't it do that job?

When I use fan it cools down the laptop such that I can put on my lap & work.

@leviport
Copy link
Member

@qnxdev this is unrelated to what you're describing. The darp9 only has a single fan, and this issue is about fan speeds on models with dual fans.

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

No branches or pull requests

3 participants