ehci-hcd: program asynclistaddr before every transfer
authorIlya Yanok <ilya.yanok@cogentembedded.com>
Sun, 15 Jul 2012 22:12:08 +0000 (22:12 +0000)
committerMarek Vasut <marex@denx.de>
Wed, 18 Jul 2012 12:43:29 +0000 (14:43 +0200)
Move or_asynclistaddr programming to ehci_submit_async()
function to make sure queue head is properly programmed
before every transfer. This solves the problem with changing
qh address.

Also remove unneeded qh_list->qh_link reprogramming at the
end of transfer.

Signed-off-by: Ilya Yanok <ilya.yanok@cogentembedded.com>
drivers/usb/host/ehci-hcd.c

index 04300be1100acb0c7bf23d4d744d0f13d5ed16b3..3be2a770bcb2d700418acc199a1ecaa39754993e 100644 (file)
@@ -348,6 +348,9 @@ ehci_submit_async(struct usb_device *dev, unsigned long pipe, void *buffer,
        flush_dcache_range((uint32_t)&qh, (uint32_t)&qh + sizeof(struct QH));
        flush_dcache_range((uint32_t)qtd, (uint32_t)qtd + sizeof(qtd));
 
+       /* Set async. queue head pointer. */
+       ehci_writel(&hcor->or_asynclistaddr, (uint32_t)&qh_list);
+
        usbsts = ehci_readl(&hcor->or_usbsts);
        ehci_writel(&hcor->or_usbsts, (usbsts & 0x3f));
 
@@ -403,8 +406,6 @@ ehci_submit_async(struct usb_device *dev, unsigned long pipe, void *buffer,
                goto fail;
        }
 
-       qh_list.qh_link = cpu_to_hc32((uint32_t)&qh_list | QH_LINK_TYPE_QH);
-
        token = hc32_to_cpu(qh.qh_overlay.qt_token);
        if (!(token & 0x80)) {
                debug("TOKEN=%#x\n", token);
@@ -741,9 +742,6 @@ int usb_lowlevel_init(void)
        qh_list.qh_overlay.qt_altnext = cpu_to_hc32(QT_NEXT_TERMINATE);
        qh_list.qh_overlay.qt_token = cpu_to_hc32(0x40);
 
-       /* Set async. queue head pointer. */
-       ehci_writel(&hcor->or_asynclistaddr, (uint32_t)&qh_list);
-
        reg = ehci_readl(&hccr->cr_hcsparams);
        descriptor.hub.bNbrPorts = HCS_N_PORTS(reg);
        printf("Register %x NbrPorts %d\n", reg, descriptor.hub.bNbrPorts);