[media] drivers/media/platform/soc_camera/soc_camera.c: fix error return code
Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> ( if@p1 (\(ret < 0\|ret != 0\)) { ... return ret; } | ret@p1 = 0 ) ... when != ret = e1 when != &ret *if(...) { ... when != ret = e2 when forall return ret; } // </smpl> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com> [g.liakhovetski@gmx.de: follow the soc_camera.c style: check "ret < 0"] Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
This commit is contained in:
parent
58dec65d71
commit
ab99cb18ae
|
@ -1184,7 +1184,8 @@ static int soc_camera_probe(struct soc_camera_device *icd)
|
||||||
sd->grp_id = soc_camera_grp_id(icd);
|
sd->grp_id = soc_camera_grp_id(icd);
|
||||||
v4l2_set_subdev_hostdata(sd, icd);
|
v4l2_set_subdev_hostdata(sd, icd);
|
||||||
|
|
||||||
if (v4l2_ctrl_add_handler(&icd->ctrl_handler, sd->ctrl_handler, NULL))
|
ret = v4l2_ctrl_add_handler(&icd->ctrl_handler, sd->ctrl_handler, NULL);
|
||||||
|
if (ret < 0)
|
||||||
goto ectrl;
|
goto ectrl;
|
||||||
|
|
||||||
/* At this point client .probe() should have run already */
|
/* At this point client .probe() should have run already */
|
||||||
|
|
Loading…
Reference in New Issue