media: vimc: Remove unneeded return statement in vimc_sen_s_stream()

Niklas Söderlund · mchehab · commit fe97d64d7258 · 2019-05-28T13:11:51.000-04:00
The other subdevice implementations in vimc (debayer and scaler) which
share their code structure with the sensor do not have an explicit
return statement at the end of the s_stream(0) code path. Align the
sensor subdevice by dropping the return statement.

Signed-off-by: Niklas Söderlund &lt;niklas.soderlund+renesas@ragnatech.se&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab+samsung@kernel.org&gt;
diff --git a/drivers/media/platform/vimc/vimc-sensor.c b/drivers/media/platform/vimc/vimc-sensor.c
@@ -221,7 +221,6 @@ static int vimc_sen_s_stream(struct v4l2_subdev *sd, int enable)
 
 		vfree(vsen->frame);
 		vsen->frame = NULL;
-		return 0;
 	}
 
 	return 0;

Original file line number	Diff line number	Diff line change
`@@ -221,7 +221,6 @@ static int vimc_sen_s_stream(struct v4l2_subdev *sd, int enable)`
`221`	`221`
`222`	`222`	`vfree(vsen->frame);`
`223`	`223`	`vsen->frame = NULL;`
`224`		`- return 0;`
`225`	`224`	`}`
`226`	`225`
`227`	`226`	`return 0;`