Hi Wolfram, Thanks for your patch! On Sun, Aug 03, 2025 at 11:12:57PM +0200, Wolfram Sang wrote: > 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@xxxxxxxxxx> > Closes: https://lore.kernel.org/r/aIyzJ7HOENL1qp1l@stanley.mountain > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Tested-by: Tommaso Merciai <tommaso.merciai.xr@xxxxxxxxxxxxxx> > --- > drivers/i3c/master/renesas-i3c.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/i3c/master/renesas-i3c.c b/drivers/i3c/master/renesas-i3c.c > index 174d3dc5d276..275f7b924288 100644 > --- 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, > -- > 2.47.2 > > Thanks & Regards, Tommaso
