According to the USB 2.0 Spec CH5.5.3 and CH8.5.3.2, only control
read (IN) needs zero length packet.

Cc: Alan Stern <st...@rowland.harvard.edu>
Signed-off-by: Peter Chen <peter.c...@freescale.com>
---
 drivers/usb/gadget/composite.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c
index 1c0c3da..c3898b5 100644
--- a/drivers/usb/gadget/composite.c
+++ b/drivers/usb/gadget/composite.c
@@ -1744,7 +1744,8 @@ unknown:
                        }
                        req->length = value;
                        req->context = cdev;
-                       req->zero = value < w_length;
+                       if (ctrl->bRequestType == USB_DIR_IN)
+                               req->zero = value < w_length;
                        value = composite_ep0_queue(cdev, req, GFP_ATOMIC);
                        if (value < 0) {
                                DBG(cdev, "ep_queue --> %d\n", value);
@@ -1820,7 +1821,8 @@ try_fun_setup:
        if (value >= 0 && value != USB_GADGET_DELAYED_STATUS) {
                req->length = value;
                req->context = cdev;
-               req->zero = value < w_length;
+               if (ctrl->bRequestType == USB_DIR_IN)
+                       req->zero = value < w_length;
                value = composite_ep0_queue(cdev, req, GFP_ATOMIC);
                if (value < 0) {
                        DBG(cdev, "ep_queue --> %d\n", value);
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to