summaryrefslogtreecommitdiffstats
path: root/drivers/mmc/host/dw_mmc.c
diff options
context:
space:
mode:
authorDoug Anderson <dianders@chromium.org>2015-04-03 11:13:05 -0700
committerUlf Hansson <ulf.hansson@linaro.org>2015-04-09 09:08:21 +0200
commit8886a6fd19775eb43b7834850d0ad3f749b70053 (patch)
tree01034613de00d968848872092b05824d73e3f8f2 /drivers/mmc/host/dw_mmc.c
parent76184ac17edf3c640390b0eddc3aa7be1095fb9f (diff)
downloadop-kernel-dev-8886a6fd19775eb43b7834850d0ad3f749b70053.zip
op-kernel-dev-8886a6fd19775eb43b7834850d0ad3f749b70053.tar.gz
mmc: dw_mmc: Increase cmd11 timeout to 500ms
Although the cmd11 interrupt should come within 2ms, that's a very short time. Let's increase the timeout to be really sure that we don't get an accidnetal timeout. One case in particular this is useful is if you've got a serial console and printk in just the right places. Under that scenario I've seen delays of up to 130ms before the interrupt fired. CMD11 is only sent during card insertion, so this extra timeout shouldn't be terrible. Fixes: 5c935165da79 ("mmc: dw_mmc: Add a timeout for sending CMD11") Signed-off-by: Doug Anderson <dianders@chromium.org> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Diffstat (limited to 'drivers/mmc/host/dw_mmc.c')
-rw-r--r--drivers/mmc/host/dw_mmc.c8
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
index 8ce9a52..b613845 100644
--- a/drivers/mmc/host/dw_mmc.c
+++ b/drivers/mmc/host/dw_mmc.c
@@ -1024,11 +1024,13 @@ static void __dw_mci_start_request(struct dw_mci *host,
if (cmd->opcode == SD_SWITCH_VOLTAGE) {
/*
- * Databook says to fail after 2ms w/ no response; give an
- * extra jiffy just in case we're about to roll over.
+ * Databook says to fail after 2ms w/ no response, but evidence
+ * shows that sometimes the cmd11 interrupt takes over 130ms.
+ * We'll set to 500ms, plus an extra jiffy just in case jiffies
+ * is just about to roll over.
*/
mod_timer(&host->cmd11_timer,
- jiffies + msecs_to_jiffies(2) + 1);
+ jiffies + msecs_to_jiffies(500) + 1);
}
if (mrq->stop)
OpenPOWER on IntegriCloud