Re: [PATCH 1/4] usb: dwc3: gadget: drop useless code

[Felipe Balbi <balbi@xxxxxx>]

On Wed, Jun 06, 2012 at 01:04:00PM +0530, Pratyush Anand wrote:
> On 6/6/2012 12:37 PM, Felipe Balbi wrote:
> >Hi Pratyush,
> >
> >On Wed, Jun 06, 2012 at 10:04:34AM +0300, Felipe Balbi wrote:
> >>We never set CMDIOC bit for Start Transfer
> >>command, so that code will never be used.
> >>
> >>Signed-off-by: Felipe Balbi<balbi@xxxxxx>
> >
> >Just FYI, I tested this series with g_mass_storage and g_zero by
> >removing a cable with a transfer going on and later reconnecting. I
> >didn't see any crashes while doing that.
> >
> >I still got one warning, but that's because I'm running 1.83a which has
> >a bug of missing disconnect IRQs which is triggering, so nevermind that
> >WARN.
> >
> >Let me know how testing goes on your side.
> >
> 
> Its almost working :)
> 
> Only one warning now I see from  dwc3_endpoint_interrupt
> 
> Transfer In-Progress while ep2in not enabled.

That's the one I added on purpose so we could track this case. Do you
think I should just drop it ?

-- 
balbi

Attachment: signature.asc
Description: Digital signature