Re: [PATCH v5 06/21] usb: dwc2: host: Always add to the tail of queues

2016-01-27 Thread Kever Yang
Hi Doug, This is obviously a bug in dwc2 driver which not meet the usb 2.0 spec, and this patch can fix it. Reviewed-by: Kever Yang Thanks, - Kever On 01/23/2016 02:18 AM, Douglas Anderson wrote: The queues the the dwc2 host controller used are truly queues. That means FIFO or first in

Re: [PATCH v5 06/21] usb: dwc2: host: Always add to the tail of queues

2016-01-27 Thread Kever Yang
Hi Doug, This is obviously a bug in dwc2 driver which not meet the usb 2.0 spec, and this patch can fix it. Reviewed-by: Kever Yang Thanks, - Kever On 01/23/2016 02:18 AM, Douglas Anderson wrote: The queues the the dwc2 host controller used are truly queues. That

[PATCH v5 06/21] usb: dwc2: host: Always add to the tail of queues

2016-01-22 Thread Douglas Anderson
The queues the the dwc2 host controller used are truly queues. That means FIFO or first in first out. Unfortunately though the code was iterating through these queues starting from the head, some places in the code was adding things to the queue by adding at the head instead of the tail. That

[PATCH v5 06/21] usb: dwc2: host: Always add to the tail of queues

2016-01-22 Thread Douglas Anderson
The queues the the dwc2 host controller used are truly queues. That means FIFO or first in first out. Unfortunately though the code was iterating through these queues starting from the head, some places in the code was adding things to the queue by adding at the head instead of the tail. That