i2c: uniphier: remove printout on handled timeouts
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Wed, 10 Apr 2024 11:24:32 +0000 (13:24 +0200)
committerAndi Shyti <andi.shyti@kernel.org>
Sun, 5 May 2024 22:56:37 +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>
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
drivers/i2c/busses/i2c-uniphier.c

index 854ac25b586285a1d79e1c2f7e0fe6ad504016e8..e1b4c80e028545fedf296b56c3361a9cc7a6e7b8 100644 (file)
@@ -71,10 +71,8 @@ static int uniphier_i2c_xfer_byte(struct i2c_adapter *adap, u32 txdata,
        writel(txdata, priv->membase + UNIPHIER_I2C_DTRM);
 
        time_left = wait_for_completion_timeout(&priv->comp, adap->timeout);
-       if (unlikely(!time_left)) {
-               dev_err(&adap->dev, "transaction timeout\n");
+       if (unlikely(!time_left))
                return -ETIMEDOUT;
-       }
 
        rxdata = readl(priv->membase + UNIPHIER_I2C_DREC);
        if (rxdatap)