Hi Lee, <snip> > +#ifdef CONFIG_PM_SLEEP > +static int st_wdog_suspend(struct device *dev) > +{ > + struct st_wdog *st_wdog = watchdog_get_drvdata(&st_wdog_dev); > + > + if (watchdog_active(&st_wdog_dev)) > + st_wdog_stop(&st_wdog_dev); > + > + st_wdog_setup(st_wdog, false); I was expecting the suspend / resume callbacks would be disabling / enabling the IP clock. Is there a reason why it doesn't? >From looking at the code by this point the watchdog has already been disabled so gating the IP seems reasonable. regards, Peter -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html