Hi Dmitry,

Thanks for your reply.

On 03/11/2018 02:15 AM, Dmitry Torokhov wrote:>> +static int gpio_keys_enable_wakeup(struct gpio_button_data *bdata)
> These new helpers need to be __maybe_unused in case we compile on system
> without suspend support.
> I also wanted a bit more of error handling, so I ended up with the
> version of the patch below. Can you please try it and let me know if I
> broke it.

sure, the patch you sent works :)

Tested-by: Jeffy Chen <jeffy.c...@rock-chips.com>

> Thanks.
> -- Dmitry
> Input: gpio-keys - add support for wakeup event action
> From: Jeffy Chen <jeffy.c...@rock-chips.com>
> Add support for specifying event actions to trigger wakeup when using
> the gpio-keys input device as a wakeup source.
> This would allow the device to configure when to wakeup the system. For
> example a gpio-keys input device for pen insert, may only want to wakeup
> the system when ejecting the pen.
> Suggested-by: Brian Norris <briannor...@chromium.org>
> Signed-off-by: Jeffy Chen <jeffy.c...@rock-chips.com>
> Reviewed-by: Rob Herring <r...@kernel.org>
> Signed-off-by: Dmitry Torokhov <dmitry.torok...@gmail.com>

Reply via email to