<div dir="ltr">Reviewed-by: Vinay Simha BN <<a href="mailto:simhavcs@gmail.com">simhavcs@gmail.com</a>></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 16, 2022 at 3:55 AM Jiri Vanek <<a href="mailto:jirivanek1@gmail.com">jirivanek1@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Function for reading from i2c device register displays error message even<br>
if reading ends correctly. Add return to avoid falling through into<br>
the fail label.<br>
<br>
Signed-off-by: Jiri Vanek <<a href="mailto:jirivanek1@gmail.com" target="_blank">jirivanek1@gmail.com</a>><br>
---<br>
 drivers/gpu/drm/bridge/tc358775.c | 1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c<br>
index 62a7ef352daa..cd2721ab02a9 100644<br>
--- a/drivers/gpu/drm/bridge/tc358775.c<br>
+++ b/drivers/gpu/drm/bridge/tc358775.c<br>
@@ -339,6 +339,7 @@ static void d2l_read(struct i2c_client *i2c, u16 addr, u32 *val)<br>
                goto fail;<br>
<br>
        pr_debug("d2l: I2C : addr:%04x value:%08x\n", addr, *val);<br>
+       return;<br>
<br>
 fail:<br>
        dev_err(&i2c->dev, "Error %d reading from subaddress 0x%x\n",<br>
-- <br>
2.30.2<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">regards,<br>vinaysimha</div>