On Thu, Feb 14, 2013 at 01:15:52PM +0200, Felipe Balbi wrote: > struct platform_device *pdev = to_platform_device(dev); > + struct omap_device *od = to_omap_device(pdev); > int ret; > > ret = pm_generic_runtime_suspend(dev); > > + if (!od) > + goto out; > + > if (!ret) What's wrong with the single line change per function of: - if (!ret) + if (!ret && to_omap_device(pdev)) ? > omap_device_idle(pdev); > > +out: > return ret; > } -- 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