On Thu, Oct 11, 2012 at 7:45 PM, Lee Jones <lee.jones@xxxxxxxxxx> wrote: > From: Jonas Aaberg <jonas.aberg@xxxxxxxxxxxxxx> Some change logs would have helped. > > Cc: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> > Cc: linux-input@xxxxxxxxxxxxxxx > Acked-by: Lee Jones <lee.jones@xxxxxxxxxx> > Signed-off-by: Jonas Aaberg <jonas.aberg@xxxxxxxxxxxxxx> > Signed-off-by: Philippe Langlais <philippe.langlais@xxxxxxxxxx> > Reviewed-by: Bengt Jonsson <bengt.g.jonsson@xxxxxxxxxxxxxx> > --- > drivers/input/keyboard/gpio_keys.c | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) > > diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c > index 7947315..78de557 100644 > --- a/drivers/input/keyboard/gpio_keys.c > +++ b/drivers/input/keyboard/gpio_keys.c > @@ -29,6 +29,7 @@ > #include <linux/of_platform.h> > #include <linux/of_gpio.h> > #include <linux/spinlock.h> > +#include <linux/pm_runtime.h> > > struct gpio_button_data { > const struct gpio_keys_button *button; > @@ -526,6 +527,7 @@ static int gpio_keys_open(struct input_dev *input) > { > struct gpio_keys_drvdata *ddata = input_get_drvdata(input); > > + pm_runtime_get_sync(input->dev.parent); I am not an expert of the runtime. However would be grateful if you explain me what it actually do. Also I did not see any runtime suspend/ resume handlers populated. -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html