Skip to content

Commit d2c87a8

Browse files
committed
replace tlp with power-profiles-daemon
TLP is often no longer the best choice since firmware provides power profiles instead. That's why we are removing TLP as the default option. We may want to bring back TLP to some old hardware if it makes sense i.e. certain thinkpad modules.
1 parent f49bb3b commit d2c87a8

File tree

10 files changed

+5
-44
lines changed

10 files changed

+5
-44
lines changed

CONTRIBUTING.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,9 @@ For example:
1414

1515
```nix
1616
{ lib }: {
17-
# Using mkDefault, because the user might want to disable tlp
18-
services.tlp.enable = lib.mkDefault true;
17+
# Using mkDefault, because the user might want to disable power-profiles-daemon
18+
services.power-profiles-daemon.enable = lib.mkDefault true;
19+
1920
# No need to use mkDefault, because the setting will merge with the user's setting
2021
boot.kernelModules = [ "tmp_smapi" ];
2122
}

asus/fa507nv/default.nix

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,6 @@
2525
lib.optionals (lib.versionOlder config.boot.kernelPackages.kernel.version "6.8")
2626
[ "rtc_cmos.use_acpi_alarm=1" ];
2727

28-
# AMD has better battery life with PPD over TLP:
29-
# https://community.frame.work/t/responded-amd-7040-sleep-states/38101/13
30-
services.power-profiles-daemon.enable = lib.mkDefault true;
31-
3228
# Adds the missing asus functionality to Linux.
3329
# https://asus-linux.org/manual/asusctl-manual/
3430
services = {

common/pc/laptop/default.nix

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,5 @@
1-
{ config, lib, ... }:
2-
3-
{
1+
{ lib, config, ... }: {
42
imports = [ ../. ];
53

6-
# Gnome 40 introduced a new way of managing power, without tlp.
7-
# However, these 2 services clash when enabled simultaneously.
8-
# https://github.com/NixOS/nixos-hardware/issues/260
9-
services.tlp.enable = lib.mkDefault (
10-
(lib.versionOlder (lib.versions.majorMinor lib.version) "21.05")
11-
|| !config.services.power-profiles-daemon.enable
12-
);
4+
config.services.power-profiles-daemon.enable = lib.mkDefault (!config.services.tlp.enable);
135
}

dell/inspiron/14-5420/README.md

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,3 @@
88
`fwupd` is used to receive firmware updates from vendors. This device is in its [supported devices list](https://fwupd.org/lvfs/devices/)
99

1010
`thermald` is used to achieve better thermal behaviour
11-
12-
##### TLP
13-
TLP is a battery power optimizer for laptops. It is enabled by default, however, since it conflicts with power-profiles-daemon which is GNOME's power management system, TLP will not be installed with if power-profiles-daemon or a recent version of GNOME is present.

framework/13-inch/common/amd.nix

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,4 @@
1919
++ lib.optionals (lib.versionOlder config.boot.kernelPackages.kernel.version "6.8") [
2020
"rtc_cmos.use_acpi_alarm=1"
2121
];
22-
23-
# AMD has better battery life with PPD over TLP:
24-
# https://community.frame.work/t/responded-amd-7040-sleep-states/38101/13
25-
services.power-profiles-daemon.enable = lib.mkDefault true;
2622
}

framework/16-inch/common/amd.nix

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,4 @@
1919
++ lib.optionals (lib.versionOlder config.boot.kernelPackages.kernel.version "6.8") [
2020
"rtc_cmos.use_acpi_alarm=1"
2121
];
22-
23-
# AMD has better battery life with PPD over TLP:
24-
# https://community.frame.work/t/responded-amd-7040-sleep-states/38101/13
25-
services.power-profiles-daemon.enable = lib.mkDefault true;
2622
}

gpd/win-2/default.nix

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,6 @@
1010
"video=eDP-1:panel_orientation=right_side_up"
1111
];
1212

13-
services.tlp.enable = lib.mkDefault (
14-
(lib.versionOlder (lib.versions.majorMinor lib.version) "21.05")
15-
|| !config.services.power-profiles-daemon.enable
16-
);
17-
1813
# Required for grub to properly display the boot menu.
1914
boot.loader.grub.gfxmodeEfi = lib.mkDefault "720x1280x32";
2015

lenovo/ideapad/15arh05/default.nix

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,4 @@
1717

1818
# Cooling management
1919
services.thermald.enable = lib.mkDefault true;
20-
21-
# tlp defaults to "powersave", which doesn't exist on this laptop
22-
services.tlp.settings = {
23-
CPU_SCALING_GOVERNOR_ON_BAT = "schedutil";
24-
};
2520
}

lenovo/legion/16aph8/default.nix

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,4 @@
3232

3333
# Avoid issues with modesetting causing blank screen
3434
services.xserver.videoDrivers = [ "nvidia" ];
35-
36-
# AMD has better battery life with PPD over TLP:
37-
# https://community.frame.work/t/responded-amd-7040-sleep-states/38101/13
38-
services.power-profiles-daemon.enable = lib.mkDefault true;
3935
}

microsoft/surface/common/default.nix

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -110,9 +110,6 @@ in
110110
kernelParams = mkDefault [ "mem_sleep_default=deep" ];
111111
};
112112

113-
# NOTE: Check the README before enabling TLP:
114-
services.tlp.enable = mkDefault false;
115-
116113
# Needed for wifi firmware, see https://github.com/NixOS/nixos-hardware/issues/364
117114
hardware = {
118115
enableRedistributableFirmware = mkDefault true;

0 commit comments

Comments
 (0)