Re: [PATCH] Input: mark serio and i8042 as suspended when hibernating too.
From: Rafael J. Wysocki <hidden>
Date: 2009-07-06 22:40:17
Also in:
lkml
On Sunday 05 July 2009, Thadeu Lima de Souza Cascardo wrote:
serio ports are not being restarted any longer because resume operations after hibernate do nothing, since the device has not been marked as suspended. This happens because suspend is only considering the SUSPEND event but not the FREEZE event. Note that this driver has still to migrate to dev_pm_ops, but this fixes this particular bug now.
Hmm, this looks like a fix for: http://bugzilla.kernel.org/show_bug.cgi?id=13643 Any objections to merging it through the suspend tree? Best, Rafael
quoted hunk ↗ jump to hunk
Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@holoscopio.com> --- drivers/input/serio/i8042.c | 7 ++++--- drivers/input/serio/serio.c | 7 ++++--- 2 files changed, 8 insertions(+), 6 deletions(-)diff --git a/drivers/input/serio/i8042.c b/drivers/input/serio/i8042.c index f919bf5..582245c 100644 --- a/drivers/input/serio/i8042.c +++ b/drivers/input/serio/i8042.c@@ -934,10 +934,11 @@ static bool i8042_suspended; static int i8042_suspend(struct platform_device *dev, pm_message_t state) { - if (!i8042_suspended && state.event == PM_EVENT_SUSPEND) { + if (!i8042_suspended && state.event == PM_EVENT_SUSPEND) i8042_controller_reset(); - i8042_suspended = true; - } + + i8042_suspended = state.event == PM_EVENT_SUSPEND || + state.event == PM_EVENT_FREEZE; return 0; }diff --git a/drivers/input/serio/serio.c b/drivers/input/serio/serio.c index fb17573..d66f494 100644 --- a/drivers/input/serio/serio.c +++ b/drivers/input/serio/serio.c@@ -935,10 +935,11 @@ static int serio_suspend(struct device *dev, pm_message_t state) { struct serio *serio = to_serio_port(dev); - if (!serio->suspended && state.event == PM_EVENT_SUSPEND) { + if (!serio->suspended && state.event == PM_EVENT_SUSPEND) serio_cleanup(serio); - serio->suspended = true; - } + + serio->suspended = state.event == PM_EVENT_SUSPEND || + state.event == PM_EVENT_FREEZE; return 0; }