summaryrefslogtreecommitdiff
path: root/payloads/libpayload/drivers/usb/ehci.c
diff options
context:
space:
mode:
authorNico Huber <nico.huber@secunet.com>2012-05-21 14:53:43 +0200
committerStefan Reinauer <stefan.reinauer@coreboot.org>2012-06-08 00:01:25 +0200
commit0421dc84dfba3b4eaa045b595320d6468b3206ef (patch)
treefad8e07e2a023cc9aacf3a1bd81a25191b26e282 /payloads/libpayload/drivers/usb/ehci.c
parentafe86c0b74cc7f5dcaefc34155daf299e8377353 (diff)
downloadcoreboot-0421dc84dfba3b4eaa045b595320d6468b3206ef.tar.xz
libpayload: Add timeouts in the EHCI USB driver
We should always have some timeout when we wait for the hardware. This adds missing timeouts to the EHCI driver. Change-Id: I13ba532a6daf47510b16b8fdbe572a21f1d8b09c Signed-off-by: Nico Huber <nico.huber@secunet.com> Reviewed-on: http://review.coreboot.org/1077 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'payloads/libpayload/drivers/usb/ehci.c')
-rw-r--r--payloads/libpayload/drivers/usb/ehci.c90
1 files changed, 64 insertions, 26 deletions
diff --git a/payloads/libpayload/drivers/usb/ehci.c b/payloads/libpayload/drivers/usb/ehci.c
index 7ee97612be..1094c78560 100644
--- a/payloads/libpayload/drivers/usb/ehci.c
+++ b/payloads/libpayload/drivers/usb/ehci.c
@@ -167,7 +167,21 @@ static int wait_for_tds(qtd_t *head)
qtd_t *cur = head;
while (1) {
if (0) dump_td(virt_to_phys(cur));
- while ((cur->token & QTD_ACTIVE) && !(cur->token & QTD_HALTED)) udelay(60);
+
+ /* wait for results */
+ /* TOTEST: how long to wait?
+ * tested with some USB2.0 flash sticks:
+ * slowest took around 180ms
+ */
+ int timeout = 10000; /* time out after 10000 * 50us == 500ms */
+ while ((cur->token & QTD_ACTIVE) && !(cur->token & QTD_HALTED)
+ && timeout--)
+ udelay(50);
+ if (timeout < 0) {
+ printf("Error: ehci: queue transfer "
+ "processing timed out.\n");
+ return 1;
+ }
if (cur->token & QTD_HALTED) {
printf("ERROR with packet\n");
dump_td(virt_to_phys(cur));
@@ -185,6 +199,51 @@ static int wait_for_tds(qtd_t *head)
return result;
}
+static int ehci_set_async_schedule(ehci_t *ehcic, int enable)
+{
+ /* Set async schedule status. */
+ if (enable)
+ ehcic->operation->usbcmd |= HC_OP_ASYNC_SCHED_EN;
+ else
+ ehcic->operation->usbcmd &= ~HC_OP_ASYNC_SCHED_EN;
+ /* Wait for the controller to accept async schedule status.
+ * This shouldn't take too long, but we should timeout nevertheless.
+ */
+ enable = enable ? HC_OP_ASYNC_SCHED_STAT : 0;
+ int timeout = 100; /* time out after 100ms */
+ while (((ehcic->operation->usbsts & HC_OP_ASYNC_SCHED_STAT) != enable)
+ && timeout--)
+ mdelay(1);
+ if (timeout < 0) {
+ debug("ehci async schedule status change timed out.\n");
+ return 1;
+ }
+ return 0;
+}
+
+static int ehci_process_async_schedule(
+ ehci_t *ehcic, ehci_qh_t *qhead, qtd_t *head)
+{
+ int result;
+
+ /* make sure async schedule is disabled */
+ if (ehci_set_async_schedule(ehcic, 0)) return 1;
+
+ /* hook up QH */
+ ehcic->operation->asynclistaddr = virt_to_phys(qhead);
+
+ /* start async schedule */
+ if (ehci_set_async_schedule(ehcic, 1)) return 1;
+
+ /* wait for result */
+ result = wait_for_tds(head);
+
+ /* disable async schedule */
+ ehci_set_async_schedule(ehcic, 0);
+
+ return result;
+}
+
static int ehci_bulk (endpoint_t *ep, int size, u8 *data, int finalize)
{
int result = 0;
@@ -239,19 +298,8 @@ static int ehci_bulk (endpoint_t *ep, int size, u8 *data, int finalize)
qh->td.token |= (ep->toggle?QTD_TOGGLE_DATA1:0);
head->token |= (ep->toggle?QTD_TOGGLE_DATA1:0);
- /* hook up QH */
- EHCI_INST(ep->dev->controller)->operation->asynclistaddr = virt_to_phys(qh);
-
- /* start async schedule */
- EHCI_INST(ep->dev->controller)->operation->usbcmd |= HC_OP_ASYNC_SCHED_EN;
- while (!(EHCI_INST(ep->dev->controller)->operation->usbsts & HC_OP_ASYNC_SCHED_STAT)) ; /* wait */
-
- /* wait for result */
- result = wait_for_tds(head);
-
- /* disable async schedule */
- EHCI_INST(ep->dev->controller)->operation->usbcmd &= ~HC_OP_ASYNC_SCHED_EN;
- while (EHCI_INST(ep->dev->controller)->operation->usbsts & HC_OP_ASYNC_SCHED_STAT) ; /* wait */
+ result = ehci_process_async_schedule(
+ EHCI_INST(ep->dev->controller), qh, head);
ep->toggle = (cur->token & QTD_TOGGLE_MASK) >> QTD_TOGGLE_SHIFT;
@@ -338,18 +386,8 @@ static int ehci_control (usbdev_t *dev, direction_t dir, int drlen, void *devreq
(hubaddr << QH_HUB_ADDRESS_SHIFT);
qh->td.next_qtd = virt_to_phys(head);
- /* hook up QH */
- EHCI_INST(dev->controller)->operation->asynclistaddr = virt_to_phys(qh);
-
- /* start async schedule */
- EHCI_INST(dev->controller)->operation->usbcmd |= HC_OP_ASYNC_SCHED_EN;
- while (!(EHCI_INST(dev->controller)->operation->usbsts & HC_OP_ASYNC_SCHED_STAT)) ; /* wait */
-
- result = wait_for_tds(head);
-
- /* disable async schedule */
- EHCI_INST(dev->controller)->operation->usbcmd &= ~HC_OP_ASYNC_SCHED_EN;
- while (EHCI_INST(dev->controller)->operation->usbsts & HC_OP_ASYNC_SCHED_STAT) ; /* wait */
+ result = ehci_process_async_schedule(
+ EHCI_INST(dev->controller), qh, head);
free_qh_and_tds(qh, head);
return result;