Re: [PATCH] i2c: qcom-cci: drop custom suspend/resume and rely on runtime PM helpers
Andi Shyti <[email protected]> Tue, 28 Jul 2026 23:37:39 +0200
| Newsgroups | org.kernel.vger.linux-i2c,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel,org.kernel.vger.stable |
|---|---|
| Message-ID | <[email protected]> |
Hi Wenmeng,
On Thu, Jun 25, 2026 at 05:42:45PM +0800, Wenmeng Liu wrote:
> cci_resume() unconditionally calls cci_resume_runtime() regardless of
> the runtime PM state.
>
> If the device is already runtime-suspended before system suspend,
> the clock is re-enabled while runtime_status remains RPM_SUSPENDED.
> As a result, pm_request_autosuspend() does not arm the timer,
> leaving the clock permanently enabled.
>
> Fixes: e517526195de ("i2c: Add Qualcomm CCI I2C driver")
> Cc: [email protected]
> Signed-off-by: Wenmeng Liu <[email protected]>
merged to i2c/i2c-fixes.
Thanks,
Andi