diff mbox

[U-Boot] usb: tegra: ULPI regression on tegra20

Message ID ba33f62b1c71ae6c32a0bf792143b2901b1bf038.1412379672.git.marcel@ziswiler.com
State Accepted
Delegated to: Marek Vasut
Headers show

Commit Message

Marcel Ziswiler Oct. 3, 2014, 11:46 p.m. UTC
Trying to enumerate USB devices connected via ULPI to T20 failed as
follows:

USB2:   ULPI integrity check failed

Git bisecting revealed the following commit being at odds:

commit 2d34151f7501ddaa599897f0d89ad576126b03eb
usb: tegra: refactor PHY type selection

Looking at above commit one quickly identifies a copy paste error which
this patch fixes. Happy ULPIing again (;-p).

Signed-off-by: Marcel Ziswiler <marcel@ziswiler.com>
---
 drivers/usb/host/ehci-tegra.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Marek Vasut Oct. 6, 2014, 2:41 p.m. UTC | #1
On Saturday, October 04, 2014 at 01:46:10 AM, Marcel Ziswiler wrote:
> Trying to enumerate USB devices connected via ULPI to T20 failed as
> follows:
> 
> USB2:   ULPI integrity check failed
> 
> Git bisecting revealed the following commit being at odds:
> 
> commit 2d34151f7501ddaa599897f0d89ad576126b03eb
> usb: tegra: refactor PHY type selection
> 
> Looking at above commit one quickly identifies a copy paste error which
> this patch fixes. Happy ULPIing again (;-p).
> 
> Signed-off-by: Marcel Ziswiler <marcel@ziswiler.com>

Applied, thanks!

Best regards,
Marek Vasut
diff mbox

Patch

diff --git a/drivers/usb/host/ehci-tegra.c b/drivers/usb/host/ehci-tegra.c
index 33e5ea9..5f0a98e 100644
--- a/drivers/usb/host/ehci-tegra.c
+++ b/drivers/usb/host/ehci-tegra.c
@@ -305,11 +305,11 @@  static void init_phy_mux(struct fdt_usb *config, uint pts,
 #if defined(CONFIG_TEGRA20)
 	if (config->periph_id == PERIPH_ID_USBD) {
 		clrsetbits_le32(&usbctlr->port_sc1, PTS1_MASK,
-				PTS_UTMI << PTS1_SHIFT);
+				pts << PTS1_SHIFT);
 		clrbits_le32(&usbctlr->port_sc1, STS1);
 	} else {
 		clrsetbits_le32(&usbctlr->port_sc1, PTS_MASK,
-				PTS_UTMI << PTS_SHIFT);
+				pts << PTS_SHIFT);
 		clrbits_le32(&usbctlr->port_sc1, STS);
 	}
 #else