Re: [PATCH 0/6] Minor USB fixes and xHCI driver

From: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
	barebox <barebox@lists.infradead.org>
Subject: Re: [PATCH 0/6] Minor USB fixes and xHCI driver
Date: Fri, 25 Jul 2014 16:40:16 +0200	[thread overview]
Message-ID: <CABJ1b_RgTF0eTSVWGy6H01bTVX8OCxwPtRpMQhsev=w53fcZLA@mail.gmail.com> (raw)
In-Reply-To: <20140725073243.GP23235@pengutronix.de>

On Fri, Jul 25, 2014 at 9:32 AM, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> On Thu, Jul 24, 2014 at 03:28:40PM +0200, Sebastian Hesselbarth wrote:
>> This patch set adds initial support for xHCI host controllers either
>> as platform_device or PCI attached device. Compared to EHCI, the
>> xHCI added even more SW stuff around the host controller interface
>> we have to deal with. From a topology point-of-view each xHCI HC
>> represents two virtual Root Hubs, one for USB 3.0 and one for USB
>> 2.0 with TT.
>>
>> The xHCI driver currently only supports virtual USB 2.0 ports of the
>> xHCI controller. If a USB 3.0 device is used, it has to be connected
>> with a USB 2.0 cable, i.e. no SuperSpeed cable. Also, I haven't been
>> able to test any USB 1.1 devices, yet. Anyway, I plan to have a look
>> at both USB 1.1 and USB 3.0 but still I consider the driver in a
>> quite good shape to be released.
>
> Do we have the chance to issue a warning when a device is connected
> with a superspeed cable? If not maybe a general warning in the xhci
> probe function is a good idea.

Sorry, I the reply is f*cked up, have to answer this through gmail.

I did some improvements and added missing pieces, now all non-SS
combinations I have tested are working, e.g LS/FS/HS on Root Hub,
on Single-TT HS Hub, and on Multi-TT HS Hub. I haven't tested any
Hub-Hub paths, but that is something that can wait IMHO.

I am now going to look at SS issues and if I don't find a quick solution,
I'll put a warning that SS is not supported, yet. I will be quite easy to
catch, as there are distinct virtual USB 3.0 ports on each xHCI, i.e.
if the port reports a device, it is SS with SS-cable on USB 3.0 port.

BTW, the error path of usb core device detect is utterly broken.. not
that it is important for most cases, but for debugging failing device
connects it is ;) I'll fix it up anytime soon.

Sebastian

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox