Felipe Balbi wrote:
> On Wed, Aug 18, 2010 at 04:55:43AM +0200, ext Keshava Munegowda wrote:
> >The PHY reset code is moved at the begining and end of the function
> >omap_start_ehc. This simplfies the writing clocks enabling code for
> >OMAP4 later.
> >
> >Signed-off-by: Keshava Munegowda <keshava_mgo...@ti.com>
> >Signed-off-by: Anand Gadiyar <gadi...@ti.com>
> >---
> > drivers/usb/host/ehci-omap.c | 52
> > +++++++++++++++++++++----------------------
> > 1 file changed, 26 insertions(+), 26 deletions(-)
> >
> >Index: linux-2.6/drivers/usb/host/ehci-omap.c
> >===================================================================
> >--- linux-2.6.orig/drivers/usb/host/ehci-omap.c
> >+++ linux-2.6/drivers/usb/host/ehci-omap.c
> >@@ -279,6 +279,24 @@ static int omap_start_ehc(struct ehci_hc
> >
> > dev_dbg(omap->dev, "starting TI EHCI USB Controller\n");
> >
> >+ if (omap->phy_reset) {
> >+ /* Refer: ISSUE1 */
> >+ if (gpio_is_valid(omap->reset_gpio_port[0])) {
> >+ gpio_request(omap->reset_gpio_port[0],
> >+ "USB1 PHY reset");
> >+ gpio_direction_output(omap->reset_gpio_port[0], 0);
> >+ }
> >+
> >+ if (gpio_is_valid(omap->reset_gpio_port[1])) {
> >+ gpio_request(omap->reset_gpio_port[1],
> >+ "USB2 PHY reset");
> >+ gpio_direction_output(omap->reset_gpio_port[1], 0);
> >+ }
> >+
> >+ /* Hold the PHY in RESET for enough time till DIR is high */
> >+ udelay(10);
> >+ }
>
> there was another patch doing this with ulpi messages and that's more
> portable as every board will have support for softreset but we can't
> guarantee all of them will have these gpio routed correctly, right ?
This is just reorganizing existing code - these are for OMAP3 boards, all
of which have been designed with a GPIO to hold the transceiver in reset
until the OMAP is configured for them.
At the point of time at which this code executes, the ULPI link is not up,
and we cannot use ulpi messages to reset the transceivers.
- Anand
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
