[Public] > > > These events only occur when the EC GPE is inadvertently set as a wakeup > > > source. Originally the EC GPE was only set as a wakeup source when using > > > the intel-vbtn or intel-hid drivers in commit 10a08fd65ec1 ("ACPI: PM: > > > Set up EC GPE for system wakeup from drivers that need it") but during > > > testing a reporter discovered that this was not enough for their ASUS > > > Zenbook UX430UNR/i7-8550U to wakeup by lid event or keypress. > > > Marking the EC GPE for wakeup universally resolved this for that > > > reporter in commit b90ff3554aa3 ("ACPI: PM: s2idle: Always set up EC GPE > > > for system wakeup"). > > > > > > However this behavior has lead to a number of problems: > > > > > > * On both Lenovo T14 and P14s the keyboard wakeup doesn't work, and > > > sometimes the power button event doesn't work. > > > * On HP 635 G7 detaching or attaching AC during suspend will cause > > > the system not to wakeup > > > * On Asus vivobook to prevent detaching AC causing resume problems > > > * On Lenovo 14ARE05 to prevent detaching AC causing resume problems > > > * On HP ENVY x360 to prevent detaching AC causing resume problems > > > > > > As there may be other Intel systems besides ASUS Zenbook UX430UNR/i7- > 8550U > > > that don't use intel-vbtn or intel-hid avoid these problems by only > > > universally marking the EC GPE wakesource on non-AMD systems. > > > > > > Link: > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchw > ork.kernel.org%2Fproject%2Flinux- > pm%2Fcover%2F5997740.FPbUVk04hV%40kreacher%2F%2322825489&dat > a=04%7C01%7Cmario.limonciello%40amd.com%7Cc2ced5bd6bbb4e62e9ac08d > 948525197%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C63762034 > 0670859712%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi > V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=oUsKS5mjAa > rc%2FhnnY%2FILZWWzUbvdBQHlH1MAcusHSIw%3D&reserved=0 > > > Link: > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.fr > eedesktop.org%2Fdrm%2Famd%2F- > %2Fissues%2F1230&data=04%7C01%7Cmario.limonciello%40amd.com%7 > Cc2ced5bd6bbb4e62e9ac08d948525197%7C3dd8961fe4884e608e11a82d994e1 > 83d%7C0%7C0%7C637620340670859712%7CUnknown%7CTWFpbGZsb3d8eyJ > WIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C > 1000&sdata=KKUiOSeAQEkFjN9X8y8k3sC3J3s48faaNLzklPO12as%3D& > reserved=0 > > > Link: > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.fr > eedesktop.org%2Fdrm%2Famd%2F- > %2Fissues%2F1629&data=04%7C01%7Cmario.limonciello%40amd.com%7 > Cc2ced5bd6bbb4e62e9ac08d948525197%7C3dd8961fe4884e608e11a82d994e1 > 83d%7C0%7C0%7C637620340670859712%7CUnknown%7CTWFpbGZsb3d8eyJ > WIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C > 1000&sdata=kDGTIkQMh%2FGG4OhssvukJ7xJ7Ld6j6bl1TXRvpS58%2Fk%3D > &reserved=0 > > > Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx> > > > Acked-by: Alex Deucher <alexander.deucher@xxxxxxx> > > > > As this seems to fix quite some issues for the AMD systems, is there any > > reason why this is not tagged as fix for stable? Are there any plans for > > backporting? > There's a number of other patches in other subsystems besides this one that are needed for successful S0i3 on AMD systems that land in 5.14 but are not stable candidates at this time. Perhaps after 5.14 is out and has been well tested it will make sense to send the whole remaining series back to stable. > If you need it in stable kernels, please send a request to > stable@xxxxxxxxxxxxxxx to include it.