summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorMarek Vasut <marex@denx.de>2015-07-27 22:39:38 +0200
committerPantelis Antoniou <pantelis.antoniou@konsulko.com>2015-08-11 20:44:30 +0300
commit9042d974d2391b58d9a51f805756fac31f3c656f (patch)
tree1bf55f4877d587863bd30e1e56a8bbea6736e096 /drivers
parentd9dbb97be0e4a550457aec5f11afefb446169c90 (diff)
downloadu-boot-imx-9042d974d2391b58d9a51f805756fac31f3c656f.zip
u-boot-imx-9042d974d2391b58d9a51f805756fac31f3c656f.tar.gz
u-boot-imx-9042d974d2391b58d9a51f805756fac31f3c656f.tar.bz2
mmc: dw_mmc: Improve handling of data transfer failure
In case the data transfer failure happens, instead of returning immediatelly, make sure the DMA is disabled, status register is cleared and the bounce buffer is stopped. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Dinh Nguyen <dinguyen@opensource.altera.com> Cc: Pantelis Antoniou <panto@antoniou-consulting.com> Cc: Tom Rini <trini@konsulko.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/mmc/dw_mmc.c16
1 files changed, 9 insertions, 7 deletions
diff --git a/drivers/mmc/dw_mmc.c b/drivers/mmc/dw_mmc.c
index 0f61f16..fcd5784 100644
--- a/drivers/mmc/dw_mmc.c
+++ b/drivers/mmc/dw_mmc.c
@@ -110,7 +110,7 @@ static int dwmci_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd,
struct dwmci_host *host = mmc->priv;
ALLOC_CACHE_ALIGN_BUFFER(struct dwmci_idmac, cur_idmac,
data ? DIV_ROUND_UP(data->blocks, 8) : 0);
- int flags = 0, i;
+ int ret = 0, flags = 0, i;
unsigned int timeout = 100000;
u32 retry = 10000;
u32 mask, ctrl;
@@ -218,20 +218,22 @@ static int dwmci_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd,
/* Error during data transfer. */
if (mask & (DWMCI_DATA_ERR | DWMCI_DATA_TOUT)) {
printf("%s: DATA ERROR!\n", __func__);
- bounce_buffer_stop(&bbstate);
- return -1;
+ ret = -EINVAL;
+ break;
}
/* Data arrived correctly. */
- if (mask & DWMCI_INTMSK_DTO)
+ if (mask & DWMCI_INTMSK_DTO) {
+ ret = 0;
break;
+ }
/* Check for timeout. */
if (get_timer(start) > timeout) {
printf("%s: Timeout waiting for data!\n",
__func__);
- bounce_buffer_stop(&bbstate);
- return TIMEOUT;
+ ret = TIMEOUT;
+ break;
}
}
@@ -246,7 +248,7 @@ static int dwmci_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd,
udelay(100);
- return 0;
+ return ret;
}
static int dwmci_setup_bus(struct dwmci_host *host, u32 freq)