media: i2c: s5c73m3: return 0 instead of 'ret'.
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Thu, 26 Jan 2023 12:18:18 +0000 (13:18 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Wed, 8 Feb 2023 07:30:29 +0000 (08:30 +0100)
Since 'ret' is known to be 0, just return '0'. This fixes a smatch warning:

s5c73m3-core.c:439 __s5c73m3_s_stream() warn: missing error code? 'ret'

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/i2c/s5c73m3/s5c73m3-core.c

index 318a4ec2d8a57c5df8de880ded27aafae9aa65ce..7938a3327d3ee4eeb81ec042d0e00e74fe6b67d4 100644 (file)
@@ -435,7 +435,7 @@ static int __s5c73m3_s_stream(struct s5c73m3 *state, struct v4l2_subdev *sd,
        state->streaming = !!on;
 
        if (!on)
-               return ret;
+               return 0;
 
        if (state->apply_fiv) {
                ret = s5c73m3_set_frame_rate(state);