i2c-nomadik driver prints error message on every faulted message. This is not a good practice, because in I2C a fault not always is an error, sometimes it is the expected result. For example, scanning bus with `i2cdetects` prints over 100 messages in dmesg (two messages per each target address).
To avoid excessive prints in the log, change the print level from err to debug. Signed-off-by: Dmitry Guzman <[email protected]> --- drivers/i2c/busses/i2c-nomadik.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/i2c/busses/i2c-nomadik.c b/drivers/i2c/busses/i2c-nomadik.c index e19ace904e79cd2d83171d9f38fc103a6c5e023b..7d93fb3876dc1324003dd19884e3fd5cbba9cfbb 100644 --- a/drivers/i2c/busses/i2c-nomadik.c +++ b/drivers/i2c/busses/i2c-nomadik.c @@ -627,7 +627,7 @@ static int write_i2c(struct nmk_i2c_dev *priv, u16 flags) if (!xfer_done) { /* Controller timed out */ - dev_err(&priv->adev->dev, "write to slave 0x%x timed out\n", + dev_dbg(&priv->adev->dev, "write to slave 0x%x timed out\n", priv->cli.slave_adr); status = -ETIMEDOUT; } @@ -661,7 +661,7 @@ static int nmk_i2c_xfer_one(struct nmk_i2c_dev *priv, u16 flags) i2c_sr = readl(priv->virtbase + I2C_SR); if (FIELD_GET(I2C_SR_STATUS, i2c_sr) == I2C_ABORT) { cause = FIELD_GET(I2C_SR_CAUSE, i2c_sr); - dev_err(&priv->adev->dev, "%s\n", + dev_dbg(&priv->adev->dev, "%s\n", cause >= ARRAY_SIZE(abort_causes) ? "unknown reason" : abort_causes[cause]); -- 2.43.0
