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 1/3] usb: tegra: fix PHY selection code
Date: Thu, 01 May 2014 10:56:24 -0600	[thread overview]
Message-ID: <53627CB8.1000201@wwwdotorg.org> (raw)
In-Reply-To: <201405011249.06271.marex@denx.de>

On 05/01/2014 04:49 AM, Marek Vasut wrote:
> On Wednesday, April 30, 2014 at 11:09:55 PM, Stephen Warren wrote:
>> From: Stephen Warren <swarren@nvidia.com>
>>
>> The TRM for Tegra30 and later all state that USBMODE_CM_HC must be set
>> before writing to hostpc1_devlc to select which PHY to use for a USB
>> controller. However, neither init_{utmi,ulpi}_usb_controller() do this
>> today, so the register writes they perform for PHY selection do not
>> work.
>>
>> For the UTMI case, this was hacked around in commit 7e44d9320ed4 "ARM:
>> Tegra: USB: EHCI: Add support for Tegra30/Tegra114" by adding code to
>> ehci_hcd_init() which sets USBMODE_CM_HC and duplicates the PHY
>> selection register write. This code doesn't cover the ULPI case, so I
>> wouldn't be surprised if ULPI doesn't work with the current code, unless
>> the ordering requirement only ends up being an issue in HW for UTMI not
>> ULPI.
>>
>> This patch fixes init_{utmi,ulpi}_usb_controller() to correctly set
>> USBMODE_CM_HC before selecting the PHY. Now that this works, we can
>> remove the duplicate UTMI-specific code in ehci_hcd_init(), thus
>> simplifying that function.
> 
> This doesn't apply on u-boot-usb/master , so I cannot pick this, sorry .

Ah right. That's because I'm working on top of a merge of
u-boot-arm/master and u-boot/master, and that includes the following
changes to the Tegra USB driver that went in through u-boot-tegra/master
-> u-boot-arm/master rather than the USB tree:

8f9fd6caafb8 usb: tegra: combine header file
b1d615f3f102 usb: tegra: fix PHY configuration
b03f4b3742a7 usb: tegra: fix USB2 powerdown for Tegra30 and later

What's the best fix here:

* Wait until the previous patches get into u-boot/master, and
u-boot-usb/master picks them up, and then apply this series.

* Apply this series to u-boot-tegra/master with your ack?

There's little point my rebasing this series on top of
u-boot-usb/master, and applying it right now, since that will just cause
the same conflicts when u-boot-usb/master gets merged into u-boot/master.

Thanks.

  reply	other threads:[~2014-05-01 16:56 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-30 21:09 [U-Boot] [PATCH 1/3] usb: tegra: fix PHY selection code Stephen Warren
2014-04-30 21:09 ` [U-Boot] [PATCH 2/3] usb: tegra: refactor PHY type selection Stephen Warren
2014-04-30 21:09 ` [U-Boot] [PATCH 3/3] usb: tegra: support device mode Stephen Warren
2014-05-01 10:49 ` [U-Boot] [PATCH 1/3] usb: tegra: fix PHY selection code Marek Vasut
2014-05-01 16:56   ` Stephen Warren [this message]
2014-05-02  4:47     ` Marek Vasut
2014-05-09 22:56       ` Stephen Warren
2014-05-14 17:26       ` Stephen Warren
2014-05-14 22:21         ` Marek Vasut
2014-05-12 12:34 ` Stefan Agner

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=53627CB8.1000201@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.