From: Marek Vasut Date: Sat, 14 Oct 2023 21:56:03 +0000 (+0200) Subject: mmc: tmio: Always check for errors after receiving an IRQ X-Git-Url: http://git.dujemihanovic.xyz/img/%7B%7B%20%24style.RelPermalink%20%7D%7D?a=commitdiff_plain;h=631dbe0c084f7459c4caa398470d4004152c2333;p=u-boot.git mmc: tmio: Always check for errors after receiving an IRQ Unconditionally check for errors even after successful reception of IRQ flag, since the hardware may set both an IRQ completion flag and an error flag at the same time. This mode of failure happens in case of an error during transfer, in which case the hardware may set the expected IRQ completion flag as well as error flags. The later is currently not checked by the driver and such an error is not detected. Improve the error detection. Signed-off-by: Marek Vasut Reviewed-by: Yoshihiro Shimoda Tested-by: Thuan Nguyen Hong --- diff --git a/drivers/mmc/tmio-common.c b/drivers/mmc/tmio-common.c index d1e2681599..83eaf361f0 100644 --- a/drivers/mmc/tmio-common.c +++ b/drivers/mmc/tmio-common.c @@ -122,7 +122,10 @@ static int tmio_sd_wait_for_irq(struct udevice *dev, struct mmc_cmd *cmd, long wait = 1000000; int ret; - while (!(tmio_sd_readl(priv, reg) & flag)) { + while (true) { + if (tmio_sd_readl(priv, reg) & flag) + return tmio_sd_check_error(dev, cmd); + if (wait-- < 0) { dev_err(dev, "timeout\n"); return -ETIMEDOUT;