Linux Samsung SOC development
 help / color / mirror / Atom feed
From: Tushar Behera <trblinux@gmail.com>
To: Sachin Kamat <sachin.kamat@samsung.com>, linux-usb@vger.kernel.org
Cc: linux-samsung-soc@vger.kernel.org, kgene.kim@samsung.com,
	balbi@ti.com, gregkh@linuxfoundation.org, dianders@chromium.org,
	gautam.vivek@samsung.com, spk.linux@gmail.com,
	Kishon Vijay Abraham I <kishon@ti.com>
Subject: Re: [PATCH 2/2] usb: dwc3: Kconfig: Select PHY drivers for Exynos DWC3
Date: Thu, 26 Jun 2014 11:26:28 +0530	[thread overview]
Message-ID: <53ABB60C.5060103@gmail.com> (raw)
In-Reply-To: <1403761178-5371-2-git-send-email-sachin.kamat@samsung.com>

On 06/26/2014 11:09 AM, Sachin Kamat wrote:
> USB DWC3 driver on Exynos platforms does not work without its
> corresponding phy driver. Hence it makes no sense to keep the
> phy driver as user selectable. Instead select it from USB config
> to make things easier for the end user.
> While at it make the config depend on ARCH_EXYNOS5 instead of
> ARCH_EXYNOS as this IP is available only on Exynos5 platforms.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> ---

Reviewed-by: Tushar Behera <tushar.b@samsung.com>

>  drivers/phy/Kconfig      |    8 +-------
>  drivers/usb/dwc3/Kconfig |    3 ++-
>  2 files changed, 3 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
> index 7fe7ef5f1322..6be353c9dc4c 100644
> --- a/drivers/phy/Kconfig
> +++ b/drivers/phy/Kconfig
> @@ -138,15 +138,9 @@ config PHY_EXYNOS5250_USB2
>  	bool
>  
>  config PHY_EXYNOS5_USBDRD
> -	tristate "Exynos5 SoC series USB DRD PHY driver"
> -	depends on ARCH_EXYNOS5 && OF
> -	depends on HAS_IOMEM
> +	tristate
>  	select GENERIC_PHY
>  	select MFD_SYSCON
> -	help
> -	  Enable USB DRD PHY support for Exynos 5 SoC series.
> -	  This driver provides PHY interface for USB 3.0 DRD controller
> -	  present on Exynos5 SoC series.
>  
>  config PHY_XGENE
>  	tristate "APM X-Gene 15Gbps PHY support"
> diff --git a/drivers/usb/dwc3/Kconfig b/drivers/usb/dwc3/Kconfig
> index 261c3b428220..501477452ad8 100644
> --- a/drivers/usb/dwc3/Kconfig
> +++ b/drivers/usb/dwc3/Kconfig
> @@ -55,7 +55,8 @@ config USB_DWC3_OMAP
>  
>  config USB_DWC3_EXYNOS
>  	tristate "Samsung Exynos Platform"
> -	depends on ARCH_EXYNOS || COMPILE_TEST
> +	depends on ARCH_EXYNOS5 || COMPILE_TEST
> +	select PHY_EXYNOS5_USBDRD
>  	default USB_DWC3
>  	help
>  	  Recent Exynos5 SoCs ship with one DesignWare Core USB3 IP inside,
> 


-- 
Tushar Behera

  reply	other threads:[~2014-06-26  5:56 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-26  5:39 [PATCH 1/2] usb: host: Kconfig: Select PHY drivers for Exynos EHCI/OHCI Sachin Kamat
2014-06-26  5:55 ` Tushar Behera
2014-06-26  8:09 ` Vivek Gautam
2014-06-26  8:25   ` Sachin Kamat
     [not found]     ` <CAK5sBcHL0f+UVFoyQT3O8m+NAv3rNjiq=aEH4xh9BV3tFbYcSg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-26  8:53       ` Sachin Kamat
2014-06-26  9:07         ` Vivek Gautam
     [not found] ` <1403761178-5371-1-git-send-email-sachin.kamat-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-06-26  5:39   ` [PATCH 2/2] usb: dwc3: Kconfig: Select PHY drivers for Exynos DWC3 Sachin Kamat
2014-06-26  5:56     ` Tushar Behera [this message]
2014-06-26  8:49   ` [PATCH 1/2] usb: host: Kconfig: Select PHY drivers for Exynos EHCI/OHCI Kishon Vijay Abraham I
2014-06-26  9:02     ` Sachin Kamat
2014-06-27 15:46 ` Felipe Balbi
2014-06-27 15:55   ` Doug Anderson
2014-06-27 15:59     ` Felipe Balbi
     [not found]       ` <20140627155900.GN8069-HgARHv6XitL9zxVx7UNMDg@public.gmane.org>
2014-06-27 16:32         ` Doug Anderson
2014-06-27 16:53           ` Olof Johansson
2014-06-30  8:32             ` Sachin Kamat

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53ABB60C.5060103@gmail.com \
    --to=trblinux@gmail.com \
    --cc=balbi@ti.com \
    --cc=dianders@chromium.org \
    --cc=gautam.vivek@samsung.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=kgene.kim@samsung.com \
    --cc=kishon@ti.com \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=sachin.kamat@samsung.com \
    --cc=spk.linux@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox