mail archive of the barebox mailing list
 help / color / mirror / Atom feed
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 17:34:37 +0200	[thread overview]
Message-ID: <CABJ1b_Qi4iSkHfdgLqH1kOcxiM06Xg0dSxzJUiKR1DyxQjDnUw@mail.gmail.com> (raw)
In-Reply-To: <CABJ1b_RgTF0eTSVWGy6H01bTVX8OCxwPtRpMQhsev=w53fcZLA@mail.gmail.com>

On Fri, Jul 25, 2014 at 4:40 PM, Sebastian Hesselbarth
<sebastian.hesselbarth@gmail.com> wrote:
> 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 just tested SS (with SS cable) on Root Hub and it worked.. but just
on one of the both ports Mirabox has. Now I am quite convinced that
the inscription on the back of Mirabox above USB ports isn't a typo ;)

Both ports are blue, i.e. they should be SS capable. The working one
has "USB 3.0" printed above it, the one that does not work has "USB 2.0"
printed over it. I thought it was just a typo, I guess it isn't.

> 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.

Ok, I admit that error reporting capabilities of the driver could be improved
a lot. OTOH, usb core itself isn't very noisy about errors too, it just silently
fails.

I'll send an updated v2 this weekend or beginning of next week.

> 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.

When I get back to above, I'll also add some more verbose error reporting
to xHCI HCD.

Sebastian

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

      reply	other threads:[~2014-07-25 15:34 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-24 13:28 Sebastian Hesselbarth
2014-07-24 13:28 ` [PATCH 1/6] USB: Fix stale usb devices in usb_device_list Sebastian Hesselbarth
2014-07-24 16:09   ` Sebastian Hesselbarth
2014-07-24 18:51     ` Sebastian Hesselbarth
2014-07-24 13:28 ` [PATCH 2/6] USB: Count detected USB devices independent of dev_index Sebastian Hesselbarth
2014-07-24 13:28 ` [PATCH 3/6] USB: EHCI: use min3 from Linux Sebastian Hesselbarth
2014-07-24 13:28 ` [PATCH 4/6] include: import {lower,upper}_32_bits helpers Sebastian Hesselbarth
2014-07-24 13:28 ` [PATCH 5/6] USB: host: add xHCI HCD, Hub, and platform driver Sebastian Hesselbarth
2014-07-24 13:46   ` Sebastian Hesselbarth
2014-07-24 13:28 ` [PATCH 6/6] USB: host: add xHCI PCI driver Sebastian Hesselbarth
2014-07-24 19:11 ` [PATCH 0/6] Minor USB fixes and xHCI driver Sebastian Hesselbarth
2014-07-25  7:32 ` Sascha Hauer
2014-07-25 14:40   ` Sebastian Hesselbarth
2014-07-25 15:34     ` Sebastian Hesselbarth [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CABJ1b_Qi4iSkHfdgLqH1kOcxiM06Xg0dSxzJUiKR1DyxQjDnUw@mail.gmail.com \
    --to=sebastian.hesselbarth@gmail.com \
    --cc=barebox@lists.infradead.org \
    --cc=s.hauer@pengutronix.de \
    --cc=thomas.petazzoni@free-electrons.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox