Hi, Get it. Thanks very much. BRs Xiubo > -----Original Message----- > From: wimvs@xxxxxxxxxxxxxxxxxx [mailto:wimvs@xxxxxxxxxxxxxxxxxx] On Behalf Of > Wim Van Sebroeck > Sent: Monday, December 22, 2014 10:05 PM > To: Xiubo Li-B47053 > Cc: linux@xxxxxxxxxxxx; linux-watchdog@xxxxxxxxxxxxxxx > Subject: Re: [PATCH v3] watchdog: imx2_wdt: Add power management support. > > Hi, > > > Add power management operations(suspend and resume) as part of > > dev_pm_ops for IMX2 watchdog driver. > > > > If PM will be supported, please make sure that the wdev->clk > > could disable the watchdog's counter input clock source or can > > mask watchdog's reset request to the core. > > > > If watchdog is still used by consumers and resumes from deep > > sleep state, we need to restart the watchdog again without > > enabling the timer. > > > > If watchdog been has started --> stopped by the consumers and > > resumes from non-deep sleep state, then start the timer again. > > > > If watchdog has been started --> stopped by the consumers and > > resumes from deep sleep state, will do nothing. The watchdog > > will be restarted by consumers next time to be used. > > > > Signed-off-by: Xiubo Li <Li.Xiubo@xxxxxxxxxxxxx> > > --- > > > > Change in V3: > > - Uses standard multi-line comments. > > V1 went upstream (my mistake), diff between V1 and V3 is now in linux- > watchdog-next. > > Kind regards, > Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
