RE: [PATCH V6 00/10] usb: dwc3: Fix broken BULK stream support to dwc3 gadget driver

From: Anurag Kumar Vulisha
Date: Sun Nov 11 2018 - 03:48:58 EST


Hi Felipe,

Please let me know if you have any comments on this patch series.
If you think patches are good, can we proceed with them ?

Thanks,
Anurag Kumar Vulisha

>-----Original Message-----
>From: Anurag Kumar Vulisha [mailto:anurag.kumar.vulisha@xxxxxxxxxx]
>Sent: Saturday, October 13, 2018 6:45 PM
>To: Felipe Balbi <balbi@xxxxxxxxxx>; Greg Kroah-Hartman
><gregkh@xxxxxxxxxxxxxxxxxxx>; Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>; Johan
>Hovold <johan@xxxxxxxxxx>; Jaejoong Kim <climbbb.kim@xxxxxxxxx>; Benjamin
>Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>; Roger Quadros <rogerq@xxxxxx>
>Cc: linux-usb@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
>v.anuragkumar@xxxxxxxxx; Thinh Nguyen <thinhn@xxxxxxxxxxxx>; Tejas Joglekar
><tejas.joglekar@xxxxxxxxxxxx>; Ajay Yugalkishore Pandey <APANDEY@xxxxxxxxxx>;
>Anurag Kumar Vulisha <anuragku@xxxxxxxxxx>
>Subject: [PATCH V6 00/10] usb: dwc3: Fix broken BULK stream support to dwc3
>gadget driver
>
>This patch series fixes the broken BULK streaming support in
>dwc3 gadget driver and also adds timer into udc/core.c to avoid deadlock for the
>endpoints which are bulk stream capable.
>
>Changes in v6:
> 1. Added timer into udc/core.c for stream capable endpoint
> as suggested by "Felipe Balbi"
>
>Changes in v5:
> 1. Removed the dev_dbg prints as suggested bt "Thinh Nguyen"
>
>Changes in v4:
> 1. Corrected the commit message and stream timeout description
> as suggested by "Thinh Nguyen"
>
>Changes in v3:
> 1. Added the changes suggested by "Thinh Nguyen"
>
>Changes in v2:
> 1. Added "usb: dwc3:" in subject heading
>
>Anurag Kumar Vulisha (10):
> usb: gadget: udc: Add timer for stream capable endpoints
> usb: dwc3: gadget: Add stream timeout handler for avoiding deadlock
> usb: dwc3: gadget: Remove references to dep->stream_capable
> usb: dwc3: update stream id in depcmd
> usb: dwc3: make controller clear transfer resources after complete
> usb: dwc3: don't issue no-op trb for stream capable endpoints
> usb: dwc3: check for requests in started list for stream capable
> endpoints
> usb: dwc3: Correct the logic for checking TRB full in
> __dwc3_prepare_one_trb()
> usb: dwc3: Check for IOC/LST bit in both event->status and TRB->ctrl
> fields
> usb: dwc3: Check MISSED ISOC bit only for ISOC endpoints
>
> drivers/usb/dwc3/core.h | 2 --
> drivers/usb/dwc3/gadget.c | 67 +++++++++++++++++++++++++++++++++-------
> drivers/usb/gadget/udc/core.c | 71
>++++++++++++++++++++++++++++++++++++++++++-
> include/linux/usb/gadget.h | 12 ++++++++
> 4 files changed, 138 insertions(+), 14 deletions(-)
>
>--
>2.1.1