Re: [PATCH v3 0/9] Tegra xHCI support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 09/16/2014 05:51 PM, Andrew Bresticker wrote:
On Tue, Sep 16, 2014 at 4:15 PM, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote:
On 09/16/2014 04:46 PM, Andrew Bresticker wrote:

On Tue, Sep 16, 2014 at 9:57 AM, Andrew Bresticker
<abrestic@xxxxxxxxxxxx> wrote:

On Tue, Sep 16, 2014 at 8:26 AM, Stephen Warren <swarren@xxxxxxxxxxxxx>
wrote:

On 09/15/2014 01:30 PM, Andrew Bresticker wrote:


On Mon, Sep 15, 2014 at 11:09 AM, Stephen Warren
<swarren@xxxxxxxxxxxxx>
wrote:


On 09/15/2014 11:06 AM, Andrew Bresticker wrote:


On Mon, Sep 15, 2014 at 12:00 AM, Tomeu Vizoso
<tomeu@xxxxxxxxxxxxxxx>
wrote:


On 12 September 2014 18:37, Andrew Bresticker
<abrestic@xxxxxxxxxxxx>
wrote:


On Tue, Sep 9, 2014 at 1:21 AM, Tomeu Vizoso
<tomeu@xxxxxxxxxxxxxxx>
wrote:


On 8 September 2014 18:22, Andrew Bresticker
<abrestic@xxxxxxxxxxxx>
wrote:


On Mon, Sep 8, 2014 at 8:34 AM, Tomeu Vizoso
<tomeu@xxxxxxxxxxxxxxx>
wrote:


On 2 September 2014 23:34, Andrew Bresticker
<abrestic@xxxxxxxxxxxx> wrote:


Tested on Venice2, Jetson TK1, and Big with a variety of USB2.0
and
USB3.0 memory sticks and ethernet dongles using controller
firmware
recently posted by Andrew Chew [2].


...


Stephen, Thierry, have either of you had a chance to test this
series?



I haven't had a chance to yet. I just went to try it out, and found
that
it
depends on a whole slew of other patches that I don't have. Is there a
git
branch somewhere to save me having to track down all the dependencies?



Yes, Tomeu has the branch he used for testing here:
http://cgit.collabora.com/git/user/tomeu/linux.git/log/?h=3.17rc4-xhci



Hmm. That git server was quite reluctant to cough up its bits, but it
did
eventually.

You'll also need the firmware Andrew Chew posted:
https://patchwork.ozlabs.org/patch/384013/



The XHCI driver can't load its firmware unless it's a module; if I make
it
built-in, it fails immediately with error -2 during "Direct firmware
loading". The driver needs to work with either immediate or deferred
firmware loading.


If you want the driver to be built-in, you'll either need to build the
firmware in as well (i.e. EXTRA_FIRMWARE) or enable
FW_LOADER_USER_HELPER_FALLBACK to load with userspace/uevent (though
apparently this is deprecated).

The following USB2 devices had problems:

0b95:7720 ASIX Electronics Corp. AX88772

[  489.140536] usb 1-3: new high-speed USB device number 81 using
xhci-tegra
[  489.260860] usb 1-3: device descriptor read/64, error -71
[  489.370804] xhci-tegra 70090000.usb: Setup ERROR: setup context
command
for slot 1.
[  489.378463] usb 1-3: hub failed to enable device, error -22
[  489.500531] usb 1-3: new high-speed USB device number 82 using
xhci-tegra
[  489.655708] usb 1-3: can't set config #1, error -71
[  489.661231] usb 1-3: USB disconnect, device number 82
[  489.940531] usb 1-3: new high-speed USB device number 83 using
xhci-tegra
[  490.060860] usb 1-3: device descriptor read/64, error -71
[  490.170805] xhci-tegra 70090000.usb: Setup ERROR: setup context
command
for slot 1.
[  490.178462] usb 1-3: hub failed to enable device, error -22


(repeats over and over)

15a4:1336 Afatech Technologies, Inc. SDHC/MicroSD/MMC/MS/M2/CF/XD Flash
Card
Reader

The power light comes on, and the activity light just keeps flashing
fast.
Usually the activity light flashes a couple times and then turns off.
There
is nothing in dmesg at all for this device.

05e3:0608 Genesys Logic, Inc. USB-2.0 4-Port HUB

Not detected. There is nothing in dmesg at all for this device.

1bcf:0c31 Sunplus Innovation Technology Inc. SPIF30x Serial-ATA bridge

Not detected. There is nothing in dmesg at all for this device.


Thanks, I'll try to figure out what's going on here.


Grr... this is due to the unfortunate UTMI pad controller design on Tegra.

Here's the issue:  When UTMI pad 0 is assigned to the EHCI controller
(as is currently the case on Jetson-TK1), the UTMI parameters from
UTMIP_BIAS_CFG0 in the global UTMI pad register space are used
regardless of the owner of the UTMI pads.  If pad 0 is assigned to the
XUSB controller, then parameters in USB2_BIAS_PAD_CTL0 in the XUSB
register space are used for all UTMI pads (again, regardless of
ownership).

I wasn't able to reproduce before because I always TFTP booted and
U-Boot programs the UTMI parameters correctly when starting the EHCI
controllers.  I suspect you and Tomeu were booting without starting
the EHCI controllers and thus were stuck with the POR values.


I am loading the kernel over the PCIe Ethernet on Jetson TK1. However, the
U-Boot boot scripts do "usb start" first, so I would assume all the USB set
is already done either way, before U-Boot attempts to probe USB devices.

Interesting - just doing "usb start" made the difference for me.

The easiest way to fix this is to just assign UTMI port 0 (i.e. the
OTG port) to the XUSB controller.  AFAIK, device mode on Tegra isn't
supported yet in the kernel, so this shouldn't break any existing use
cases.  If we wanted to support device mode in the future, it would
just have to be done with the XUSB controller instead.  The
alternative is some ugly driver that programs the correct register set
depending on which controller UTMI port 0 is assigned to.

BTW, here's a patch to assign UTMI port 0 to XUSB:

diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
index a473750..dacb0d0 100644
--- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
+++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
@@ -1644,7 +1644,7 @@

                  padctl_default: pinmux {
                          otg {
-                               nvidia,lanes = "otg-1", "otg-2";
+                               nvidia,lanes = "otg-0", "otg-1", "otg-2";
                                  nvidia,function = "xusb";
                          };


That's a bit better; I now see dmesg entries when I plug in my hub where
before I got none. However, I still get some descriptor read errors etc.,

I see those sometimes with my ethernet dongle too, but it still seems
to work ok.

and the USB HDD I have behind the hub isn't fully detected.

Can you post the dmesg output?  Does it work when not behind the hub?
So far I haven't seen any failures with my pile of USB devices after
the above change.

Hmm. I think I must have screwed up my testing after applying that patch; likely I retested a kernel without that patch.

Today, I retested 4 flash devices, 1 SD reader, 1 HDD, 1 mouse, 1 keyboard, 1 Ethernet dongle, and 1 hub. All worked fine. I tested the HDD/mouse/keyboard/ethernet all together behind the hub too, and with it either plugged in at power on, or plugged in after boot, it all worked.

So, I think we're good:-)
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux