[lkml]   [2014]   [Jan]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRE: [PATCH 3.12 033/118] usb: xhci: Link TRB must not occur within a USB payload burst [NEW HARDWARE]
> On 01/17/2014 06:34 AM, David Laight wrote:
> > Can you try the patch I posted that stops the ownership on LINK TRBs
> > being changed before that on the linked-to TRB?
> Sadly, the patch didn't fix the ASMedia lockup behavior, however :(
> I did notice that the lockup occurred only when copying *to* the usb3
> drive, and not when copying from it. I think that may be new behavior
> but I can't swear to it.

If the behaviour has changed then the fix is on the right lines.
If reads used to lock up, and don't with the patch then that is
an improvement.
It might be that the tx issue is actually a different 'bug'.

> Just to confirm, here are the first few lines of the patch I used.
> Please let me know if it's the wrong patch:
> ...
> +
> + field4 = (field4 & ~TRB_CYCLE) | ring->cycle_state;
> + if (trb == &ring->enqueue_first->generic)
> + field4 ^= TRB_CYCLE;
> +
> trb->field[0] = cpu_to_le32(field1);

That looks like my earlier 'test' patch.
The fuller patch is

There shouldn't be any difference in the way the chip is driven, the
later patch just rips out a load of code that is no longer needed.
Mostly it simplifies the way the ownership of ring entries is passed
to the hardware.


 \ /
  Last update: 2014-01-20 12:01    [W:0.207 / U:20.464 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site