Hi Remy,

could you please review this patch? It's waiting for your approval.

Regards, Ilya.

On 13.12.2011 03:15, Ilya Yanok wrote:
> Taken from Beagle code. Tested on mcx board (AM3517-based).
> 
> Signed-off-by: Ilya Yanok <ya...@emcraft.com>
> ---
> Changes from V3:
>  - None
> Changes from V2:
>  - None
> Changes from V1:
>  - CONFIG_OMAP_EHCI_PHYx_RESET changed to CONFIG_OMAP_EHCI_PHYx_RESET_GPIO
>  - phy reset moved to separate function
>  - Calls to gpio_set_value after gpio_direction_output removed
> 
>  board/ti/beagle/beagle.c       |  101 --------------------------
>  drivers/usb/host/Makefile      |    1 +
>  drivers/usb/host/ehci-omap.c   |  156 
> ++++++++++++++++++++++++++++++++++++++++
>  include/configs/omap3_beagle.h |    3 +
>  4 files changed, 160 insertions(+), 101 deletions(-)
>  create mode 100644 drivers/usb/host/ehci-omap.c
> 
> diff --git a/board/ti/beagle/beagle.c b/board/ti/beagle/beagle.c
> index 6a457cb..3d63028 100644
> --- a/board/ti/beagle/beagle.c
> +++ b/board/ti/beagle/beagle.c
> @@ -42,15 +42,6 @@
>  #include <asm/arch/sys_proto.h>
>  #include <asm/gpio.h>
>  #include <asm/mach-types.h>
> -#ifdef CONFIG_USB_EHCI
> -#include <usb.h>
> -#include <asm/arch/clocks.h>
> -#include <asm/arch/clocks_omap3.h>
> -#include <asm/arch/ehci_omap3.h>
> -/* from drivers/usb/host/ehci-core.h */
> -extern struct ehci_hccr *hccr;
> -extern volatile struct ehci_hcor *hcor;
> -#endif
>  #include "beagle.h"
>  #include <command.h>
>  
> @@ -441,104 +432,12 @@ int board_mmc_init(bd_t *bis)
>  #endif
>  
>  #ifdef CONFIG_USB_EHCI
> -
> -#define GPIO_PHY_RESET 147
> -
> -/* Reset is needed otherwise the kernel-driver will throw an error. */
> -int ehci_hcd_stop(void)
> -{
> -     pr_debug("Resetting OMAP3 EHCI\n");
> -     gpio_set_value(GPIO_PHY_RESET, 0);
> -     writel(OMAP_UHH_SYSCONFIG_SOFTRESET, OMAP3_UHH_BASE + 
> OMAP_UHH_SYSCONFIG);
> -     /* disable USB clocks */
> -     struct prcm *prcm_base = (struct prcm *)PRCM_BASE;
> -     sr32(&prcm_base->iclken_usbhost, 0, 1, 0);
> -     sr32(&prcm_base->fclken_usbhost, 0, 2, 0);
> -     sr32(&prcm_base->iclken3_core, 2, 1, 0);
> -     sr32(&prcm_base->fclken3_core, 2, 1, 0);
> -     return 0;
> -}
> -
>  /* Call usb_stop() before starting the kernel */
>  void show_boot_progress(int val)
>  {
>       if(val == 15)
>               usb_stop();
>  }
> -
> -/*
> - * Initialize the OMAP3 EHCI controller and PHY on the BeagleBoard.
> - * Based on "drivers/usb/host/ehci-omap.c" from Linux 2.6.37.
> - * See there for additional Copyrights.
> - */
> -int ehci_hcd_init(void)
> -{
> -     pr_debug("Initializing OMAP3 ECHI\n");
> -
> -     /* Put the PHY in RESET */
> -     gpio_request(GPIO_PHY_RESET, "");
> -     gpio_direction_output(GPIO_PHY_RESET, 0);
> -     gpio_set_value(GPIO_PHY_RESET, 0);
> -
> -     /* Hold the PHY in RESET for enough time till DIR is high */
> -     /* Refer: ISSUE1 */
> -     udelay(10);
> -
> -     struct prcm *prcm_base = (struct prcm *)PRCM_BASE;
> -     /* Enable USBHOST_L3_ICLK (USBHOST_MICLK) */
> -     sr32(&prcm_base->iclken_usbhost, 0, 1, 1);
> -     /*
> -      * Enable USBHOST_48M_FCLK (USBHOST_FCLK1)
> -      * and USBHOST_120M_FCLK (USBHOST_FCLK2)
> -      */
> -     sr32(&prcm_base->fclken_usbhost, 0, 2, 3);
> -     /* Enable USBTTL_ICLK */
> -     sr32(&prcm_base->iclken3_core, 2, 1, 1);
> -     /* Enable USBTTL_FCLK */
> -     sr32(&prcm_base->fclken3_core, 2, 1, 1);
> -     pr_debug("USB clocks enabled\n");
> -
> -     /* perform TLL soft reset, and wait until reset is complete */
> -     writel(OMAP_USBTLL_SYSCONFIG_SOFTRESET,
> -             OMAP3_USBTLL_BASE + OMAP_USBTLL_SYSCONFIG);
> -     /* Wait for TLL reset to complete */
> -     while (!(readl(OMAP3_USBTLL_BASE + OMAP_USBTLL_SYSSTATUS)
> -                     & OMAP_USBTLL_SYSSTATUS_RESETDONE));
> -     pr_debug("TLL reset done\n");
> -
> -     writel(OMAP_USBTLL_SYSCONFIG_ENAWAKEUP |
> -             OMAP_USBTLL_SYSCONFIG_SIDLEMODE |
> -             OMAP_USBTLL_SYSCONFIG_CACTIVITY,
> -             OMAP3_USBTLL_BASE + OMAP_USBTLL_SYSCONFIG);
> -
> -     /* Put UHH in NoIdle/NoStandby mode */
> -     writel(OMAP_UHH_SYSCONFIG_ENAWAKEUP
> -             | OMAP_UHH_SYSCONFIG_SIDLEMODE
> -             | OMAP_UHH_SYSCONFIG_CACTIVITY
> -             | OMAP_UHH_SYSCONFIG_MIDLEMODE,
> -             OMAP3_UHH_BASE + OMAP_UHH_SYSCONFIG);
> -
> -     /* setup burst configurations */
> -     writel(OMAP_UHH_HOSTCONFIG_INCR4_BURST_EN
> -             | OMAP_UHH_HOSTCONFIG_INCR8_BURST_EN
> -             | OMAP_UHH_HOSTCONFIG_INCR16_BURST_EN,
> -             OMAP3_UHH_BASE + OMAP_UHH_HOSTCONFIG);
> -
> -     /*
> -      * Refer ISSUE1:
> -      * Hold the PHY in RESET for enough time till
> -      * PHY is settled and ready
> -      */
> -     udelay(10);
> -     gpio_set_value(GPIO_PHY_RESET, 1);
> -
> -     hccr = (struct ehci_hccr *)(OMAP3_EHCI_BASE);
> -     hcor = (struct ehci_hcor *)(OMAP3_EHCI_BASE + 0x10);
> -
> -     pr_debug("OMAP3 EHCI init done\n");
> -     return 0;
> -}
> -
>  #endif /* CONFIG_USB_EHCI */
>  
>  #ifndef CONFIG_SPL_BUILD
> diff --git a/drivers/usb/host/Makefile b/drivers/usb/host/Makefile
> index 77e217f..975c3e5 100644
> --- a/drivers/usb/host/Makefile
> +++ b/drivers/usb/host/Makefile
> @@ -43,6 +43,7 @@ endif
>  COBJS-$(CONFIG_USB_EHCI_MXC) += ehci-mxc.o
>  COBJS-$(CONFIG_USB_EHCI_MXS) += ehci-mxs.o
>  COBJS-$(CONFIG_USB_EHCI_MX5) += ehci-mx5.o
> +COBJS-$(CONFIG_USB_EHCI_OMAP) += ehci-omap.o
>  COBJS-$(CONFIG_USB_EHCI_PPC4XX) += ehci-ppc4xx.o
>  COBJS-$(CONFIG_USB_EHCI_IXP4XX) += ehci-ixp.o
>  COBJS-$(CONFIG_USB_EHCI_KIRKWOOD) += ehci-kirkwood.o
> diff --git a/drivers/usb/host/ehci-omap.c b/drivers/usb/host/ehci-omap.c
> new file mode 100644
> index 0000000..93d3bb7
> --- /dev/null
> +++ b/drivers/usb/host/ehci-omap.c
> @@ -0,0 +1,156 @@
> +/*
> + * (C) Copyright 2011 Ilya Yanok, Emcraft Systems
> + * (C) Copyright 2004-2008
> + * Texas Instruments, <www.ti.com>
> + *
> + * Derived from Beagle Board code by
> + *   Sunil Kumar <sunilsain...@gmail.com>
> + *   Shashi Ranjan <shashiranjanmc...@gmail.com>
> + *
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc.
> + */
> +#include <common.h>
> +#include <usb.h>
> +#include <asm/io.h>
> +#include <asm/gpio.h>
> +#include <asm/arch/clocks.h>
> +#include <asm/arch/clocks_omap3.h>
> +#include <asm/arch/ehci_omap3.h>
> +#include <asm/arch/sys_proto.h>
> +#include "ehci-core.h"
> +
> +inline int __board_usb_init(void)
> +{
> +     return 0;
> +}
> +int board_usb_init(void) __attribute__((weak, alias("__board_usb_init")));
> +
> +#if defined(CONFIG_OMAP_EHCI_PHY1_RESET_GPIO) || \
> +     defined(CONFIG_OMAP_EHCI_PHY2_RESET_GPIO)
> +/* controls PHY(s) reset signal(s) */
> +static inline void omap_ehci_phy_reset(int on, int delay)
> +{
> +     /*
> +      * Refer ISSUE1:
> +      * Hold the PHY in RESET for enough time till
> +      * PHY is settled and ready
> +      */
> +     if (delay && !on)
> +             udelay(delay);
> +#ifdef CONFIG_OMAP_EHCI_PHY1_RESET_GPIO
> +     gpio_request(CONFIG_OMAP_EHCI_PHY1_RESET_GPIO, "USB PHY1 reset");
> +     gpio_direction_output(CONFIG_OMAP_EHCI_PHY1_RESET_GPIO, !on);
> +#endif
> +#ifdef CONFIG_OMAP_EHCI_PHY2_RESET_GPIO
> +     gpio_request(CONFIG_OMAP_EHCI_PHY2_RESET_GPIO, "USB PHY2 reset");
> +     gpio_direction_output(CONFIG_OMAP_EHCI_PHY2_RESET_GPIO, !on);
> +#endif
> +
> +     /* Hold the PHY in RESET for enough time till DIR is high */
> +     /* Refer: ISSUE1 */
> +     if (delay && on)
> +             udelay(delay);
> +}
> +#else
> +#define omap_ehci_phy_reset(on, delay)       do {} while (0)
> +#endif
> +
> +/* Reset is needed otherwise the kernel-driver will throw an error. */
> +int ehci_hcd_stop(void)
> +{
> +     debug("Resetting OMAP3 EHCI\n");
> +     omap_ehci_phy_reset(1, 0);
> +     writel(OMAP_UHH_SYSCONFIG_SOFTRESET,
> +                     OMAP3_UHH_BASE + OMAP_UHH_SYSCONFIG);
> +     /* disable USB clocks */
> +     struct prcm *prcm_base = (struct prcm *)PRCM_BASE;
> +     sr32(&prcm_base->iclken_usbhost, 0, 1, 0);
> +     sr32(&prcm_base->fclken_usbhost, 0, 2, 0);
> +     sr32(&prcm_base->iclken3_core, 2, 1, 0);
> +     sr32(&prcm_base->fclken3_core, 2, 1, 0);
> +     return 0;
> +}
> +
> +/*
> + * Initialize the OMAP3 EHCI controller and PHY.
> + * Based on "drivers/usb/host/ehci-omap.c" from Linux 2.6.37.
> + * See there for additional Copyrights.
> + */
> +int ehci_hcd_init(void)
> +{
> +     int ret;
> +
> +     debug("Initializing OMAP3 EHCI\n");
> +
> +     ret = board_usb_init();
> +     if (ret < 0)
> +             return ret;
> +
> +     /* Put the PHY in RESET */
> +     omap_ehci_phy_reset(1, 10);
> +
> +     struct prcm *prcm_base = (struct prcm *)PRCM_BASE;
> +     /* Enable USBHOST_L3_ICLK (USBHOST_MICLK) */
> +     sr32(&prcm_base->iclken_usbhost, 0, 1, 1);
> +     /*
> +      * Enable USBHOST_48M_FCLK (USBHOST_FCLK1)
> +      * and USBHOST_120M_FCLK (USBHOST_FCLK2)
> +      */
> +     sr32(&prcm_base->fclken_usbhost, 0, 2, 3);
> +     /* Enable USBTTL_ICLK */
> +     sr32(&prcm_base->iclken3_core, 2, 1, 1);
> +     /* Enable USBTTL_FCLK */
> +     sr32(&prcm_base->fclken3_core, 2, 1, 1);
> +     debug("USB clocks enabled\n");
> +
> +     /* perform TLL soft reset, and wait until reset is complete */
> +     writel(OMAP_USBTLL_SYSCONFIG_SOFTRESET,
> +             OMAP3_USBTLL_BASE + OMAP_USBTLL_SYSCONFIG);
> +     /* Wait for TLL reset to complete */
> +     while (!(readl(OMAP3_USBTLL_BASE + OMAP_USBTLL_SYSSTATUS)
> +                     & OMAP_USBTLL_SYSSTATUS_RESETDONE))
> +             ;
> +     debug("TLL reset done\n");
> +
> +     writel(OMAP_USBTLL_SYSCONFIG_ENAWAKEUP |
> +             OMAP_USBTLL_SYSCONFIG_SIDLEMODE |
> +             OMAP_USBTLL_SYSCONFIG_CACTIVITY,
> +             OMAP3_USBTLL_BASE + OMAP_USBTLL_SYSCONFIG);
> +
> +     /* Put UHH in NoIdle/NoStandby mode */
> +     writel(OMAP_UHH_SYSCONFIG_ENAWAKEUP
> +             | OMAP_UHH_SYSCONFIG_SIDLEMODE
> +             | OMAP_UHH_SYSCONFIG_CACTIVITY
> +             | OMAP_UHH_SYSCONFIG_MIDLEMODE,
> +             OMAP3_UHH_BASE + OMAP_UHH_SYSCONFIG);
> +
> +     /* setup burst configurations */
> +     writel(OMAP_UHH_HOSTCONFIG_INCR4_BURST_EN
> +             | OMAP_UHH_HOSTCONFIG_INCR8_BURST_EN
> +             | OMAP_UHH_HOSTCONFIG_INCR16_BURST_EN,
> +             OMAP3_UHH_BASE + OMAP_UHH_HOSTCONFIG);
> +
> +     omap_ehci_phy_reset(0, 10);
> +
> +     hccr = (struct ehci_hccr *)(OMAP3_EHCI_BASE);
> +     hcor = (struct ehci_hcor *)(OMAP3_EHCI_BASE + 0x10);
> +
> +     debug("OMAP3 EHCI init done\n");
> +     return 0;
> +}
> diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h
> index 91af8a0..b4d6443 100644
> --- a/include/configs/omap3_beagle.h
> +++ b/include/configs/omap3_beagle.h
> @@ -129,6 +129,9 @@
>  /* USB EHCI */
>  #define CONFIG_CMD_USB
>  #define CONFIG_USB_EHCI
> +#define CONFIG_USB_EHCI_OMAP
> +/*#define CONFIG_EHCI_DCACHE*/ /* leave it disabled for now */
> +#define CONFIG_OMAP_EHCI_PHY1_RESET_GPIO     147
>  #define CONFIG_SYS_USB_EHCI_MAX_ROOT_PORTS 3
>  #define CONFIG_USB_HOST_ETHER
>  #define CONFIG_USB_ETHER_SMSC95XX

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to