media: ov5645: Fix write_reg return code

I2C transfer functions return number of successful operations (on success).

Do not return the received positive return code but instead return 0 on
success. The users of write_reg function already use this logic.

Signed-off-by: Todor Tomov <todor.tomov@linaro.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

diff --git a/drivers/media/i2c/ov5645.c b/drivers/media/i2c/ov5645.c
index 4e3142a7e5a7a76daeeb605a391a85f893bc39c3..b3f762578f7f8dbac0d09bdc6a873f5d274a12a9 100644 (file)
--- a/drivers/media/i2c/ov5645.c
+++ b/drivers/media/i2c/ov5645.c
@@ -600,11 +600,13 @@ static int ov5645_write_reg(struct ov5645 *ov5645, u16 reg, u8 val)
        regbuf[2] = val;
 
        ret = i2c_master_send(ov5645->i2c_client, regbuf, 3);
-       if (ret < 0)
+       if (ret < 0) {
                dev_err(ov5645->dev, "%s: write reg error %d: reg=%x, val=%x\n",
                        __func__, ret, reg, val);
+               return ret;
+       }
 
-       return ret;
+       return 0;
 }
 
 static int ov5645_read_reg(struct ov5645 *ov5645, u16 reg, u8 *val)