2012/1/2 Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>: > 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? If that didn't include,,what's happen? I think that is already included...Anyway..this patch looks fine to me.. (if use with your patch[2/2]..) Thanks, Jaehoon Chung > >> > +#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 -- 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