Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1543405

RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

Jerry Huang <jerry.huang@nxp.com> writes:
>> there's no need for that. This patch is in good format. I do have a question,
>> however: how do you know this will work for all users? Burst size is a function
>> of how wide the interconnect where dwc3 is attached to, is.
> So I need to generate one new property in usb node to identify my platform?

Well, we probably need a property to be passed, yes. But let's go
through it all first :-)

>> You could very well be degrading performance for some users here. Can you
>> send me the result of the following commands *without* this patch applied?
>> 
>> # mkdir -p /d
>> # mount -t debugfs none /d
>> # cat /d/*dwc3*/regdump
>> 
> Below is the regdump:
> root@ls1043ardb:/d/3000000.usb3# cat regdump
> GSBUSCFG0 = 0x00100080

so you already have INCR256 here. There's one note in the databook which
just caught my attention. It states the following:

	"Undefined burst length has priority over all other burst lenghts."

This means that setting both INCR16 and undefined INCR is
unnecessary. Only Undefined INCR will be taken into consideration. Can
you check with your HW engineers what's the largest burst the
interconnect is supposed to support?

> GSBUSCFG1 = 0x00000700

8 AXI pipelined requests

> GSNPSID = 0x5533280a

2.80a cool :-)

I'll check these settings on my platform as well and see if there's any
setting which would improve transfer speed. This is a very good idea,
btw, but we need to be careful about how to play with it.

-- 
balbi

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Jerry Huang <jerry.huang@nxp.com> - 2016-12-16 06:50 +0100
  RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Felipe Balbi <balbi@kernel.org> - 2016-12-16 10:20 +0100
    RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Jerry Huang <jerry.huang@nxp.com> - 2016-12-16 11:40 +0100
      RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Felipe Balbi <balbi@kernel.org> - 2016-12-16 12:50 +0100
        RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Jerry Huang <jerry.huang@nxp.com> - 2016-12-16 17:20 +0100
          RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Felipe Balbi <balbi@kernel.org> - 2016-12-16 18:10 +0100
            RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Jerry Huang <jerry.huang@nxp.com> - 2016-12-19 10:20 +0100
              RE: [PATCH] USB3/DWC3: Enable undefined length INCR burst type Felipe Balbi <balbi@kernel.org> - 2016-12-19 10:30 +0100

csiph-web