]> git.dujemihanovic.xyz Git - u-boot.git/commit
usb: gadget: fastboot: Dequeue the previous IN request for the current request
authorPaul Kocialkowski <contact@paulk.fr>
Sat, 4 Jul 2015 14:46:16 +0000 (16:46 +0200)
committerMarek Vasut <marex@denx.de>
Wed, 22 Jul 2015 06:57:54 +0000 (08:57 +0200)
commitbc9071c9f318aa69c815927b3828096d1976de1b
tree0af1c0a0d99248db55604028fb0a3699f6c9274e
parent94b385fa236c85b00ee0bceced05aad2550c2208
usb: gadget: fastboot: Dequeue the previous IN request for the current request

Recent versions of the fastboot tool will query the partition type before doing
an operation on a partition (such as erase, flash, etc). It will then submit
the operation as soon as the response for the partition type is received.

Usually, the MUSB controller will see that the partition type request return
status was read by the host at the very same time as the actual operation
request is submitted by the host. However, the operation will be read first
(int_rx is handled first in musb_interrupt) and after it is completed, the
fastboot USB gadget driver will send another return status. Hence, this happens
before the musb gadget framework has had a chance to handle the previous
acknowledgement that the host read the return status and dequeue the request.

The host will then usually empty the FIFO by the time musb_interrupt gets around
handling the return status acknowledgement (for the previous request, this is
still on the same musb_interrupt call), so no other interrupt is generated and
the most recent return status acknowledgement remains unaccounted for.

It will then be used as a response for the next command, and the proper response
for it will be delayed to the next command, and so on.

Dequeuing the previous IN request in the fastboot code ensures that no previous
return status remains. It is acceptable to do it since there is no callback to
it anyways.

Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
drivers/usb/gadget/f_fastboot.c