TO: Andy Ross <[email protected]> CC: "Nisha Kumar-Mayernik" <[email protected]>
drivers/external_drivers/camera/drivers/media/i2c/mt9v113.c:523:1-4: WARNING: end returns can be simpified drivers/external_drivers/camera/drivers/media/i2c/mt9v113.c:410:1-4: WARNING: end returns can be simpified drivers/external_drivers/camera/drivers/media/i2c/mt9v113.c:1417:5-8: WARNING: end returns can be simpified if tested value is negative or 0 drivers/external_drivers/camera/drivers/media/i2c/mt9v113.c:1432:5-8: WARNING: end returns can be simpified if tested value is negative or 0 Simplify a trivial if-return sequence. Possibly combine with a preceding function call. Generated by: scripts/coccinelle/misc/simple_return.cocci CC: Andy Ross <[email protected]> Signed-off-by: Fengguang Wu <[email protected]> --- mt9v113.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) --- a/drivers/external_drivers/camera/drivers/media/i2c/mt9v113.c +++ b/drivers/external_drivers/camera/drivers/media/i2c/mt9v113.c @@ -407,10 +407,7 @@ static int mt9v113_write_reg_array(struc } } - err = __mt9v113_flush_reg_array(client, &ctrl); - if (err) - return err; - return 0; + return __mt9v113_flush_reg_array(client, &ctrl); } static int mt9v113_wait_standby(struct v4l2_subdev *sd, int flag) @@ -520,10 +517,7 @@ static int mt9v113_refresh(struct v4l2_s dev_err(&client->dev, "err refresh seq: %d", ret); return ret; } - ret = mt9v113_wait_refresh(sd); - if (ret) - return ret; - return 0; + return mt9v113_wait_refresh(sd); } static int mt9v113_wait_pll_lock(struct v4l2_subdev *sd) _______________________________________________ kbuild mailing list [email protected] https://lists.01.org/mailman/listinfo/kbuild
