Re: [PATCH 1/2] usb: add helper to extract bits 12:11 of wMaxPacketSize

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Bin Liu <b-liu@xxxxxx> writes:
>> > >> > 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.

looking at webcam.c, it'll set wMaxPacket correctly (to 0x1400) if you
pass streaming_maxpacket=3072. So that's one thing.

I'll look at this log with more detail tomorrow, though.

-- 
balbi
--
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



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux