2014-02-21 16:44:19

by Sylwester Nawrocki

[permalink] [raw]
Subject: [PATCH v3 1/2] phy: Select PHY_EXYNOS_MIPI_VIDEO by default for ARCH_EXYNOS

Instead of requiring user to figure out when PHY_EXYNOS_MIPI_VIDEO needs
to be selected select it by default for Exynos SoCs. Also enable it when
COMPILE_TEST is selected. If required the MIPI CSIS/DPHY driver can be
then disabled or compiled in as module.

Signed-off-by: Sylwester Nawrocki <[email protected]>
Acked-by: Kyungmin Park <[email protected]>
---
Changes since v2:
- s/default y/default y if ARCH_S5PV210 || ARCH_EXYNOS

Changes since first version:
- use ARCH_EXYNOS instead of "ARCH_EXYNOS4 || ARCH_EXYNOS5"
- add dependency on COMPILE_TEST, and ARCH_S5PV210 for
PHY_EXYNOS_MIPI_VIDEO
---
drivers/phy/Kconfig | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
index afa2354..d71be95 100644
--- a/drivers/phy/Kconfig
+++ b/drivers/phy/Kconfig
@@ -16,8 +16,11 @@ config GENERIC_PHY
framework should select this config.

config PHY_EXYNOS_MIPI_VIDEO
- depends on HAS_IOMEM
tristate "S5P/EXYNOS SoC series MIPI CSI-2/DSI PHY driver"
+ depends on HAS_IOMEM
+ depends on ARCH_S5PV210 || ARCH_EXYNOS || COMPILE_TEST
+ select GENERIC_PHY
+ default y if ARCH_S5PV210 || ARCH_EXYNOS
help
Support for MIPI CSI-2 and MIPI DSI DPHY found on Samsung S5P
and EXYNOS SoCs.
--
1.7.9.5


2014-02-21 16:44:24

by Sylwester Nawrocki

[permalink] [raw]
Subject: [PATCH v3 2/2] phy: Select PHY_EXYNOS_DP_VIDEO by default for ARCH_EXYNOS

Instead of requiring user to figure out when PHY_EXYNOS_DP_VIDEO needs
to be selected select it by default for Exynos SoCs. Also enable it
when COMPILE_TEST is selected. If required the display port PHY driver
can be then disabled or compiled in as module.

Cc: Jingoo Han <[email protected]>
Signed-off-by: Sylwester Nawrocki <[email protected]>
Acked-by: Kyungmin Park <[email protected]>
---
Changes since v2:
- s/default y/default ARCH_EXYNOS

Changes since first version:
- use ARCH_EXYNOS instead of "ARCH_EXYNOS4 || ARCH_EXYNOS5"
- add dependency on COMPILE_TEST
---
drivers/phy/Kconfig | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
index d71be95..fe1b1f4 100644
--- a/drivers/phy/Kconfig
+++ b/drivers/phy/Kconfig
@@ -57,6 +57,8 @@ config TWL4030_USB
config PHY_EXYNOS_DP_VIDEO
tristate "EXYNOS SoC series Display Port PHY driver"
depends on OF
+ depends on ARCH_EXYNOS || COMPILE_TEST
+ default ARCH_EXYNOS
select GENERIC_PHY
help
Support for Display Port PHY found on Samsung EXYNOS SoCs.
--
1.7.9.5

2014-02-23 23:46:10

by Jingoo Han

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] phy: Select PHY_EXYNOS_DP_VIDEO by default for ARCH_EXYNOS

On Saturday, February 22, 2014 1:44 AM, Sylwester Nawrocki wrote:
>
> Instead of requiring user to figure out when PHY_EXYNOS_DP_VIDEO needs
> to be selected select it by default for Exynos SoCs. Also enable it
> when COMPILE_TEST is selected. If required the display port PHY driver
> can be then disabled or compiled in as module.
>
> Cc: Jingoo Han <[email protected]>

Acked-by: Jingoo Han <[email protected]>

Best regards,
Jingoo Han

> Signed-off-by: Sylwester Nawrocki <[email protected]>
> Acked-by: Kyungmin Park <[email protected]>
> ---
> Changes since v2:
> - s/default y/default ARCH_EXYNOS
>
> Changes since first version:
> - use ARCH_EXYNOS instead of "ARCH_EXYNOS4 || ARCH_EXYNOS5"
> - add dependency on COMPILE_TEST
> ---
> drivers/phy/Kconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
> index d71be95..fe1b1f4 100644
> --- a/drivers/phy/Kconfig
> +++ b/drivers/phy/Kconfig
> @@ -57,6 +57,8 @@ config TWL4030_USB
> config PHY_EXYNOS_DP_VIDEO
> tristate "EXYNOS SoC series Display Port PHY driver"
> depends on OF
> + depends on ARCH_EXYNOS || COMPILE_TEST
> + default ARCH_EXYNOS
> select GENERIC_PHY
> help
> Support for Display Port PHY found on Samsung EXYNOS SoCs.
> --
> 1.7.9.5????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?