drivers: mediatek: Fix error handling in mtk_i2c_do_transfer

Errors were handled only if an I2C transfer timed out
and received a NACK which is very unlikely. This commit
changes the condition such that errors are handled if
an I2C transfer times out or received a NACK.

Signed-off-by: Francois Berder <fberder@outlook.fr>
Reviewed-by: Heiko Schocher <hs@denx.de>

diff --git a/drivers/i2c/mtk_i2c.c b/drivers/i2c/mtk_i2c.c
index 2f331d3..5592fe9 100644 (file)
--- a/drivers/i2c/mtk_i2c.c
+++ b/drivers/i2c/mtk_i2c.c
@@ -621,7 +621,7 @@ static int mtk_i2c_do_transfer(struct mtk_i2c_priv *priv,
        i2c_writel(priv, REG_INTR_MASK, ~(restart_flag | I2C_HS_NACKERR |
                  I2C_ACKERR | I2C_TRANSAC_COMP));
 
-       if (!tmo && trans_error != 0) {
+       if (tmo || trans_error != 0) {
                if (tmo) {
                        ret = -ETIMEDOUT;
                        if (!priv->filter_msg)