i2c: qcom-geni: remove printout on handled timeouts
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Wed, 10 Apr 2024 11:24:25 +0000 (13:24 +0200)
committerAndi Shyti <andi.shyti@kernel.org>
Sun, 5 May 2024 22:56:35 +0000 (00:56 +0200)
I2C and SMBus timeouts are not something the user needs to be informed
about on controller level. The client driver may know if that really is
a problem and give more detailed information to the user. The controller
should just pass this information upwards. Remove the printout.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Bjorn Andersson <andersson@kernel.org>
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
drivers/i2c/busses/i2c-qcom-geni.c

index da94df466e83c9d34c6212681c087cafc8a6b788..090b4846ed62b1fee74b7b8d23de8173a51bc209 100644 (file)
@@ -630,11 +630,8 @@ static int geni_i2c_gpi_xfer(struct geni_i2c_dev *gi2c, struct i2c_msg msgs[], i
                dma_async_issue_pending(gi2c->tx_c);
 
                timeout = wait_for_completion_timeout(&gi2c->done, XFER_TIMEOUT);
-               if (!timeout) {
-                       dev_err(gi2c->se.dev, "I2C timeout gpi flags:%d addr:0x%x\n",
-                               gi2c->cur->flags, gi2c->cur->addr);
+               if (!timeout)
                        gi2c->err = -ETIMEDOUT;
-               }
 
                if (gi2c->err) {
                        ret = gi2c->err;