diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2023-08-24 15:59:39 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2023-08-28 10:01:49 +0200 |
commit | 29e8e1cefcdff3a68d4e4531ae31ed9b6e93b0c7 (patch) | |
tree | b0d9f4ce594537249d06e60f937f439c7fbd460e /drivers/mci/sdhci.c | |
parent | 0fa2d4866808761416d53083779149017f4a53de (diff) | |
download | barebox-29e8e1cefcdff3a68d4e4531ae31ed9b6e93b0c7.tar.gz barebox-29e8e1cefcdff3a68d4e4531ae31ed9b6e93b0c7.tar.xz |
mci: sdhci: don't hang indefinitely waiting for DMA completion
barebox was observed to hang on an i.MX8MP while reading ext_csd from
a broken eMMC that even bootROM refused to boot from. For PIO, we
already have a maximum wait time of 10s. Do the same for DMA.
It's unlikely that in that state any further communication with the card
will work, but at least, the system isn't stuck and the shell can be
reached. The reset is done on the off-chance that the DMA would've completed
after the 10s and corrupted memory if not disabled.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
Link: https://lore.barebox.org/20230824135939.1614716-1-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/mci/sdhci.c')
-rw-r--r-- | drivers/mci/sdhci.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/drivers/mci/sdhci.c b/drivers/mci/sdhci.c index b0b83bfaa9..9829a78cb6 100644 --- a/drivers/mci/sdhci.c +++ b/drivers/mci/sdhci.c @@ -252,6 +252,7 @@ int sdhci_transfer_data_dma(struct sdhci *sdhci, struct mci_data *data, dma_addr_t dma) { struct device *dev = sdhci->mci->hw_dev; + u64 start; int nbytes; u32 irqstat; int ret; @@ -261,6 +262,8 @@ int sdhci_transfer_data_dma(struct sdhci *sdhci, struct mci_data *data, nbytes = data->blocks * data->blocksize; + start = get_time_ns(); + do { irqstat = sdhci_read32(sdhci, SDHCI_INT_STATUS); @@ -304,6 +307,13 @@ int sdhci_transfer_data_dma(struct sdhci *sdhci, struct mci_data *data, if (irqstat & SDHCI_INT_XFER_COMPLETE) break; + + if (is_timeout(start, 10 * SECOND)) { + dev_alert(dev, "DMA wait timed out. Resetting, but recovery unlikely\n"); + sdhci_reset(sdhci, SDHCI_RESET_ALL); + ret = -ETIMEDOUT; + goto out; + } } while (1); ret = 0; |