This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: ov5640: add error trace in case of i2c read failure
Author:  Hugues Fruchet <hugues.fruc...@st.com>
Date:    Wed Jan 31 07:46:17 2018 -0500

Add an error trace in ov5640_read_reg() in case of i2c_transfer()
failure.
Uniformize error traces using dev_err instead v4l2_err.

Signed-off-by: Hugues Fruchet <hugues.fruc...@st.com>
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>

 drivers/media/i2c/ov5640.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

---

diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c
index 99a590263070..6f1846081dbe 100644
--- a/drivers/media/i2c/ov5640.c
+++ b/drivers/media/i2c/ov5640.c
@@ -839,7 +839,7 @@ static int ov5640_write_reg(struct ov5640_dev *sensor, u16 
reg, u8 val)
 
        ret = i2c_transfer(client->adapter, &msg, 1);
        if (ret < 0) {
-               v4l2_err(&sensor->sd, "%s: error: reg=%x, val=%x\n",
+               dev_err(&client->dev, "%s: error: reg=%x, val=%x\n",
                        __func__, reg, val);
                return ret;
        }
@@ -868,8 +868,11 @@ static int ov5640_read_reg(struct ov5640_dev *sensor, u16 
reg, u8 *val)
        msg[1].len = 1;
 
        ret = i2c_transfer(client->adapter, msg, 2);
-       if (ret < 0)
+       if (ret < 0) {
+               dev_err(&client->dev, "%s: error: reg=%x\n",
+                       __func__, reg);
                return ret;
+       }
 
        *val = buf[0];
        return 0;

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to