This makes bfin_pm_ops const. Also, checkpatch.pl complains about the use of DEV_PM_OPS: ERROR: Macros with complex values should be enclosed in parenthesis Signed-off-by: Daniel Mack <zonque@xxxxxxxxx> --- drivers/usb/musb/blackfin.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/usb/musb/blackfin.c b/drivers/usb/musb/blackfin.c index 72e2056..7db75c9 100644 --- a/drivers/usb/musb/blackfin.c +++ b/drivers/usb/musb/blackfin.c @@ -562,12 +562,9 @@ static int bfin_resume(struct device *dev) return 0; } -static struct dev_pm_ops bfin_pm_ops = { - .suspend = bfin_suspend, - .resume = bfin_resume, -}; +static SIMPLE_DEV_PM_OPS(bfin_pm_ops, bfin_suspend, bfin_resume); -#define DEV_PM_OPS &bfin_pm_ops +#define DEV_PM_OPS (&bfin_pm_ops) #else #define DEV_PM_OPS NULL #endif -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html