Re: [PATCH v2 4/6] pwm: omap-dmtimer: Fix pwm disabling sequence

From: Tony Lindgren
Date: Fri Mar 13 2020 - 11:34:55 EST


* Tony Lindgren <tony@xxxxxxxxxxx> [200312 00:59]:
> * Lokesh Vutla <lokeshvutla@xxxxxx> [200311 04:14]:
> > However, I see an issue with the patch itself as pm_runtime is not disabled
> > after the pwm is stopped. Not sure how that could be nullified with this approach.
>
> Hmm yeah not sure what could be used to clear things
> when the current cycle is completed unless there's
> some interrupt for it.

You could enable pm_runtime_use_autosuspend() for pwm use,
then set the timeout to the cycle length, then in the
runtime_suspend make sure the enable bit is cleared if
requested.

But this too seems inaccurate, it would be best to clear
the enable bit on some cycle completion interrupt if
such thing is available.

Regards,

Tony