Messages in this thread | | | Date | Wed, 12 Jan 2022 10:19:08 +0100 | Subject | Re: [PATCH] media: i2c: remove unneeded variable | From | Hans Verkuil <> |
| |
On 12/01/2022 10:17, cgel.zte@gmail.com wrote: > From: Changcheng Deng <deng.changcheng@zte.com.cn> > > Remove unneeded variable used to store return value.
Please add the driver name in the subject prefix! Why is that so hard to remember?
Regards,
Hans
> > Reported-by: Zeal Robot <zealci@zte.com.cn> > Signed-off-by: Changcheng Deng <deng.changcheng@zte.com.cn> > --- > drivers/media/i2c/ov5693.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/i2c/ov5693.c b/drivers/media/i2c/ov5693.c > index 2784fcf67f3b..a55910f6283a 100644 > --- a/drivers/media/i2c/ov5693.c > +++ b/drivers/media/i2c/ov5693.c > @@ -950,7 +950,6 @@ static int ov5693_set_fmt(struct v4l2_subdev *sd, > unsigned int width, height; > unsigned int hblank; > int exposure_max; > - int ret = 0; > > crop = __ov5693_get_pad_crop(ov5693, state, format->pad, format->which); > > @@ -982,7 +981,7 @@ static int ov5693_set_fmt(struct v4l2_subdev *sd, > format->format = *fmt; > > if (format->which == V4L2_SUBDEV_FORMAT_TRY) > - return ret; > + return 0; > > mutex_lock(&ov5693->lock); > > @@ -1012,7 +1011,7 @@ static int ov5693_set_fmt(struct v4l2_subdev *sd, > exposure_max)); > > mutex_unlock(&ov5693->lock); > - return ret; > + return 0; > } > > static int ov5693_get_selection(struct v4l2_subdev *sd,
| |