Hi,
On Tue, Aug 19, 2014 at 09:12:29PM +0530, Kishon Vijay Abraham I wrote:
@@ -987,7 +987,6 @@ static int composite_bind(struct usb_gadget *gadget)
memcpy(&cdev->desc, composite->dev, sizeof(struct usb_device_descriptor));
- cdev->desc.bMaxPacketSize0 = gadget->ep0->maxpacket;
why are you dropping this line ?
Actually faced an issue with this. DWC3 gadget sets ep0 maxpacket size to 512 and if the device is operating at hight speed, the bMaxPacketSize0 field should be 64?
right, but that's changed on connection done IRQ.
If the maxpacket size is set to 512, the host reports Invalid EP0 Maxpacket.
huh ? who broke dwc3 ? connection done happens before we receive the first setup packet, how can host complain about anything ?
composite_bind() (where maxpacket is being set) happens much before connection done interrupt. So it's not dwc3's fault.
seems like you're just missing a few fixes to composite.c, such as commit 765f5b830e547229bb752e7b232ee83e2b3d49d5