All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/5] Tegra 2 USB ULPI series
Date: Thu, 23 Aug 2012 12:42:27 -0600	[thread overview]
Message-ID: <50367993.50407@wwwdotorg.org> (raw)
In-Reply-To: <1345580337-9377-1-git-send-email-dev@lynxeye.de>

On 08/21/2012 02:18 PM, Lucas Stach wrote:
> With this series we are able to initialize USB controllers
> using an external ULPI phy AKA USB2 on Tegra 2 devices.
> 
> This was tested to work on a Toradex Colibri T20 board,
> where USB2 is used to access the ASIX ethernet chipset.
> Testing was done with "tegra20: usb: rework set_host_mode" [1]
> applied. I did not spot any regressions on the UTMI ports.
> 
> v2 incorporates all the review feedback I've got so far,
> including now trying harder to enable VBus in all configurations.
> 
> Patch 3 is already in u-boot-usb and only provided as the ULPI
> init code now depends on it.
> Igor could you please take a look at Patch 4?
> 
> Patchset is based on top of u-boot-tegra/next
> 
> [1] http://lists.denx.de/pipermail/u-boot/2012-August/130177.html

OK, with the addition of an appropriate pin_mux_usb() function to
harmony.c, this works now. So, please consider the series,

Tested-by: Stephen Warren <swarren@wwwdotorg.org>

Note that I did see the following:

Tegra20 (Harmony) # usb start
(Re)start USB...
USB:   Register 10011 NbrPorts 1
USB EHCI 1.00
scanning bus for devices... Register 10011 NbrPorts 1
USB EHCI 1.00
scanning bus for devices... 5 USB Device(s) found
       scanning bus for storage devices... Device NOT ready
   Request Sense returned 02 3A 00
2 Storage Device(s) found
       scanning bus for ethernet devices... 1 Ethernet Device(s) found

However, both "usb tree" and then "ext2ls" on the USB device succeeded
without issue, so I think that's tested well enough. I even booted a
kernel from the USB device, and the kernel was able to use the USB
device for the root filesystem, so U-Boot hadn't broken it in any way:-)

      parent reply	other threads:[~2012-08-23 18:42 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-21 20:18 [U-Boot] [PATCH v2 0/5] Tegra 2 USB ULPI series Lucas Stach
2012-08-21 20:18 ` [U-Boot] [PATCH v2 1/5] tegra20: complete periph_id enum Lucas Stach
2012-08-21 20:18 ` [U-Boot] [PATCH v2 2/5] tegra20: add clock_set_pllout function Lucas Stach
2012-08-21 20:18 ` [U-Boot] [PATCH v2 3/5] usb: fix ulpi_set_vbus prototype Lucas Stach
2012-08-21 20:18 ` [U-Boot] [PATCH v2 4/5] usb: ulpi: add indicator configuration function Lucas Stach
2012-09-05  7:52   ` Igor Grinberg
2012-09-05  8:51     ` Marek Vasut
2012-09-05 16:25       ` Tom Warren
2012-09-06  0:06         ` Lucas Stach
2012-09-07  0:11           ` Marek Vasut
2012-08-21 20:18 ` [U-Boot] [PATCH v2 5/5] tegra20: add USB ULPI init code Lucas Stach
2012-09-05  8:22   ` Igor Grinberg
2012-08-23 18:42 ` Stephen Warren [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=50367993.50407@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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.