On Thu, Oct 11, 2012 at 7:45 PM, Lee Jones <lee.jo...@linaro.org> wrote: > From: Jonas Aaberg <jonas.ab...@stericsson.com> Some change logs would have helped.
> > Cc: Dmitry Torokhov <dmitry.torok...@gmail.com> > Cc: linux-in...@vger.kernel.org > Acked-by: Lee Jones <lee.jo...@linaro.org> > Signed-off-by: Jonas Aaberg <jonas.ab...@stericsson.com> > Signed-off-by: Philippe Langlais <philippe.langl...@linaro.org> > Reviewed-by: Bengt Jonsson <bengt.g.jons...@stericsson.com> > --- > 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-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/