[PATCH 0/5] Make xHCI driver endian-safe, add a barrier, some debug

[PATCH 0/5] Make xHCI driver endian-safe, add a barrier, some debug

[Matt Evans]

Hi Sarah,


This series addresses the endian issues with the xHCI driver, and has brought
lovely USB 3 to PPC. :-)  I've tested various types of traffic on ppc4xx and
POWER7 and (some sound driver bugs notwithstanding) all seems fine.  Also
addresses an ordering problem we found and the recursive nature of the event
handling, plus the addition of some debug.

This should apply to 2.6.38/Linus' tree.


Thanks!


Matt

Re: [PATCH 0/5] Make xHCI driver endian-safe, add a barrier, some debug

[Sarah Sharp]

On Fri, Mar 25, 2011 at 06:43:44PM +1100, Matt Evans wrote:
> Hi Sarah,
> 
> 
> This series addresses the endian issues with the xHCI driver, and has brought
> lovely USB 3 to PPC. :-)  I've tested various types of traffic on ppc4xx and
> POWER7 and (some sound driver bugs notwithstanding) all seems fine.  Also
> addresses an ordering problem we found and the recursive nature of the event
> handling, plus the addition of some debug.

Thanks for doing this work, Matt!  I appreciate it.

> This should apply to 2.6.38/Linus' tree.

You say that these apply against 2.6.38, but recently a lot of xHCI
changes went into Linus' tree to support USB 3.0 hubs in 2.6.39.  Will
these patches still apply against Linus' latest tree?  If not, I suggest
you base your patches against Greg KH's usb-linus branch, as that's my
tree base:

http://git.kernel.org/?p=linux/kernel/git/gregkh/usb-2.6.git;a=shortlog;h=refs/heads/usb-linus

Also, I haven't read too far into the patches, but the first patch seems
to have several one or two letter variable names, like "f" and "di".
Can you make those variable names more descriptive?  Thanks.

Sarah Sharp

Re: [PATCH 0/5] Make xHCI driver endian-safe, add a barrier, some debug

[Matt Evans]

On 29/03/11 09:16, Sarah Sharp wrote:
> On Fri, Mar 25, 2011 at 06:43:44PM +1100, Matt Evans wrote:
>> Hi Sarah,
>>
>>
>> This series addresses the endian issues with the xHCI driver, and has brought
>> lovely USB 3 to PPC. :-)  I've tested various types of traffic on ppc4xx and
>> POWER7 and (some sound driver bugs notwithstanding) all seems fine.  Also
>> addresses an ordering problem we found and the recursive nature of the event
>> handling, plus the addition of some debug.
> 
> Thanks for doing this work, Matt!  I appreciate it.

Hey, no worries!  I've got some $3 USB speakers I wanted to connect to the
$15,000 POWER server, you know how it goes.

>> This should apply to 2.6.38/Linus' tree.
> 
> You say that these apply against 2.6.38, but recently a lot of xHCI
> changes went into Linus' tree to support USB 3.0 hubs in 2.6.39.  Will
> these patches still apply against Linus' latest tree?  If not, I suggest
> you base your patches against Greg KH's usb-linus branch, as that's my
> tree base:
> 
> http://git.kernel.org/?p=linux/kernel/git/gregkh/usb-2.6.git;a=shortlog;h=refs/heads/usb-linus

Sorry, I wasn't too explicit about that; I'd meant "Linus' tree as of todayish"
and I believe I caught said USB 3.0 changes, but I'll rebase from usb-linus
anyway to make sure we're fine.

> Also, I haven't read too far into the patches, but the first patch seems
> to have several one or two letter variable names, like "f" and "di".
> Can you make those variable names more descriptive?  Thanks.

Sure; they were single-use throwaways to break long RMW lines but I've rejiggled
& removed them.

Thanks for looking, I'll repost v3 with tidyups.


Cheers,


Matt