Fabio Estevam <festevam@xxxxxxxxx> wrote: >Hi Dmitry, > >On Thu, Apr 11, 2013 at 2:28 AM, Dmitry Torokhov ><dmitry.torokhov@xxxxxxxxx> wrote: >> Fabio Estevam <festevam@xxxxxxxxx> wrote: >> >>>From: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx> >>> >>>SIMPLE_DEV_PM_OPS should be inside the CONFIG_PM_SLEEP 'if' block >>>because >>>imx_kbd_suspend and imx_kbd_resume definitions are inside this block. >> >> No, it does not. It compiles just fine if CONFIG_PM_SLEEP is not >defined. > >I am not sure how to disable CONFIG_PM_SLEEP properly in the >defconfig, but I did this quick test to simulate it: > >--- a/drivers/input/keyboard/imx_keypad.c >+++ b/drivers/input/keyboard/imx_keypad.c >@@ -536,7 +536,7 @@ static int imx_keypad_probe(struct platform_device >*pdev) > return 0; > } > >-#ifdef CONFIG_PM_SLEEP >+#if 0 This is not the same thing as SIMPLE_DEV_PM_OPS change their definition depending on CONFIG_PM_SLEEP. Thanks. -- Dmitry -- 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
