]> git.karo-electronics.de Git - karo-tx-linux.git/commit
usb: gadget: fsl_udc_core: dTD's next dtd pointer need to be updated once written
authorPeter Chen <peter.chen@freescale.com>
Sun, 1 Apr 2012 07:17:16 +0000 (15:17 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 1 Jun 2012 07:15:48 +0000 (15:15 +0800)
commit6a7ecec6315dfdeac08d163146b0260735e95f39
treed1a643ae9d40c2ce0b665782eee9c3cf6904209b
parent4c03c1245df37a3a64ae6aeed2bdec8780e879a6
usb: gadget: fsl_udc_core: dTD's next dtd pointer need to be updated once written

commit 4d0947dec4db1224354e2f6f00ae22ce38e62a43 upstream.

dTD's next dtd pointer need to be updated once CPU writes it, or this
request may not be handled by controller, then host will get NAK from
device forever.

This problem occurs when there is a request is handling, we need to add
a new request to dTD list, if this new request is added before the current
one is finished, the new request is intended to added as next dtd pointer
at current dTD, but without wmb(), the dTD's next dtd pointer may not be
updated when the controller reads it. In that case, the controller will
still get Terminate Bit is 1 at dTD's next dtd pointer, that means there is
no next request, then this new request is missed by controller.

Signed-off-by: Peter Chen <peter.chen@freescale.com>
Acked-by: Li Yang <leoli@freescale.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/gadget/fsl_udc_core.c