On Aug 8, 2011, at 3:47 PM, Kumar Gala wrote:

> From: Ramneek Mehresh <ramneek.mehr...@freescale.com>
> 
> Add UTMI and ULPI PHY support for USB controller on qoriq series of
> processors with internal UTMI PHY implemented, for example P1010/P1014
> - Use both getenv() and hwconfig to get USB phy type till getenv()
>   is depricated
> - Introduce CONFIG_SYS_FSL_USB_INTERNAL_UTMI_PHY to specify if soc
>   has internal UTMI phy
> 
> Signed-off-by: Ramneek Mehresh <ramneek.mehr...@freescale.com>
> CC: Remy Bohmer <li...@bohmer.net>
> Signed-off-by: Kumar Gala <ga...@kernel.crashing.org>
> ---
> * Add whitespace and formatting fixes

Remy,

Can you just ack this and I'll pull it in via the 85xx tree?

- k

> 
> arch/powerpc/include/asm/config_mpc85xx.h |    2 +
> drivers/usb/host/ehci-fsl.c               |   39 ++++++++++++++++++++++++++--
> 2 files changed, 38 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/powerpc/include/asm/config_mpc85xx.h 
> b/arch/powerpc/include/asm/config_mpc85xx.h
> index 04ca989..d9d04e7 100644
> --- a/arch/powerpc/include/asm/config_mpc85xx.h
> +++ b/arch/powerpc/include/asm/config_mpc85xx.h
> @@ -97,6 +97,7 @@
> #define CONFIG_NUM_DDR_CONTROLLERS    1
> #define CONFIG_SYS_CCSRBAR_DEFAULT    0xff700000
> #define CONFIG_SYS_FSL_PCIE_COMPAT    "fsl,qoriq-pcie-v2.2"
> +#define CONFIG_SYS_FSL_USB_INTERNAL_UTMI_PHY
> 
> /* P1011 is single core version of P1020 */
> #elif defined(CONFIG_P1011)
> @@ -141,6 +142,7 @@
> #define CONFIG_SYS_FSL_ERRATUM_ESDHC111
> #define CONFIG_NUM_DDR_CONTROLLERS    1
> #define CONFIG_SYS_CCSRBAR_DEFAULT    0xff700000
> +#define CONFIG_SYS_FSL_USB_INTERNAL_UTMI_PHY
> 
> /* P1015 is single core version of P1024 */
> #elif defined(CONFIG_P1015)
> diff --git a/drivers/usb/host/ehci-fsl.c b/drivers/usb/host/ehci-fsl.c
> index 6e0043a..5a65d92 100644
> --- a/drivers/usb/host/ehci-fsl.c
> +++ b/drivers/usb/host/ehci-fsl.c
> @@ -1,5 +1,5 @@
> /*
> - * (C) Copyright 2009 Freescale Semiconductor, Inc.
> + * (C) Copyright 2009, 2011 Freescale Semiconductor, Inc.
>  *
>  * (C) Copyright 2008, Excito Elektronik i Sk=E5ne AB
>  *
> @@ -26,6 +26,7 @@
> #include <usb.h>
> #include <asm/io.h>
> #include <usb/ehci-fsl.h>
> +#include <hwconfig.h>
> 
> #include "ehci.h"
> #include "ehci-core.h"
> @@ -39,6 +40,11 @@
> int ehci_hcd_init(void)
> {
>       struct usb_ehci *ehci;
> +     char usb_phy[5];
> +     const char *phy_type = NULL;
> +     size_t len;
> +
> +     usb_phy[0] = '\0';
> 
>       ehci = (struct usb_ehci *)CONFIG_SYS_FSL_USB_ADDR;
>       hccr = (struct ehci_hccr *)((uint32_t)&ehci->caplength);
> @@ -52,10 +58,37 @@ int ehci_hcd_init(void)
>       out_be32(&ehci->snoop2, 0x80000000 | SNOOP_SIZE_2GB);
> 
>       /* Init phy */
> -     if (!strcmp(getenv("usb_phy_type"), "utmi"))
> -             out_le32(&(hcor->or_portsc[0]), PORT_PTS_UTMI);
> +     if (hwconfig_sub("usb1", "phy_type"))
> +             phy_type = hwconfig_subarg("usb1", "phy_type", &len);
>       else
> +             phy_type = getenv("usb_phy_type");
> +
> +     if (!phy_type) {
> +#ifdef CONFIG_SYS_FSL_USB_INTERNAL_UTMI_PHY
> +             /* if none specified assume internal UTMI */
> +             strcpy(usb_phy, "utmi");
> +             phy_type = usb_phy;
> +#else
> +             printf("WARNING: USB phy type not defined !!\n");
> +             return -1;
> +#endif
> +     }
> +
> +     if (!strcmp(phy_type, "utmi")) {
> +#if defined(CONFIG_SYS_FSL_USB_INTERNAL_UTMI_PHY)
> +             setbits_be32(&ehci->control, PHY_CLK_SEL_UTMI);
> +             setbits_be32(&ehci->control, UTMI_PHY_EN);
> +             udelay(1000); /* delay required for PHY Clk to appear */
> +#endif
> +             out_le32(&(hcor->or_portsc[0]), PORT_PTS_UTMI);
> +     } else {
> +#if defined(CONFIG_SYS_FSL_USB_INTERNAL_UTMI_PHY)
> +             clrbits_be32(&ehci->control, UTMI_PHY_EN);
> +             setbits_be32(&ehci->control, PHY_CLK_SEL_ULPI);
> +             udelay(1000); /* delay required for PHY Clk to appear */
> +#endif
>               out_le32(&(hcor->or_portsc[0]), PORT_PTS_ULPI);
> +     }
> 
>       /* Enable interface. */
>       setbits_be32(&ehci->control, USB_EN);
> -- 
> 1.7.3.4
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

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

Reply via email to