Patchwork ARM: imx_v6_v7_defconfig: Add Chipidea USB driver support

login
register
mail settings
Submitter Fabio Estevam
Date Aug. 8, 2012, 10:06 p.m.
Message ID <1344463589-8115-1-git-send-email-festevam@gmail.com>
Download mbox | patch
Permalink /patch/175967/
State New
Headers show

Comments

Fabio Estevam - Aug. 8, 2012, 10:06 p.m.
From: Fabio Estevam <fabio.estevam@freescale.com>

Add Chipidea USB driver support.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/configs/imx_v6_v7_defconfig |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
Dirk Behme - Aug. 9, 2012, 5:45 a.m.
On 09.08.2012 00:06, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Add Chipidea USB driver support.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/configs/imx_v6_v7_defconfig |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
> index f725b96..5f6138d 100644
> --- a/arch/arm/configs/imx_v6_v7_defconfig
> +++ b/arch/arm/configs/imx_v6_v7_defconfig
> @@ -179,6 +179,9 @@ CONFIG_SND_SOC_IMX_MC13783=y
>  CONFIG_USB=y
>  CONFIG_USB_EHCI_HCD=y
>  CONFIG_USB_EHCI_MXC=y
> +CONFIG_USB_CHIPIDEA=y
> +CONFIG_USB_CHIPIDEA_HOST=y
> +CONFIG_USB_MXS_PHY=y
>  CONFIG_USB_STORAGE=y
>  CONFIG_MMC=y
>  CONFIG_MMC_SDHCI=y

I'm not sure if we still need

CONFIG_USB_EHCI_HCD=y
CONFIG_USB_EHCI_MXC=y

or if this can be removed? Besides this:

Acked-by: Dirk Behme <dirk.behme@de.bosch.com>

Thanks

Dirk
Sascha Hauer - Aug. 9, 2012, 6:32 a.m.
On Thu, Aug 09, 2012 at 07:45:19AM +0200, Dirk Behme wrote:
> On 09.08.2012 00:06, Fabio Estevam wrote:
> >From: Fabio Estevam <fabio.estevam@freescale.com>
> >
> >Add Chipidea USB driver support.
> >
> >Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> >---
> > arch/arm/configs/imx_v6_v7_defconfig |    3 +++
> > 1 files changed, 3 insertions(+), 0 deletions(-)
> >
> >diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
> >index f725b96..5f6138d 100644
> >--- a/arch/arm/configs/imx_v6_v7_defconfig
> >+++ b/arch/arm/configs/imx_v6_v7_defconfig
> >@@ -179,6 +179,9 @@ CONFIG_SND_SOC_IMX_MC13783=y
> > CONFIG_USB=y
> > CONFIG_USB_EHCI_HCD=y
> > CONFIG_USB_EHCI_MXC=y
> >+CONFIG_USB_CHIPIDEA=y
> >+CONFIG_USB_CHIPIDEA_HOST=y
> >+CONFIG_USB_MXS_PHY=y
> > CONFIG_USB_STORAGE=y
> > CONFIG_MMC=y
> > CONFIG_MMC_SDHCI=y
> 
> I'm not sure if we still need
> 
> CONFIG_USB_EHCI_HCD=y
> CONFIG_USB_EHCI_MXC=y
> 
> or if this can be removed? Besides this:

It is still needed for the non devicetree platforms.

Sascha
Sascha Hauer - Aug. 10, 2012, 9:04 a.m.
On Wed, Aug 08, 2012 at 07:06:29PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Add Chipidea USB driver support.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

Applied, thanks (to the defconfig branch btw)

Sascha

> ---
>  arch/arm/configs/imx_v6_v7_defconfig |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
> index f725b96..5f6138d 100644
> --- a/arch/arm/configs/imx_v6_v7_defconfig
> +++ b/arch/arm/configs/imx_v6_v7_defconfig
> @@ -179,6 +179,9 @@ CONFIG_SND_SOC_IMX_MC13783=y
>  CONFIG_USB=y
>  CONFIG_USB_EHCI_HCD=y
>  CONFIG_USB_EHCI_MXC=y
> +CONFIG_USB_CHIPIDEA=y
> +CONFIG_USB_CHIPIDEA_HOST=y
> +CONFIG_USB_MXS_PHY=y
>  CONFIG_USB_STORAGE=y
>  CONFIG_MMC=y
>  CONFIG_MMC_SDHCI=y
> -- 
> 1.7.1
> 
>

Patch

diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index f725b96..5f6138d 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -179,6 +179,9 @@  CONFIG_SND_SOC_IMX_MC13783=y
 CONFIG_USB=y
 CONFIG_USB_EHCI_HCD=y
 CONFIG_USB_EHCI_MXC=y
+CONFIG_USB_CHIPIDEA=y
+CONFIG_USB_CHIPIDEA_HOST=y
+CONFIG_USB_MXS_PHY=y
 CONFIG_USB_STORAGE=y
 CONFIG_MMC=y
 CONFIG_MMC_SDHCI=y