Re: [PATCH v2] phy: phy-twl4030-usb: fix denied runtime access

From: Tony Lindgren
Date: Wed Feb 20 2019 - 17:31:38 EST


* Andreas Kemnade <andreas@xxxxxxxxxxxx> [180922 09:48]:
> When runtime is not enabled, pm_runtime_get_sync() returns -EACCESS,
> the counter will be incremented but the resume callback not called,
> so enumeration and charging will not start properly.
> To avoid that happen, disable irq on suspend and recheck on resume.
>
> Practically this happens when the device is woken up from suspend by
> plugging in usb.
>
> Signed-off-by: Andreas Kemnade <andreas@xxxxxxxxxxxx>
> ---
> Changes in v2:
> * use pm suspend/resume callback instead of delayed_work
> as suggested by Dmitry

Hmm it just occurred to me that this issue too might be fixed with commit
c6e2bd956936 ("i2c: omap: Use noirq system sleep pm ops to idle device
for suspend"). Andreas, care to check? That is if this is still an
issue.

Regards,

Tony