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

main/power-profiles-daemon: fix tlp conflict #3326

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

subnut
Copy link
Contributor

@subnut subnut commented Dec 22, 2024

Description

tlp is also used independently to set battery charging limits on supported hardware.
That functionality does not clash with power-profiles-daemon.

Checklist

tlp is also used independently to set battery charging limits on
supported hardware. That functionality does not clash with
power-profiles-daemon.
@q66
Copy link
Member

q66 commented Jan 4, 2025

i'm still not quite sure if the conflict is a good thing at all (because it leads to stuff like this), as far as i can tell there is nothing really preventing you from having both installed? unless they dbus-activate or something? (i guess power-profiles-daemon does dbus-activate, tlp does not)

@valpackett
Copy link
Contributor

valpackett commented Jan 5, 2025

There's a patch around to make ppd use tlp when it's enabled: Rongronggg9/power-profiles-daemon@e5536af

@subnut
Copy link
Contributor Author

subnut commented Jan 6, 2025

i'm still not quite sure if the conflict is a good thing at all

yeah that's true... I guess we could remove the conflict completely, and let the user determine what they want to use.

I prefer this.

@q66 lmk if I should update this PR to make this change (or you could commit it directly, in which case this PR can be closed)


[...] make ppd use tlp [...]

oh god.

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.

3 participants