diff options
author | Wolfram Sang <wsa+renesas@sang-engineering.com> | 2024-04-10 13:24:32 +0200 |
---|---|---|
committer | Andi Shyti <andi.shyti@kernel.org> | 2024-05-06 00:56:37 +0200 |
commit | 1d428f5d3827fd9826abe3b14609aaa2037bc00c (patch) | |
tree | 0d80a2adcfb1cde6981167979d4f1c6fe2840b55 /drivers/i2c/busses/i2c-uniphier.c | |
parent | i2c: uniphier-f: remove printout on handled timeouts (diff) | |
download | linux-1d428f5d3827fd9826abe3b14609aaa2037bc00c.tar.xz linux-1d428f5d3827fd9826abe3b14609aaa2037bc00c.zip |
i2c: uniphier: remove printout on handled timeouts
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>
Diffstat (limited to 'drivers/i2c/busses/i2c-uniphier.c')
-rw-r--r-- | drivers/i2c/busses/i2c-uniphier.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/i2c/busses/i2c-uniphier.c b/drivers/i2c/busses/i2c-uniphier.c index 854ac25b5862..e1b4c80e0285 100644 --- a/drivers/i2c/busses/i2c-uniphier.c +++ b/drivers/i2c/busses/i2c-uniphier.c @@ -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) |