On Wed, Jul 08, 2026 at 05:05:52PM +0800, Guangshuo Li wrote: > The change referenced by the Fixes tag avoids a runtime PM resume during > probe by powering the ISP domain directly with dev_pm_genpd_resume() > before calling i2c_dw_probe(). Runtime PM is enabled only after > i2c_dw_probe() has succeeded. > > However, the i2c_dw_probe() failure path still uses the old runtime PM > cleanup path. At that point pm_runtime_enable() has not been called yet, > so pm_runtime_disable() is not paired with any enable. The failure path > also skips the dev_pm_genpd_suspend() needed to balance the direct > dev_pm_genpd_resume() call. > > Suspend the genpd on the i2c_dw_probe() failure path instead of disabling > runtime PM. This keeps the direct genpd resume/suspend pairing symmetric > on both success and failure paths. > > Fixes: e2f1ada8e089 ("i2c: designware: amdisp: Fix resume-probe race condition issue") > Signed-off-by: Guangshuo Li > Acked-by: Pratap Nirujogi > Acked-by: Mukesh Kumar Savaliya > Acked-by: Bin Du > Tested-by: Bin Du Acked-by: Wolfram Sang I'll put it in my for-andi branch, but actually this is a fix for this cycle.