From: Vitalii Mordan <mordan@ispras.ru>
Date: Wed, 12 Feb 2025 17:28:03 +0000 (+0300)
Subject: i2c: pxa: fix call balance of i2c->clk handling routines
X-Git-Tag: v6.15-rc1~74^2~1^2~39
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=be7113d2e2a6f20cbee99c98d261a1fd6fd7b549;p=linux-block.git

i2c: pxa: fix call balance of i2c->clk handling routines

If the clock i2c->clk was not enabled in i2c_pxa_probe(), it should not be
disabled in any path.

Found by Linux Verification Center (linuxtesting.org) with Klever.

Signed-off-by: Vitalii Mordan <mordan@ispras.ru>
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
Link: https://lore.kernel.org/r/20250212172803.1422136-1-mordan@ispras.ru
---

diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
index cb6988482673..4415a29f749b 100644
--- a/drivers/i2c/busses/i2c-pxa.c
+++ b/drivers/i2c/busses/i2c-pxa.c
@@ -1503,7 +1503,10 @@ static int i2c_pxa_probe(struct platform_device *dev)
 				i2c->adap.name);
 	}
 
-	clk_prepare_enable(i2c->clk);
+	ret = clk_prepare_enable(i2c->clk);
+	if (ret)
+		return dev_err_probe(&dev->dev, ret,
+				     "failed to enable clock\n");
 
 	if (i2c->use_pio) {
 		i2c->adap.algo = &i2c_pxa_pio_algorithm;