There is a if (ret) return -EAGAIN a few lines above, so if we reach the final return, ret must be zero. Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> --- drivers/i2c/busses/i2c-imx-early.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-imx-early.c b/drivers/i2c/busses/i2c-imx-early.c index 472e4be83a70..b381f99f4003 100644 --- a/drivers/i2c/busses/i2c-imx-early.c +++ b/drivers/i2c/busses/i2c-imx-early.c @@ -126,7 +126,7 @@ static int i2c_fsl_start(struct fsl_i2c *fsl_i2c) temp |= I2CR_MTX | I2CR_TXAK; fsl_i2c_write_reg(temp, fsl_i2c, FSL_I2C_I2CR); - return ret; + return 0; } static void i2c_fsl_stop(struct fsl_i2c *fsl_i2c) -- 2.30.2 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox