mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-01-10 15:19:51 +00:00
media: i2c: imx219: Remove unneeded goto
Simplify the imx219_set_stream() by removing an unneeded goto statement, and its corresponding error label. The natural flow of the function is correct. While at it, drop a useless comment. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Dave Stevenson <dave.stevenson@raspberrypi.com> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
This commit is contained in:
parent
d03dfb7d4c
commit
d83b40ef9c
@ -760,19 +760,11 @@ static int imx219_set_stream(struct v4l2_subdev *sd, int enable)
|
|||||||
|
|
||||||
state = v4l2_subdev_lock_and_get_active_state(sd);
|
state = v4l2_subdev_lock_and_get_active_state(sd);
|
||||||
|
|
||||||
if (enable) {
|
if (enable)
|
||||||
/*
|
|
||||||
* Apply default & customized values
|
|
||||||
* and then start streaming.
|
|
||||||
*/
|
|
||||||
ret = imx219_start_streaming(imx219, state);
|
ret = imx219_start_streaming(imx219, state);
|
||||||
if (ret)
|
else
|
||||||
goto unlock;
|
|
||||||
} else {
|
|
||||||
imx219_stop_streaming(imx219);
|
imx219_stop_streaming(imx219);
|
||||||
}
|
|
||||||
|
|
||||||
unlock:
|
|
||||||
v4l2_subdev_unlock_state(state);
|
v4l2_subdev_unlock_state(state);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user