* 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