On Wed Apr 30 11:27:42 2025 +0300, Dan Carpenter wrote:
> The "ret" variable is not initialized on success. Set it to zero.
>
> Fixes: e2a3b695bc5f ("media: i2c: ds90ub960: Configure serializer using
> back-channel")
> Cc: sta...@vger.kernel.org
> Signed-off-by: Dan Carpenter <dan.carpen...@linaro.org>
> Reviewed-by: Jai Luthra <jai.lut...@ideasonboard.com>
> Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>
Patch committed.
Thanks,
Hans Verkuil
drivers/media/i2c/ds90ub960.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
diff --git a/drivers/media/i2c/ds90ub960.c b/drivers/media/i2c/ds90ub960.c
index 665efd661882..ed9ace1a5476 100644
--- a/drivers/media/i2c/ds90ub960.c
+++ b/drivers/media/i2c/ds90ub960.c
@@ -2126,7 +2126,7 @@ static int ub960_rxport_bc_ser_config(struct ub960_rxport
*rxport)
struct ub960_data *priv = rxport->priv;
struct device *dev = &priv->client->dev;
u8 nport = rxport->nport;
- int ret;
+ int ret = 0;
/* Skip port if serializer's address is not known */
if (rxport->ser.addr < 0) {
