TO: Andy Ross <[email protected]> CC: "Nisha Kumar-Mayernik" <[email protected]>
drivers/external_drivers/camera/drivers/media/i2c/mt9d113.c:936:1-4: WARNING: end returns can be simpified drivers/external_drivers/camera/drivers/media/i2c/mt9d113.c:411:1-4: WARNING: end returns can be simpified drivers/external_drivers/camera/drivers/media/i2c/mt9d113.c:1661:5-8: WARNING: end returns can be simpified if tested value is negative or 0 drivers/external_drivers/camera/drivers/media/i2c/mt9d113.c:1676: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]> --- mt9d113.c | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) --- a/drivers/external_drivers/camera/drivers/media/i2c/mt9d113.c +++ b/drivers/external_drivers/camera/drivers/media/i2c/mt9d113.c @@ -408,10 +408,7 @@ static int mt9d113_write_reg_array(struc } } - err = __mt9d113_flush_reg_array(client, &ctrl); - if (err) - return err; - return 0; + return __mt9d113_flush_reg_array(client, &ctrl); } static int mt9d113_wait_standby(struct v4l2_subdev *sd, int flag) @@ -933,11 +930,7 @@ static int mt9d113_init_common(struct v4 /* * Load soc2030 patch */ - ret = mt9d113_patch(sd); - if (ret) - return ret; - - return 0; + return mt9d113_patch(sd); } static int power_up(struct v4l2_subdev *sd) _______________________________________________ kbuild mailing list [email protected] https://lists.01.org/mailman/listinfo/kbuild
