Re: [PATCH v3] usb: dwc3: gadget: handle request->zero

2015-12-22 Thread Felipe Balbi
Hi, John Youn writes: > On 12/22/2015 9:31 AM, Felipe Balbi wrote: >> >> Hi, >> >> John Youn writes: >>> On 12/3/2015 7:18 AM, Felipe Balbi wrote: So far, dwc3 has always missed request->zero handling for every endpoint. Let's implement that so we can handle cases where transfe

Re: [PATCH v3] usb: dwc3: gadget: handle request->zero

2015-12-22 Thread Alan Stern
On Tue, 22 Dec 2015, John Youn wrote: > >> The following fixes it: > >> > >> - if (ret == 0 && request->zero && (request->length % ep->maxpacket > >> == 0)) > >> + if (ret == 0 && request->zero && (request->length % ep->maxpacket > >> == 0) && > >> + (request->length != 0))

Re: [PATCH v3] usb: dwc3: gadget: handle request->zero

2015-12-22 Thread John Youn
On 12/22/2015 9:31 AM, Felipe Balbi wrote: > > Hi, > > John Youn writes: >> On 12/3/2015 7:18 AM, Felipe Balbi wrote: >>> So far, dwc3 has always missed request->zero >>> handling for every endpoint. Let's implement >>> that so we can handle cases where transfer must >>> be finished with a ZLP.

Re: [PATCH v3] usb: dwc3: gadget: handle request->zero

2015-12-22 Thread Felipe Balbi
Hi, John Youn writes: > On 12/3/2015 7:18 AM, Felipe Balbi wrote: >> So far, dwc3 has always missed request->zero >> handling for every endpoint. Let's implement >> that so we can handle cases where transfer must >> be finished with a ZLP. >> >> Note that dwc3 is a little special. Even though >

Re: [PATCH v3] usb: dwc3: gadget: handle request->zero

2015-12-21 Thread John Youn
On 12/3/2015 7:18 AM, Felipe Balbi wrote: > So far, dwc3 has always missed request->zero > handling for every endpoint. Let's implement > that so we can handle cases where transfer must > be finished with a ZLP. > > Note that dwc3 is a little special. Even though > we're dealing with a ZLP, we sti