On Mon, Jan 02, 2012 at 11:07:59AM +0900, Jaehoon Chung wrote: > On 12/30/2011 11:24 AM, Mark Brown wrote: > > +#include <linux/pm.h> > I think this include is unnecessary. I think you are wrong. What makes you say that you think it is not needed? > > +#ifdef CONFIG_PM > > static const struct dev_pm_ops sdhci_s3c_pmops = { > > - .suspend = sdhci_s3c_suspend, > > - .resume = sdhci_s3c_resume, > > + SET_SYSTEM_SLEEP_PM_OPS(sdhci_s3c_suspend, sdhci_s3c_resume) Note that above. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html