On Sat Jul 4 15:16:38 2026 +0800, Pengpeng Hou wrote:
> dcmi_runtime_resume() returns clock enable failures, but dcmi_resume()
> currently ignores the result of pm_runtime_force_resume() and always
> returns success.
> 
> Return the pm_runtime_force_resume() result so system resume does not
> hide a failed runtime resume.
> 
> Signed-off-by: Pengpeng Hou <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/st/stm32/stm32-dcmi.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

---

diff --git a/drivers/media/platform/st/stm32/stm32-dcmi.c 
b/drivers/media/platform/st/stm32/stm32-dcmi.c
index eeb0199864dd..b1f9af8edf6e 100644
--- a/drivers/media/platform/st/stm32/stm32-dcmi.c
+++ b/drivers/media/platform/st/stm32/stm32-dcmi.c
@@ -2273,9 +2273,7 @@ static int dcmi_resume(struct device *dev)
        pinctrl_pm_select_default_state(dev);
 
        /* clock enable */
-       pm_runtime_force_resume(dev);
-
-       return 0;
+       return pm_runtime_force_resume(dev);
 }
 
 static const struct dev_pm_ops dcmi_pm_ops = {
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to