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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to