Hi Geert and Rafael,

> > I've always[*] considered a pm_runtime_get_sync() failure to be fatal
> > (or: cannot happen), and that there's nothing that can be done to
> > recover.  Hence I never checked the function's return value.
> > Was that wrong?
> 
> No, it wasn't.  It is the right thing to do in the majority of cases.

OK, if *not checking* the retval is the major use case, then I
understand that ref counting takes place.

However, that probably means that for most patches I am getting, the
better fix would be to remove the error checking? (I assume most people
put the error check in there to be on the "safe side" without having a
real argument to really do it.)

And thanks for putting more hints to kernel doc! I think this will help
the case a lot.

Kind regards,

   Wolfram