On Tue, Sep 27, 2016 at 11:02:54PM +0800, yfw wrote: > > > On 2016/9/27 23:00, Bin Liu wrote: > >On Tue, Sep 27, 2016 at 10:53:54PM +0800, yfw wrote: > >>Hi, > >> > >>On 2016/9/27 22:28, Bin Liu wrote: > >>>On Tue, Sep 27, 2016 at 09:20:56AM -0500, Bin Liu wrote: > >>>>Felipe, > >>>> > >>>>On Tue, Sep 27, 2016 at 10:18:26AM +0300, Felipe Balbi wrote: > >>>> > >>>>[snip] > >>>> > >>>>>>>>Does this mean the issue of isoc high bandwidth transfer was fixed by > >>>>>>>>this patchset per your test? > >>>>>>> > >>>>>>>No, I couldn't get g_webcam to work yet. > >>>>>> > >>>>>>In mainline, g_webcam is broken with DWC3. Also these two patches don't > >>>>>>fix the issue on v4.4.21. > >>>>> > >>>>>care to send tracepoint output? Best if you could cherry pick my latest > >>>>>tracepoint changes so we have the best output possible. > >>>>> > >>>>>I have also built a branch with v4.4.21 + all dwc3 patches (except for > >>>>>device properties and PCI stuff) which you could use for testing. If you > >>>>>run with that, then I can get proper trace output and I can try to > >>>>>figure out what's missing. > >>>>> > >>>>>Branch is here: > >>>>> > >>>>> git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git usb/v4.4.21+dwc3 > >>>> > >>>>This branch does not generate the isoch traffic (on ep2in). ftrace > >>>>attached (dwc3-g_webcam-v4.4.21+tp.ftrace). > >>>> > >>>>I also attached the ftrace log for v4.4.21 tag > >>>>(dwc3-g_webcam-v4.4.21.ftrace) for comparison, which has ep2in isoch > >>>>traffic, but only one transaction per SOF. > >>> > >>>Sorry, I didn't realize the ftrace file size is huge. Attached the > >>>tarball here. > >>>(dwc3-g_webcam-ftrace.tgz) > >>> > >>>Regards, > >>>-Bin. > >>> > >>From the ftrace log: > >> > >>>irq/441-dwc3-769 [001] d..2 101.207679: dwc3_gadget: ep2in-isoc: req > >>>ed971b00 dma ad9d2000 length 5120 > >>5120 looks wrong to me. 5120 == 0x1400. There is an issue in uvc_video which > >>set the req_size according maxpacket (which is 0x1400 for high bandwidth isoc). > What about this? Yeah, seems not right to me too. The UDC would only have 1024 bytes to transmit. > > >> > >>>irq/441-dwc3-769 [001] d..2 101.207682: dwc3_prepare_trb: ep2in-isoc: > >>>trb f2037020 bph 00000000 bpl ad9d2000 size 00001400 ctrl 00000c69 > >> > >>Looks like the "bits 12:11 of wMaxPacketSize" patch from Felipe was not applied. > > > >No, I didn't. I was trying to give a trace for the base line to start > >with. > Get it. Waiting for your further testing result. I am waiting for Felipe's next instruction - going on v4.4.21 tag or his v4.4.21+dwc3 HEAD. Regards, -Bin. > > Regards > Yin, Fengwei > > > > >Regards, > >-Bin. > > > >> > >>>irq/441-dwc3-769 [001] d..2 101.208357: dwc3_complete_trb: ep2in-isoc: > >>>trb f2037020 bph 00000000 bpl ad9d2000 size 10001400 ctrl 1bfd0c68 > >> > >>MissedIsoc happened. > >> > >> > >>Regards > >>Yin, Fengwei > >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
