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

From: Felipe Balbi
Date: Fri Mar 10 2017 - 06:28:46 EST



Hi,

Jerry Huang <jerry.huang@xxxxxxx> 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

Attachment: signature.asc
Description: PGP signature