i3c: renesas: Simplify return statement in 'renesas_i3c_daa'

Wolfram Sang · alexandrebelloni · commit bc7dd24c114e · 2025-09-16T17:06:43.000+02:00
There was already a bail out for 'ret < 0', so we can always return success at the end of the function. Reported-by: Dan Carpenter <dan.carpenter@linaro.org> Closes: https://lore.kernel.org/r/aIyzJ7HOENL1qp1l@stanley.mountain Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Tested-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com> Link: https://lore.kernel.org/r/20250803211256.18513-2-wsa+renesas@sang-engineering.com Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
diff --git a/drivers/i3c/master/renesas-i3c.c b/drivers/i3c/master/renesas-i3c.c
@@ -679,7 +679,7 @@ static int renesas_i3c_daa(struct i3c_master_controller *m)
 			i3c_master_add_i3c_dev_locked(m, i3c->addrs[pos]);
 	}
 
-	return ret < 0 ? ret : 0;
+	return 0;
 }
 
 static bool renesas_i3c_supports_ccc_cmd(struct i3c_master_controller *m,

Original file line number	Diff line number	Diff line change
`@@ -679,7 +679,7 @@ static int renesas_i3c_daa(struct i3c_master_controller *m)`
`679`	`679`	`i3c_master_add_i3c_dev_locked(m, i3c->addrs[pos]);`
`680`	`680`	`}`
`681`	`681`
`682`		`- return ret < 0 ? ret : 0;`
	`682`	`+ return 0;`
`683`	`683`	`}`
`684`	`684`
`685`	`685`	`static bool renesas_i3c_supports_ccc_cmd(struct i3c_master_controller *m,`