On Thursday, April 23, 2015 at 07:02:54 PM, Ramneek Mehresh wrote:
> Add xhci driver support for all FSL socs
> 
> Signed-off-by: Ramneek Mehresh <ramneek.mehr...@freescale.com>

Next time, please follow

http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions

> ---
>  arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h |   6 ++
>  drivers/usb/host/Makefile                         |   1 +
>  drivers/usb/host/xhci-fsl.c                       | 107
> ++++++++++++++++++++++ include/linux/usb/xhci-fsl.h                      |
>  54 +++++++++++ 4 files changed, 168 insertions(+)
>  create mode 100644 drivers/usb/host/xhci-fsl.c
>  create mode 100644 include/linux/usb/xhci-fsl.h
> 
> diff --git a/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h
> b/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h index 3a64afc..9c1f1ce
> 100644
> --- a/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h
> +++ b/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h
> @@ -538,4 +538,10 @@ struct ccsr_cci400 {
>       } pcounter[4];                  /* Performance Counter */
>       u8 res_e004[0x10000 - 0xe004];
>  };
> +
> +/* USB-XHCI */
> +#define FSL_XHCI_BASE 0x3100000
> +#define FSL_OCP1_SCP_BASE 0x4a084c00
> +#define FSL_OTG_WRAPPER_BASE 0x4A020000
> +
>  #endif       /* __ASM_ARCH_LS102XA_IMMAP_H_ */

This file looks like a dumpster . Maybe it's about time for someone
in Freescale to clean that mess up. I don't want you to fix it before
this patchset is applied, but certainly this could use some fixing.

[...]

> +static int fsl_xhci_core_exit(struct fsl_xhci *fsl_xhci)
> +{
> +     /* Currently fsl socs do not support PHY shutdown from
> +      * sw. But this support may be added in future socs.
> +      */

Multiline comment ;-)

> +     return 0;
> +}

[...]

> diff --git a/include/linux/usb/xhci-fsl.h b/include/linux/usb/xhci-fsl.h
> new file mode 100644
> index 0000000..8eaab2c
> --- /dev/null
> +++ b/include/linux/usb/xhci-fsl.h
> @@ -0,0 +1,54 @@
> +/*
> + * Copyright 2015 Freescale Semiconductor, Inc.
> + *
> + * FSL USB HOST xHCI Controller
> + *
> + * Author: Ramneek Mehresh<ramneek.mehr...@freescale.com>
> + *
> + * SPDX-License-Identifier:  GPL-2.0+
> + */
> +
> +#ifndef _ASM_ARCH_XHCI_FSL_H_
> +#define _ASM_ARCH_XHCI_FSL_H_
> +
> +/* Default to the FSL XHCI defines */
> +#define USB3_PWRCTL_CLK_CMD_MASK     0x3FE000
> +#define USB3_PWRCTL_CLK_FREQ_MASK    0xFFC
> +#define USB3_PHY_PARTIAL_RX_POWERON     BIT(6)
> +#define USB3_PHY_RX_POWERON          BIT(14)
> +#define USB3_PHY_TX_POWERON          BIT(15)
> +#define USB3_PHY_TX_RX_POWERON       (USB3_PHY_RX_POWERON | 
USB3_PHY_TX_POWERON)
> +#define USB3_PWRCTL_CLK_CMD_SHIFT   14
> +#define USB3_PWRCTL_CLK_FREQ_SHIFT   22

How much of this is actually DWC3 stuff please ? Or are all
these bits really FSL-specific ?
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to