i2c: qup: remove printout on handled timeouts
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Wed, 10 Apr 2024 11:24:26 +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-qup.c

index 598102d16677a1e20f264ca26fe4066320851c18..c9b43a3c4bd3c04badea2288806abb912e967cdb 100644 (file)
@@ -793,10 +793,8 @@ static int qup_i2c_bam_schedule_desc(struct qup_i2c_dev *qup)
                dma_async_issue_pending(qup->brx.dma);
        }
 
-       if (!wait_for_completion_timeout(&qup->xfer, qup->xfer_timeout)) {
-               dev_err(qup->dev, "normal trans timed out\n");
+       if (!wait_for_completion_timeout(&qup->xfer, qup->xfer_timeout))
                ret = -ETIMEDOUT;
-       }
 
        if (ret || qup->bus_err || qup->qup_err) {
                reinit_completion(&qup->xfer);