Subject: Re: linux-next: manual merge of the usb tree with the omap tree

From: Felipe Balbi <[email-masked]>
Date: Thu, 3 Mar 2011 19:39:42 +0200

On Thu, Mar 03, 2011 at 08:02:09AM -0800, Greg KH wrote:
> On Thu, Mar 03, 2011 at 10:48:29AM +0200, Felipe Balbi wrote:
> > On Wed, Mar 02, 2011 at 04:57:55PM +1100, Stephen Rothwell wrote:
> > > Hi Greg,
> > > 
> > > Today's linux-next merge of the usb tree got a conflict in
> > > arch/arm/mach-omap2/board-4430sdp.c between commit
> > > 1dbea0f5e23b6c647db72fa4a048cb7140625e13 ("arm: omap4: 4430sdp: drop ehci
> > > support") from the omap tree and commit
> > > 181b250cf53233a7a7c6d7e1e9df402506712e93 ("arm: omap: usb: create common
> > > enums and structures for ehci and ohci") from the usb tree.
> > > 
> > > The former removed the code modified by the latter, so I did that.
> > 
> > This can be fixed if Greg applies this patch:
> > 
> > commit 076bfacd5bba0f2e419474488c8f8c060c7799d8
> > Author: Anand Gadiyar <[email-masked]>
> > Date:   Wed Feb 16 16:47:19 2011 +0530
> 
> Care to provide it to me in a format that I can apply it?  :)

heh, my bad. It's attached.

-- 
balbi
>>From 076bfacd5bba0f2e419474488c8f8c060c7799d8 Mon Sep 17 00:00:00 2001
From: Anand Gadiyar <[email-masked]>
Date: Wed, 16 Feb 2011 16:47:19 +0530
Subject: [PATCH] arm: omap4: 4430sdp: drop ehci support
Organization: Texas Instruments\n

Most revisions of the OMAP4 Blaze/SDP platform do not have
the EHCI signals routed by default. The pads are routed
for the alternate HSI functionality instead, and explicit
board modifications are needed to route the signals to
the USB PHY on the board.

Also, turning on the PHY connected to the EHCI port causes
a board reboot during bootup due to an unintended short
on the rails - this affects many initial revisions of the
board, and needs a minor board mod to fix (or as a
workaround, one should not attempt to power on the
USB PHY).

Given that these boards need explicit board mods to even
get EHCI working (separate from the accidental short above),
we should not attempt to enable EHCI by default.

So drop the EHCI support from the board files for the
Blaze/SDP platforms.

Signed-off-by: Anand Gadiyar <[email-masked]>
Cc: Keshava Munegowda <[email-masked]>
Cc: Tony Lindgren <[email-masked]>
Signed-off-by: Felipe Balbi <[email-masked]>
---
 arch/arm/mach-omap2/board-4430sdp.c |   19 -------------------
 1 files changed, 0 insertions(+), 19 deletions(-)

diff --git a/arch/arm/mach-omap2/board-4430sdp.c b/arch/arm/mach-omap2/board-4430sdp.c
index 1230121..f603f3b 100644
--- a/arch/arm/mach-omap2/board-4430sdp.c
+++ b/arch/arm/mach-omap2/board-4430sdp.c
@@ -44,7 +44,6 @@
 #define ETH_KS8851_IRQ			34
 #define ETH_KS8851_POWER_ON		48
 #define ETH_KS8851_QUART		138
-#define OMAP4SDP_MDM_PWR_EN_GPIO	157
 #define OMAP4_SFH7741_SENSOR_OUTPUT_GPIO	184
 #define OMAP4_SFH7741_ENABLE_GPIO		188
 
@@ -251,16 +250,6 @@ static void __init omap_4430sdp_init_irq(void)
 	gic_init_irq();
 }
 
-static const struct usbhs_omap_board_data usbhs_bdata __initconst = {
-	.port_mode[0]	= OMAP_EHCI_PORT_MODE_PHY,
-	.port_mode[1]	= OMAP_USBHS_PORT_MODE_UNUSED,
-	.port_mode[2]	= OMAP_USBHS_PORT_MODE_UNUSED,
-	.phy_reset	= false,
-	.reset_gpio_port[0]  = -EINVAL,
-	.reset_gpio_port[1]  = -EINVAL,
-	.reset_gpio_port[2]  = -EINVAL,
-};
-
 static struct omap_musb_board_data musb_board_data = {
 	.interface_type		= MUSB_INTERFACE_UTMI,
 	.mode			= MUSB_OTG,
@@ -577,14 +566,6 @@ static void __init omap_4430sdp_init(void)
 	omap_serial_init();
 	omap4_twl6030_hsmmc_init(mmc);
 
-	/* Power on the ULPI PHY */
-	status = gpio_request(OMAP4SDP_MDM_PWR_EN_GPIO, "USBB1 PHY VMDM_3V3");
-	if (status)
-		pr_err("%s: Could not get USBB1 PHY GPIO\n", __func__);
-	else
-		gpio_direction_output(OMAP4SDP_MDM_PWR_EN_GPIO, 1);
-
-	usbhs_init(&usbhs_bdata);
 	usb_musb_init(&musb_board_data);
 
 	status = omap_ethernet_init();
-- 
1.7.4.rc2