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


Groups > linux.kernel > #1597140 > unrolled thread

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

Started byFelipe Balbi <balbi@kernel.org>
First post2017-03-10 12:30 +0100
Last post2017-03-10 12:30 +0100
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  RE: [PATCH v4 3/3] USB3/DWC3: Enable undefined length INCR burst type Felipe Balbi <balbi@kernel.org> - 2017-03-10 12:30 +0100

#1597140 — RE: [PATCH v4 3/3] USB3/DWC3: Enable undefined length INCR burst type

FromFelipe Balbi <balbi@kernel.org>
Date2017-03-10 12:30 +0100
SubjectRE: [PATCH v4 3/3] USB3/DWC3: Enable undefined length INCR burst type
Message-ID<tjqUa-3fF-23@gated-at.bofh.it>

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

Hi,

Jerry Huang <jerry.huang@nxp.com> writes:
>> >> --
>> >> 1.7.9.5
>> > Hi, Balbi and all guys,
>> > Any comment for these patches? Can they be accepted?
>> 
>> Rob had comments which you didn't reply yet. I cannot take this patchset
>> yet ;-)
>> 
> Balbi,
>
> I look into his mail again, which was based v3, and I replied it.
>
> He had different understanding for undefined length burst mode.
>
> It seems he think for this mode, just setting bit[0] (INCRBrstEna) and
> don't need to set other field.
>
> However, according to the DWC USB3.0 controller databook, when it is
> undefined length INCR burst mode, we still need to set one max burst
> type, such as INCR8, which means controller will use any length less
> than or equal to this INCR8.

Rob, do you agree with the patch now?

-- 
balbi

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web