All of lore.kernel.org
 help / color / mirror / Atom feed
From: Aaron Williams <Aaron.Williams@caviumnetworks.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] XHCI Issues
Date: Mon, 13 Jan 2014 20:28:04 -0800	[thread overview]
Message-ID: <52D4BCD4.7010908@caviumnetworks.com> (raw)
In-Reply-To: <52D49EE7.6060006@caviumnetworks.com>

On 01/13/2014 06:20 PM, Aaron Williams wrote:
> Hi all,
>
> I am bringing up XHCI support for our SOC and have run into several 
> issues with U-Boot's XHCI code.
>
> 1. I need to use a wrapper I call xhci_readl/xhci_writel since all of 
> our I/O is to 64-bit addresses rather than readl/writel which work for 
> PCIe. While xhci_readl/xhci_writel is used in most places there are a 
> few places this is missing.
>
> 2. A memory address used by a pointer is not a DMA address on MIPS. On 
> MIPS a wrapper is needed for all read/write operations to descriptors 
> to map the addresses. Generally U-Boot runs in KSEG0 where a pointer 
> always has bit 31 set. In our case we run it in virtual memory with 
> U-Boot loaded at the top of memory (which might be as high as 
> 64GB+256MB).
>
> 3. xhci_alloc_virt_device is missing a conversion from big endian to 
> little endian on line 401 of xhci-mem.c.
>
> 4. I keep getting hit with
>
> BUG_ON(GET_COMP_CODE(le32_to_cpu(event->trans_event.transfer_len
>         != COMP_STOP)));
> on line 491 in xhci-ring.c. Commenting this out seems to work fine.
>
> 5. I also hit the BUG_ON on line 722 of xhci-ring.c since on MIPS a 
> pointer is not a DMA address and the fact that we use 64-bit addresses 
> where pointers are 32-bits.
>
> I ran into similar issues with EHCI where again we have to map 
> pointers to physical addresses and use our own wrappers for register 
> accesses.
>
> At some point I would love to merge our stuff back into the mainline 
> U-Boot but it's going to be a huge job given that we probably have 
> almost 250K lines of code involved for our MIPS SOCs.
>
> -Aaron
>
I have a follow-up:

I am seeing error messages when hub ports are reset.

In the output below I have a USB 3 thumb drive plugged into USB0 and a 
USB 2 thumb drive plugged into USB1. Each USB device has "2" ports, 
where port 0 is treated as a USB 3 port and port 1 is treated as a USB 
1/2 port. I think the "cannot reset port X!?" message is invalid since 
otherwise everything appears to be working.

I am also seeing rather slow transfers of around 1.9MiB/sec. Is this 
normal for XHCI? This is for both USB 2 and a high-speed USB 3 thumb 
drive when loading a 40MiB Linux kernel image.

-Aaron

-- 
Aaron Williams
Software Engineer
Cavium, Inc.
(408) 943-7198  (510) 789-8988 (cell)

  reply	other threads:[~2014-01-14  4:28 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-14  2:20 [U-Boot] XHCI Issues Aaron Williams
2014-01-14  4:28 ` Aaron Williams [this message]
2014-01-15  7:13   ` Aaron Williams

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=52D4BCD4.7010908@caviumnetworks.com \
    --to=aaron.williams@caviumnetworks.com \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.