>-----Original Message----- >From: Miaoqian Lin [mailto:linmq006@xxxxxxxxx] >Sent: Monday, March 7, 2022 1:22 PM >To: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>; Mauro Carvalho >Chehab <mchehab@xxxxxxxxxx>; Krzysztof Kozlowski ><krzysztof.kozlowski@xxxxxxxxxxxxx>; Alim Akhtar ><alim.akhtar@xxxxxxxxxxx>; Kyungmin Park ><kyungmin.park@xxxxxxxxxxx>; Younghwan Joo ><yhwan.joo@xxxxxxxxxxx>; linux-media@xxxxxxxxxxxxxxx; linux-arm- >kernel@xxxxxxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx; linux- >kernel@xxxxxxxxxxxxxxx >Cc: linmq006@xxxxxxxxx >Subject: [PATCH] media: exynos4-is: Fix PM disable depth imbalance in >fimc_is_probe > >If probe fails then we need to call pm_runtime_disable() to balance out the >previous pm_runtime_enable() call. > >Fixes: 9a761e436843 ("[media] exynos4-is: Add Exynos4x12 FIMC-IS driver") >Signed-off-by: Miaoqian Lin <linmq006@xxxxxxxxx> >--- Thanks for fixing this Reviewed-by: Alim Akhtar <alim.akhtar@xxxxxxxxxxx> > drivers/media/platform/exynos4-is/fimc-is.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > >diff --git a/drivers/media/platform/exynos4-is/fimc-is.c >b/drivers/media/platform/exynos4-is/fimc-is.c >index e55e411038f4..81b290dace3a 100644 >--- a/drivers/media/platform/exynos4-is/fimc-is.c >+++ b/drivers/media/platform/exynos4-is/fimc-is.c >@@ -830,7 +830,7 @@ static int fimc_is_probe(struct platform_device *pdev) > > ret = pm_runtime_resume_and_get(dev); > if (ret < 0) >- goto err_irq; >+ goto err_pm_disable; > > vb2_dma_contig_set_max_seg_size(dev, DMA_BIT_MASK(32)); > >@@ -864,6 +864,8 @@ static int fimc_is_probe(struct platform_device *pdev) > pm_runtime_put_noidle(dev); > if (!pm_runtime_enabled(dev)) > fimc_is_runtime_suspend(dev); >+err_pm_disable: >+ pm_runtime_disable(dev); > err_irq: > free_irq(is->irq, is); > err_clk: >-- >2.17.1
