i2c: ali15x3: remove printout on handled timeouts
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Tue, 23 Apr 2024 12:13:21 +0000 (14:13 +0200)
committerAndi Shyti <andi.shyti@kernel.org>
Sun, 5 May 2024 22:56:49 +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: Jean Delvare <jdelvare@suse.de>
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
drivers/i2c/busses/i2c-ali15x3.c

index d2fa30deb054c7fafe0d2e1749ca53e684a1fa32..956e5020d71e2f9dd35f5c678bbbf0e5bdf650d6 100644 (file)
@@ -294,10 +294,8 @@ static int ali15x3_transaction(struct i2c_adapter *adap)
                 && (timeout++ < MAX_TIMEOUT));
 
        /* If the SMBus is still busy, we give up */
-       if (timeout > MAX_TIMEOUT) {
+       if (timeout > MAX_TIMEOUT)
                result = -ETIMEDOUT;
-               dev_err(&adap->dev, "SMBus Timeout!\n");
-       }
 
        if (temp & ALI15X3_STS_TERM) {
                result = -EIO;