Hi Sundar, On Fri, Dec 03, 2010 at 08:35:46PM +0530, Sundar Iyer wrote: Acked-by: Samuel Ortiz <sameo@xxxxxxxxxxxxxxx> One remark though: > + > +static const struct dev_pm_ops tc3589x_dev_pm_ops = { > + .suspend = tc3589x_suspend, > + .resume = tc3589x_resume, > +}; > +#endif SIMPLE_DEV_PM_OPS or UNIVERSAL_DEV_PM_OPS may be useful and save one ifdef here. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- 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