From: Hugues Fruchet Date: Mon, 22 Jan 2018 10:06:40 +0000 (-0500) Subject: media: ov5640: fix spurious streamon failures X-Git-Url: http://git.cdn.openwrt.org/?a=commitdiff_plain;h=6146fde355b4eef760102a8ce0d063c6ecf53cb8;p=openwrt%2Fstaging%2Fblogic.git media: ov5640: fix spurious streamon failures Time to time, stream on is failing with a strange positive error. Error code is returned erroneously by ov5640_set_ctrl_exposure() due to ov5640_get_vts() return value wrongly treated as error. Fix this by forcing ret to 0 after ov5640_get_vts() success call, in order that ret is set to success for rest of code sequence. Signed-off-by: Hugues Fruchet Signed-off-by: Sakari Ailus Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c index f0177421b752..e2dd352224c7 100644 --- a/drivers/media/i2c/ov5640.c +++ b/drivers/media/i2c/ov5640.c @@ -2057,6 +2057,7 @@ static int ov5640_set_ctrl_exposure(struct ov5640_dev *sensor, int exp) if (ret < 0) return ret; max_exp += ret; + ret = 0; if (ctrls->exposure->val < max_exp) ret = ov5640_set_exposure(sensor, ctrls->exposure->val);