]> www.infradead.org Git - users/jedix/linux-maple.git/commit
pinctrl: amd: Take suspend type into consideration which pins are non-wake
authorMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Mon, 6 Jan 2025 17:41:15 +0000 (18:41 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 14 Jan 2025 13:33:59 +0000 (14:33 +0100)
commitf31f33dbb3bab572bad9fe7b849ab0dcbe6fd279
tree12b43630a2645a7b01458b39a8c43495958c92f5
parent451bc9aea9a1a6fe53969e81a5cb1bd785c0d989
pinctrl: amd: Take suspend type into consideration which pins are non-wake

Some laptops have pins which are a wake source for S0i3/S3 but which
aren't a wake source for S4/S5 and which cause issues when left unmasked
during hibernation (S4).

For example HP EliteBook 855 G7 has pin #24 that causes instant wakeup
(hibernation failure) if left unmasked (it is a wake source only for
S0i3/S3).
GPIO pin #24 on this platform is likely dedicated to WWAN XMM7360
modem since this pin triggers wake notify to WWAN modem's parent PCIe
port.

Fix this by considering a pin a wake source only if it is marked as one
for the current suspend type (S0i3/S3 vs S4/S5).

Since Z-wake pins only make sense at runtime these were excluded from
both of suspend categories, so pins with only the Z-wake flag set are
effectively treated as non-wake pins.

Fixes: 2fff0b5e1a6b ("pinctrl: amd: Mask non-wake source pins with interrupt enabled at suspend")
Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
Link: https://lore.kernel.org/d4b2d076366fdd08a0c1cd9b7ecd91dc95e07269.1736184752.git.mail@maciej.szmigiero.name
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-amd.c
drivers/pinctrl/pinctrl-amd.h