This makes am35x_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/am35x.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/usb/musb/am35x.c b/drivers/usb/musb/am35x.c index 5c310c6..df32a29 100644 --- a/drivers/usb/musb/am35x.c +++ b/drivers/usb/musb/am35x.c @@ -616,12 +616,9 @@ static int am35x_resume(struct device *dev) return 0; } -static struct dev_pm_ops am35x_pm_ops = { - .suspend = am35x_suspend, - .resume = am35x_resume, -}; +static SIMPLE_DEV_PM_OPS(am35x_pm_ops, am35x_suspend, am35x_resume); -#define DEV_PM_OPS &am35x_pm_ops +#define DEV_PM_OPS (&am35x_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