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. While at it, also add a definition for IMX_KBD_PM_OPS, which can handle the case when CONFIG_PM_SLEEP is not selected. Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx> --- drivers/input/keyboard/imx_keypad.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/input/keyboard/imx_keypad.c b/drivers/input/keyboard/imx_keypad.c index 03c8cc5..2663c91 100644 --- a/drivers/input/keyboard/imx_keypad.c +++ b/drivers/input/keyboard/imx_keypad.c @@ -580,15 +580,17 @@ err_clk: return ret; } -#endif - static SIMPLE_DEV_PM_OPS(imx_kbd_pm_ops, imx_kbd_suspend, imx_kbd_resume); +#define IMX_KBD_PM_OPS (&imx_kbd_pm_ops) +#else +#define IMX_KBD_PM_OPS NULL +#endif /* CONFIG_PM_SLEEP */ static struct platform_driver imx_keypad_driver = { .driver = { .name = "imx-keypad", .owner = THIS_MODULE, - .pm = &imx_kbd_pm_ops, + .pm = IMX_KBD_PM_OPS, .of_match_table = of_match_ptr(imx_keypad_of_match), }, .probe = imx_keypad_probe, -- 1.7.9.5 -- 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