From b68875656c3bad36c4d5b9f44b7689e3e1864e09 Mon Sep 17 00:00:00 2001 From: Andrew Goodbody Date: Mon, 28 Jul 2025 16:04:43 +0100 Subject: [PATCH] i2c: rcar_iic: Do not use unitialised variable In rcar_iic_xfer if nmsgs == 0 the ret will not be assigned to. As ret will always be 0 if the for loop is entered, may as well just return 0. This issue was found by Smatch. Signed-off-by: Andrew Goodbody Reviewed-by: Heiko Schocher --- drivers/i2c/rcar_iic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/rcar_iic.c b/drivers/i2c/rcar_iic.c index e019d06be41..42d575e29ba 100644 --- a/drivers/i2c/rcar_iic.c +++ b/drivers/i2c/rcar_iic.c @@ -191,7 +191,7 @@ static int rcar_iic_xfer(struct udevice *dev, struct i2c_msg *msg, int nmsgs) return -EREMOTEIO; } - return ret; + return 0; } static int rcar_iic_set_speed(struct udevice *dev, uint speed) -- 2.47.2