Skip to content

fix: enable acpi.ec_no_wakeup in Thinkpad T14 AMD Gen 6#1827

Open
codingCoffee wants to merge 2 commits intoNixOS:masterfrom
codingCoffee:thinkpad-t14-gen6
Open

fix: enable acpi.ec_no_wakeup in Thinkpad T14 AMD Gen 6#1827
codingCoffee wants to merge 2 commits intoNixOS:masterfrom
codingCoffee:thinkpad-t14-gen6

Conversation

@codingCoffee
Copy link
Copy Markdown
Contributor

Description of changes

Add acpi.ec_no_wakeup=1 kernel param to stop the embedded controller from waking the system during s2idle. Fixes wake-on-plug for USB-C charger and peripheral hot-plug on ThinkPad T14 Gen 6 AMD.

This is the same fix required in Gen 5 of the Thinkpad T14 series.

Things done
  • Tested the changes in your own NixOS Configuration
  • Tested the changes end-to-end by using your fork of nixos-hardware and
    importing it via <nixos-hardware> or Flake input

Signed-off-by: Ameya Shenoy <shenoy.ameya@gmail.com>
Signed-off-by: Ameya Shenoy <shenoy.ameya@gmail.com>
@codingCoffee
Copy link
Copy Markdown
Contributor Author

@Mic92 can you check out this PR, I've tested it and it works on my Thinkpad T14 Gen 6. It's very similar to this PR

@m00wl
Copy link
Copy Markdown
Member

m00wl commented Apr 23, 2026

In the table in the README, you link to the old gen5 directory (lenovo/thinkpad/t14/amd/gen5).
In addition, there is also #1748. They seem to not need acpi.ec_no_wakeup=1 anymore.
I have access to a Gen6, can you tell me how to confirm that the parameter is needed?

@codingCoffee
Copy link
Copy Markdown
Contributor Author

codingCoffee commented Apr 23, 2026

I too have a Gen 6, let me check and confirm as well but in case you want to check in the meantime here are the steps

  • suspend your laptop
  • plug in either a charging cable in a usb-c port or plugin in a device (mobile etc) into a usb c port
  • the laptop should wakeup from suspend withot the kernel param

Maybe linux kernel 7.0.0 already solves this without the kernel param. I'll have to check

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.

2 participants