[U-Boot] [U-Boot, 2/2] usb: gadget: fastboot: Dequeue the previous IN request for the current request

[Eugeniu Rosca]

-bouncing e-mails
cc: Łukasz

On Mon, Mar 18, 2019 at 06:15:48PM +0100, Eugeniu Rosca wrote:
> Hi Paul, hello Marek,
> 
> On Mon, Mar 18, 2019 at 11:12:02AM +0100, Paul Kocialkowski wrote:
> > Hi,
> > 
> > Le lundi 18 mars 2019 à 10:56 +0100, Eugeniu Rosca a écrit :
> > > Hi Marek, Paul, cc: Alex
> > > 
> > > jFYI/FWIW, reverting [1-2] allows getting rid of below warnings on
> > > R-Car3, when running basic fastboot commands (e.g. fastboot getvar):
> > > 
> > > [    8.035764] status: -104 ep 'ep1' trans: 0
> > > [   18.744354] status: -104 ep 'ep1' trans: 28
> > > [   18.748950] status: -104 ep 'ep1' trans: 9
> > > [   18.753370] status: -104 ep 'ep1' trans: 28
> > > [   26.064761] status: -104 ep 'ep1' trans: 28
> > 
> > I don't think reverting this patch in mainline U-Boot would be a good
> > idea, as it is required to get fastboot to work at all on sunxi
> > platforms with the MUSB controller.
> > 
> > Did you investigate the issue to see what is happening here precisely?
> 
> I attempted to record some traces to visualize the functions called
> during my use-case, but it seems like lib/trace.c and cmd/trace.c are
> not usable on arm64 (no U-Boot console output on my H3-ULCB-KF). Anyone
> with a hint in this direction?
> 
> Anyway, given that I use an out-of-tree USB gadget driver, any logs I
> can share will only be of limited/little help, I assume. I expect the
> discussion to make more sense during upstreaming of USBHS R-Car Linux
> driver. Now, it is more of a heads up.
> 
> Best regards,
> Eugeniu.