Vikas,
On Mon, Jul 29, 2013 at 7:19 PM, Vikas Sajjan vikas.sajjan@linaro.orgwrote:
Moves the properties of DP controller to exynos5.dtsi which are common across exynos5 SoCs like Exynos5250 and Exynos5420.
The PHY DP Node is based on Jingoo Han's jg1.han@samsung.com patch at https://patchwork.linuxtv.org/patch/19189/
Signed-off-by: Vikas Sajjan vikas.sajjan@linaro.org
arch/arm/boot/dts/exynos5.dtsi | 29 +++++++++++++++++++++++++++++ arch/arm/boot/dts/exynos5250-arndale.dts | 9 --------- arch/arm/boot/dts/exynos5250-smdk5250.dts | 7 ------- arch/arm/boot/dts/exynos5250.dtsi | 12 +----------- 4 files changed, 30 insertions(+), 27 deletions(-)
diff --git a/arch/arm/boot/dts/exynos5.dtsi b/arch/arm/boot/dts/exynos5.dtsi index 80a4630..682ecf9 100644 --- a/arch/arm/boot/dts/exynos5.dtsi +++ b/arch/arm/boot/dts/exynos5.dtsi @@ -118,4 +118,33 @@ status = "disabled"; };
dp_phy: video-phy@10040720 {
I don't think we can make this common. Because, DPTX_PHY for Exynos5420 falls at address 0x10040728.
compatible = "samsung,exynos5250-dp-video-phy";reg = <0x10040720 4>;#phy-cells = <0>;};dp-controller {compatible = "samsung,exynos5-dp";reg = <0x145b0000 0x1000>;interrupts = <10 3>;interrupt-parent = <&combiner>;#address-cells = <1>;#size-cells = <0>;samsung,color-space = <0>;samsung,dynamic-range = <0>;samsung,ycbcr-coeff = <0>;samsung,color-depth = <1>;samsung,link-rate = <0x0a>;samsung,lane-count = <4>;
Only SMDK eDP panels support 4 lane, not all the panels! I think this property is actually dependent on the type of eDP panel the board carries. So, we cannot make this property common across SOC. My comment holds good for the "link-rate" property as well.
phys = <&dp_phy>;phy-names = "dp";status = "disabled";};}; diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts index 96d528d..01eef7d 100644 --- a/arch/arm/boot/dts/exynos5250-arndale.dts +++ b/arch/arm/boot/dts/exynos5250-arndale.dts @@ -509,15 +509,6 @@ }; };
dp-controller {samsung,color-space = <0>;samsung,dynamic-range = <0>;samsung,ycbcr-coeff = <0>;samsung,color-depth = <1>;samsung,link-rate = <0x0a>;samsung,lane-count = <4>;};fimd: fimd@14400000 { display-timings { native-mode = <&timing0>;diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts index 3b32c26..109f9f7 100644 --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts @@ -251,13 +251,6 @@ };
dp-controller {
samsung,color-space = <0>;samsung,dynamic-range = <0>;samsung,ycbcr-coeff = <0>;samsung,color-depth = <1>;samsung,link-rate = <0x0a>;samsung,lane-count = <4>;pinctrl-names = "default"; pinctrl-0 = <&dp_hpd>; };diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi index 20c32fc..3b5bf13 100644 --- a/arch/arm/boot/dts/exynos5250.dtsi +++ b/arch/arm/boot/dts/exynos5250.dtsi @@ -615,19 +615,9 @@ };
dp-controller {
compatible = "samsung,exynos5-dp";reg = <0x145b0000 0x1000>;interrupts = <10 3>;interrupt-parent = <&combiner>; clocks = <&clock 342>; clock-names = "dp";#address-cells = <1>;#size-cells = <0>;dptx-phy {reg = <0x10040720>;samsung,enable-mask = <1>;};
status = "okay"; }; fimd {-- 1.7.9.5
-- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Regards,
Ajay