Commit e181dce8 authored by Marc Pignat's avatar Marc Pignat Committed by Pierre Ossman

mmc: at91_mci: show timeouts

Detect command timeout (or mci controller hangs).
Signed-off-by: default avatarMarc Pignat <marc.pignat@hevs.ch>
Signed-off-by: default avatarHans J Koch <hjk@linutronix.de>
Signed-off-by: default avatarNicolas Ferre <nicolas.ferre@atmel.com>
Signed-off-by: default avatarPierre Ossman <drzeus@drzeus.cx>
parent 80f92546
...@@ -125,8 +125,33 @@ struct at91mci_host ...@@ -125,8 +125,33 @@ struct at91mci_host
/* Latest in the scatterlist that has been enabled for transfer */ /* Latest in the scatterlist that has been enabled for transfer */
int transfer_index; int transfer_index;
/* Timer for timeouts */
struct timer_list timer;
}; };
static void at91_timeout_timer(unsigned long data)
{
struct at91mci_host *host;
host = (struct at91mci_host *)data;
if (host->request) {
dev_err(host->mmc->parent, "Timeout waiting end of packet\n");
if (host->cmd && host->cmd->data) {
host->cmd->data->error = -ETIMEDOUT;
} else {
if (host->cmd)
host->cmd->error = -ETIMEDOUT;
else
host->request->cmd->error = -ETIMEDOUT;
}
mmc_request_done(host->mmc, host->request);
}
}
/* /*
* Copy from sg to a dma block - used for transfers * Copy from sg to a dma block - used for transfers
*/ */
...@@ -557,9 +582,10 @@ static void at91_mci_process_next(struct at91mci_host *host) ...@@ -557,9 +582,10 @@ static void at91_mci_process_next(struct at91mci_host *host)
else if ((!(host->flags & FL_SENT_STOP)) && host->request->stop) { else if ((!(host->flags & FL_SENT_STOP)) && host->request->stop) {
host->flags |= FL_SENT_STOP; host->flags |= FL_SENT_STOP;
at91_mci_send_command(host, host->request->stop); at91_mci_send_command(host, host->request->stop);
} } else {
else del_timer(&host->timer);
mmc_request_done(host->mmc, host->request); mmc_request_done(host->mmc, host->request);
}
} }
/* /*
...@@ -618,6 +644,8 @@ static void at91_mci_request(struct mmc_host *mmc, struct mmc_request *mrq) ...@@ -618,6 +644,8 @@ static void at91_mci_request(struct mmc_host *mmc, struct mmc_request *mrq)
host->request = mrq; host->request = mrq;
host->flags = 0; host->flags = 0;
mod_timer(&host->timer, jiffies + HZ);
at91_mci_process_next(host); at91_mci_process_next(host);
} }
...@@ -936,6 +964,8 @@ static int __init at91_mci_probe(struct platform_device *pdev) ...@@ -936,6 +964,8 @@ static int __init at91_mci_probe(struct platform_device *pdev)
mmc_add_host(mmc); mmc_add_host(mmc);
setup_timer(&host->timer, at91_timeout_timer, (unsigned long)host);
/* /*
* monitor card insertion/removal if we can * monitor card insertion/removal if we can
*/ */
...@@ -996,6 +1026,7 @@ static int __exit at91_mci_remove(struct platform_device *pdev) ...@@ -996,6 +1026,7 @@ static int __exit at91_mci_remove(struct platform_device *pdev)
} }
at91_mci_disable(host); at91_mci_disable(host);
del_timer_sync(&host->timer);
mmc_remove_host(mmc); mmc_remove_host(mmc);
free_irq(host->irq, host); free_irq(host->irq, host);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment