]> git.dujemihanovic.xyz Git - u-boot.git/commit
usb: ci_udc: set ep->req.actual after transfer
authorStephen Warren <swarren@nvidia.com>
Thu, 24 Apr 2014 23:52:37 +0000 (17:52 -0600)
committerMarek Vasut <marex@denx.de>
Wed, 30 Apr 2014 08:30:57 +0000 (10:30 +0200)
commit8aac6e9c537a2dd0c76acc3a0768ae7af5e166ab
treeb79b50c76f6b05ac0621782677f0bbf65146fb93
parentf5c03006dd1794362a2be7136fed9e664bc1775b
usb: ci_udc: set ep->req.actual after transfer

At least drivers/usb/gadget/storage_common.c expects that ep->req.actual
contain the number of bytes actually transferred. (At least in practice,
I observed it failing to work correctly unless this was the case).

However, ci_udc.c modifies ep->req.length instead. I assume that .length
 is supposed to represent the allocated buffer size, whereas .actual is
supposed to represent the actual number of bytes transferred. In the OUT
transaction case, this may happen simply because the host sends a smaller
 packet than the max possible size, which is quite legal. In the IN case,
transferring fewer bytes than requested could presumably happen as an
error.

Modify handle_ep_complete() to write to .actual rather than modifying
.length.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
drivers/usb/gadget/ci_udc.c