On 30.09.2013 18:25, Felipe Balbi wrote: > On Fri, Sep 27, 2013 at 08:47:10PM +0200, Daniel Mack wrote: >> On 25.09.2013 15:48, Felipe Balbi wrote: >>> On Tue, Sep 24, 2013 at 08:57:08AM +0200, Daniel Mack wrote: >>>> On 23.09.2013 23:20, Felipe Balbi wrote: >>>>> On Sun, Sep 22, 2013 at 01:46:58PM +0200, Daniel Mack wrote: >>>> >>>>>> -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); >>>>> >>>>> you could as well remove the DEV_PM_OPS trickery. Ditto for all other >>>>> patches >>>> >>>> Yes, I noticed that too. That's why I sent out a v2 of the series :) >>> >>> Awesome :-) Thanks dude >> >> So are you happy with this v2? >> It's just a cosmetic cleanup after all ... > > I can't see v2 anywhere... The only version I have is the one with > DEV_PM_OPS still in it. Ok, I'll resend them later. Daniel -- 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