Input: gpio-keys - add system suspend support for dedicated wakeirqs
authorTony Lindgren <tony@atomide.com>
Wed, 29 Nov 2023 11:06:15 +0000 (13:06 +0200)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Thu, 30 Nov 2023 20:06:55 +0000 (12:06 -0800)
commit3717194f249227a3dfd8433bd9374cc7e0cf823d
tree08aa6898e873b2cd33a1ecaf1e09e3807951b0f3
parentd2ff98b7926e0956c0b92f7d5066cd8a65e7dfef
Input: gpio-keys - add system suspend support for dedicated wakeirqs

Some SoCs have a separate dedicated wake-up interrupt controller that can
be used to wake up the system from deeper idle states. We already support
configuring a separate interrupt for a gpio-keys button to be used with a
gpio line. However, we are lacking support system suspend for cases where
a separate interrupt needs to be used in deeper sleep modes.

Because of it's nature, gpio-keys does not know about the runtime PM state
of the button gpios, and may have several gpio buttons configured for each
gpio-keys device instance. Implementing runtime PM support for gpio-keys
does not help, and we cannot use drivers/base/power/wakeirq.c support. We
need to implement custom wakeirq support for gpio-keys.

For handling a dedicated wakeirq for system suspend, we enable and disable
it with gpio_keys_enable_wakeup() and gpio_keys_disable_wakeup() that we
already use based on device_may_wakeup().

Some systems may have a dedicated wakeirq that can also be used as the
main interrupt, this is already working for gpio-keys. Let's add some
wakeirq related comments while at it as the usage with a gpio line and
separate interrupt line may not be obvious.

Tested-by: Dhruva Gole <d-gole@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Link: https://lore.kernel.org/r/20231129110618.27551-2-tony@atomide.com
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/keyboard/gpio_keys.c
include/linux/gpio_keys.h