* [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs
@ 2014-06-30 8:56 Sachin Kamat
2014-06-30 8:56 ` [PATCH 2/2] phy: Kconfig: Update config for Exynos USB DRD Sachin Kamat
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Sachin Kamat @ 2014-06-30 8:56 UTC (permalink / raw)
To: linux-kernel; +Cc: linux-samsung-soc, kishon, dianders, gautam.vivek, spk.linux
Since the USB 2.0 PHYs are required only with EHCI/OHCI USB drivers,
make it depend on them and default to ARCH_EXYNOS as they are meant
for Exynos platforms. Also, make the sub-drivers silent options enabling
them based on the SoC platforms that they are meant to work with. This
will make life easier for end users who do not have any way knowing the
dependencies.
Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
---
drivers/phy/Kconfig | 35 +++++++++++------------------------
1 file changed, 11 insertions(+), 24 deletions(-)
diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
index 16a2f067c242..9aab5dfb0adf 100644
--- a/drivers/phy/Kconfig
+++ b/drivers/phy/Kconfig
@@ -122,43 +122,30 @@ config PHY_SUN4I_USB
config PHY_SAMSUNG_USB2
tristate "Samsung USB 2.0 PHY driver"
+ depends on USB_EHCI_EXYNOS || USB_OHCI_EXYNOS
select GENERIC_PHY
select MFD_SYSCON
+ default ARCH_EXYNOS
help
Enable this to support the Samsung USB 2.0 PHY driver for Samsung
- SoCs. This driver provides the interface for USB 2.0 PHY. Support for
- particular SoCs has to be enabled in addition to this driver. Number
- and type of supported phys depends on the SoC.
+ SoCs. This driver provides the interface for USB 2.0 PHY. Support
+ for particular PHYs will be enabled based on the SoC type in addition
+ to this driver.
config PHY_EXYNOS4210_USB2
- bool "Support for Exynos 4210"
+ bool
depends on PHY_SAMSUNG_USB2
- depends on CPU_EXYNOS4210
- help
- Enable USB PHY support for Exynos 4210. This option requires that
- Samsung USB 2.0 PHY driver is enabled and means that support for this
- particular SoC is compiled in the driver. In case of Exynos 4210 four
- phys are available - device, host, HSIC0 and HSIC1.
+ default CPU_EXYNOS4210
config PHY_EXYNOS4X12_USB2
- bool "Support for Exynos 4x12"
+ bool
depends on PHY_SAMSUNG_USB2
- depends on (SOC_EXYNOS4212 || SOC_EXYNOS4412)
- help
- Enable USB PHY support for Exynos 4x12. This option requires that
- Samsung USB 2.0 PHY driver is enabled and means that support for this
- particular SoC is compiled in the driver. In case of Exynos 4x12 four
- phys are available - device, host, HSIC0 and HSIC1.
+ default SOC_EXYNOS4212 || SOC_EXYNOS4412
config PHY_EXYNOS5250_USB2
- bool "Support for Exynos 5250"
+ bool
depends on PHY_SAMSUNG_USB2
- depends on SOC_EXYNOS5250
- help
- Enable USB PHY support for Exynos 5250. This option requires that
- Samsung USB 2.0 PHY driver is enabled and means that support for this
- particular SoC is compiled in the driver. In case of Exynos 5250 four
- phys are available - device, host, HSIC0 and HSIC.
+ default SOC_EXYNOS5250 || SOC_EXYNOS5420
config PHY_EXYNOS5_USBDRD
tristate "Exynos5 SoC series USB DRD PHY driver"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] phy: Kconfig: Update config for Exynos USB DRD
2014-06-30 8:56 [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs Sachin Kamat
@ 2014-06-30 8:56 ` Sachin Kamat
2014-07-08 4:19 ` [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs Sachin Kamat
2014-07-08 15:45 ` Tomasz Figa
2 siblings, 0 replies; 6+ messages in thread
From: Sachin Kamat @ 2014-06-30 8:56 UTC (permalink / raw)
To: linux-kernel; +Cc: linux-samsung-soc, kishon, dianders, gautam.vivek, spk.linux
USB DWC3 driver on Exynos platform does not work without its
corresponding phy driver. Hence make the PHY driver depend on
Exynos DWC3 driver and default it to yes to make things easier
for the end user.
Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
---
drivers/phy/Kconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
index 9aab5dfb0adf..6f90ee3c7ed1 100644
--- a/drivers/phy/Kconfig
+++ b/drivers/phy/Kconfig
@@ -151,8 +151,10 @@ config PHY_EXYNOS5_USBDRD
tristate "Exynos5 SoC series USB DRD PHY driver"
depends on ARCH_EXYNOS5 && OF
depends on HAS_IOMEM
+ depends on USB_DWC3_EXYNOS
select GENERIC_PHY
select MFD_SYSCON
+ default y
help
Enable USB DRD PHY support for Exynos 5 SoC series.
This driver provides PHY interface for USB 3.0 DRD controller
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs
2014-06-30 8:56 [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs Sachin Kamat
2014-06-30 8:56 ` [PATCH 2/2] phy: Kconfig: Update config for Exynos USB DRD Sachin Kamat
@ 2014-07-08 4:19 ` Sachin Kamat
2014-07-08 15:45 ` Tomasz Figa
2 siblings, 0 replies; 6+ messages in thread
From: Sachin Kamat @ 2014-07-08 4:19 UTC (permalink / raw)
To: open list
Cc: linux-samsung-soc, Kishon Vijay Abraham I, Doug Anderson,
Vivek Gautam, Sachin Kamat
On Mon, Jun 30, 2014 at 2:26 PM, Sachin Kamat <sachin.kamat@samsung.com> wrote:
> Since the USB 2.0 PHYs are required only with EHCI/OHCI USB drivers,
> make it depend on them and default to ARCH_EXYNOS as they are meant
> for Exynos platforms. Also, make the sub-drivers silent options enabling
> them based on the SoC platforms that they are meant to work with. This
> will make life easier for end users who do not have any way knowing the
> dependencies.
>
> Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
> ---
> drivers/phy/Kconfig | 35 +++++++++++------------------------
> 1 file changed, 11 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
> index 16a2f067c242..9aab5dfb0adf 100644
> --- a/drivers/phy/Kconfig
> +++ b/drivers/phy/Kconfig
> @@ -122,43 +122,30 @@ config PHY_SUN4I_USB
>
> config PHY_SAMSUNG_USB2
> tristate "Samsung USB 2.0 PHY driver"
> + depends on USB_EHCI_EXYNOS || USB_OHCI_EXYNOS
> select GENERIC_PHY
> select MFD_SYSCON
> + default ARCH_EXYNOS
> help
> Enable this to support the Samsung USB 2.0 PHY driver for Samsung
> - SoCs. This driver provides the interface for USB 2.0 PHY. Support for
> - particular SoCs has to be enabled in addition to this driver. Number
> - and type of supported phys depends on the SoC.
> + SoCs. This driver provides the interface for USB 2.0 PHY. Support
> + for particular PHYs will be enabled based on the SoC type in addition
> + to this driver.
>
> config PHY_EXYNOS4210_USB2
> - bool "Support for Exynos 4210"
> + bool
> depends on PHY_SAMSUNG_USB2
> - depends on CPU_EXYNOS4210
> - help
> - Enable USB PHY support for Exynos 4210. This option requires that
> - Samsung USB 2.0 PHY driver is enabled and means that support for this
> - particular SoC is compiled in the driver. In case of Exynos 4210 four
> - phys are available - device, host, HSIC0 and HSIC1.
> + default CPU_EXYNOS4210
>
> config PHY_EXYNOS4X12_USB2
> - bool "Support for Exynos 4x12"
> + bool
> depends on PHY_SAMSUNG_USB2
> - depends on (SOC_EXYNOS4212 || SOC_EXYNOS4412)
> - help
> - Enable USB PHY support for Exynos 4x12. This option requires that
> - Samsung USB 2.0 PHY driver is enabled and means that support for this
> - particular SoC is compiled in the driver. In case of Exynos 4x12 four
> - phys are available - device, host, HSIC0 and HSIC1.
> + default SOC_EXYNOS4212 || SOC_EXYNOS4412
>
> config PHY_EXYNOS5250_USB2
> - bool "Support for Exynos 5250"
> + bool
> depends on PHY_SAMSUNG_USB2
> - depends on SOC_EXYNOS5250
> - help
> - Enable USB PHY support for Exynos 5250. This option requires that
> - Samsung USB 2.0 PHY driver is enabled and means that support for this
> - particular SoC is compiled in the driver. In case of Exynos 5250 four
> - phys are available - device, host, HSIC0 and HSIC.
> + default SOC_EXYNOS5250 || SOC_EXYNOS5420
>
> config PHY_EXYNOS5_USBDRD
> tristate "Exynos5 SoC series USB DRD PHY driver"
> --
> 1.7.9.5
Kishon,
Any comments on these patches?
--
Regards,
Sachin.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs
2014-06-30 8:56 [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs Sachin Kamat
2014-06-30 8:56 ` [PATCH 2/2] phy: Kconfig: Update config for Exynos USB DRD Sachin Kamat
2014-07-08 4:19 ` [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs Sachin Kamat
@ 2014-07-08 15:45 ` Tomasz Figa
2014-07-09 3:42 ` Sachin Kamat
2 siblings, 1 reply; 6+ messages in thread
From: Tomasz Figa @ 2014-07-08 15:45 UTC (permalink / raw)
To: Sachin Kamat, linux-kernel
Cc: linux-samsung-soc, kishon, dianders, gautam.vivek, spk.linux
On 30.06.2014 10:56, Sachin Kamat wrote:
> Since the USB 2.0 PHYs are required only with EHCI/OHCI USB drivers,
That's not true. They are also required for USB gadget controller
supported by the DWC2 gadget driver (formerly s3c-hsotg). Otherwise the
change makes sense to me.
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs
2014-07-08 15:45 ` Tomasz Figa
@ 2014-07-09 3:42 ` Sachin Kamat
2014-07-09 4:07 ` Jingoo Han
0 siblings, 1 reply; 6+ messages in thread
From: Sachin Kamat @ 2014-07-09 3:42 UTC (permalink / raw)
To: Tomasz Figa
Cc: open list, linux-samsung-soc, Kishon Vijay Abraham I,
Doug Anderson, Vivek Gautam
Hi Tomasz,
On Tue, Jul 8, 2014 at 9:15 PM, Tomasz Figa <t.figa@samsung.com> wrote:
> On 30.06.2014 10:56, Sachin Kamat wrote:
>> Since the USB 2.0 PHYs are required only with EHCI/OHCI USB drivers,
>
> That's not true. They are also required for USB gadget controller
> supported by the DWC2 gadget driver (formerly s3c-hsotg). Otherwise the
> change makes sense to me.
Do you propose I amend just the patch description or also add a dependency on
USB_DWC2 along with EHCI and OHCI. I do not have anything to test this. Based
on your input I will re-spin the patch.
--
Regards,
Sachin.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs
2014-07-09 3:42 ` Sachin Kamat
@ 2014-07-09 4:07 ` Jingoo Han
0 siblings, 0 replies; 6+ messages in thread
From: Jingoo Han @ 2014-07-09 4:07 UTC (permalink / raw)
To: 'Sachin Kamat', 'Tomasz Figa'
Cc: 'open list', 'linux-samsung-soc',
'Kishon Vijay Abraham I', 'Doug Anderson',
'Vivek Gautam', 'Jingoo Han'
On Wednesday, July 09, 2014 12:42 PM, Sachin Kamat wrote:
> On Tue, Jul 8, 2014 at 9:15 PM, Tomasz Figa <t.figa@samsung.com> wrote:
> > On 30.06.2014 10:56, Sachin Kamat wrote:
> >> Since the USB 2.0 PHYs are required only with EHCI/OHCI USB drivers,
> >
> > That's not true. They are also required for USB gadget controller
> > supported by the DWC2 gadget driver (formerly s3c-hsotg). Otherwise the
> > change makes sense to me.
>
> Do you propose I amend just the patch description or also add a dependency on
> USB_DWC2 along with EHCI and OHCI. I do not have anything to test this. Based
> on your input I will re-spin the patch.
I agree with Tomasz Figa's statement. Personally, I think that it would
be good to amend the commit message and add a dependency on USB_DWC2. :-)
Thank you.
Best regards,
Jingoo Han
>
> --
> Regards,
> Sachin.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-09 4:07 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-30 8:56 [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs Sachin Kamat
2014-06-30 8:56 ` [PATCH 2/2] phy: Kconfig: Update config for Exynos USB DRD Sachin Kamat
2014-07-08 4:19 ` [PATCH 1/2] phy: Kconfig: Re-organize Exynos USB 2.0 PHY configs Sachin Kamat
2014-07-08 15:45 ` Tomasz Figa
2014-07-09 3:42 ` Sachin Kamat
2014-07-09 4:07 ` Jingoo Han
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.