On Wed, Mar 04, 2015 at 12:12:36PM +0200, Peter Ujfalusi wrote: > On 03/03/2015 05:07 PM, Mark Brown wrote: > > I see there's actually a balanced put in the resume path... I see > > what's going on here but I'm thinking that perhaps something more > > explicit that calls the ops directly and checks pm_runtime_is_enabled() > > might be clearer? > The issue with the pm_runtime_is_enabled() is going to be in the resume path, > as I did get_sync() it will return true at the end all the time. You can just store the state before you do the enable...