diff mbox

[V9,13/14] ARM: dts: snow: represent the connection between bridge and panel using videoport and endpoints

Message ID 1421771935-31618-14-git-send-email-ajaykumar.rs@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ajay Kumar Jan. 20, 2015, 4:38 p.m. UTC
Define videoports and use endpoints to describe the connection between
the encoder, bridge and the panel, instead of using phandles.

Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
Acked-by: Inki Dae <inki.dae@samsung.com>
Tested-by: Rahul Sharma <rahul.sharma@samsung.com>
Tested-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
Tested-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Tested-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
---
 arch/arm/boot/dts/exynos5250-snow.dts |   30 ++++++++++++++++++++++++++++--
 1 file changed, 28 insertions(+), 2 deletions(-)

Comments

kgene@kernel.org Jan. 23, 2015, 7:01 a.m. UTC | #1
Ajay Kumar wrote:
> 
> Define videoports and use endpoints to describe the connection between
> the encoder, bridge and the panel, instead of using phandles.
> 
> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
> Acked-by: Inki Dae <inki.dae@samsung.com>
> Tested-by: Rahul Sharma <rahul.sharma@samsung.com>
> Tested-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
> Tested-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> Tested-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> ---
>  arch/arm/boot/dts/exynos5250-snow.dts |   30 ++++++++++++++++++++++++++++--
>  1 file changed, 28 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
> index b9aeec4..1bd5d3a 100644
> --- a/arch/arm/boot/dts/exynos5250-snow.dts
> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
> @@ -183,7 +183,20 @@
>  			powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>;
>  			reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>;
>  			edid-emulation = <5>;
> -			panel = <&panel>;
> +
> +			ports {
> +				port@0 {
> +					bridge_out: endpoint {
> +						remote-endpoint = <&panel_in>;
> +					};
> +				};
> +
> +				port@1 {
> +					bridge_in: endpoint {
> +						remote-endpoint = <&dp_out>;
> +					};
> +				};
> +			};
>  		};
>  	};
> 
> @@ -228,6 +241,12 @@
>  		compatible = "auo,b116xw03";
>  		power-supply = <&fet6>;
>  		backlight = <&backlight>;
> +
> +		port {
> +			panel_in: endpoint {
> +				remote-endpoint = <&bridge_out>;
> +			};
> +		};
>  	};
>  };
> 
> @@ -242,7 +261,14 @@
>  	samsung,link-rate = <0x0a>;
>  	samsung,lane-count = <2>;
>  	samsung,hpd-gpio = <&gpx0 7 GPIO_ACTIVE_HIGH>;
> -	bridge = <&ptn3460>;
> +
> +	ports {
> +		port@0 {
> +			dp_out: endpoint {
> +				remote-endpoint = <&bridge_in>;
> +			};
> +		};
> +	};
>  };
> 
>  &ehci {
> --
> 1.7.9.5

I'm fine on the DT changes in this series, shall I take 13/14 and 14/14 in
Samsung tree?

- Kukjin
Ajay kumar Jan. 27, 2015, 4:27 a.m. UTC | #2
Hi Kukjin,

On Fri, Jan 23, 2015 at 12:31 PM, Kukjin Kim <kgene@kernel.org> wrote:
> Ajay Kumar wrote:
>>
>> Define videoports and use endpoints to describe the connection between
>> the encoder, bridge and the panel, instead of using phandles.
>>
>> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
>> Acked-by: Inki Dae <inki.dae@samsung.com>
>> Tested-by: Rahul Sharma <rahul.sharma@samsung.com>
>> Tested-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
>> Tested-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
>> Tested-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
>> ---
>>  arch/arm/boot/dts/exynos5250-snow.dts |   30 ++++++++++++++++++++++++++++--
>>  1 file changed, 28 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
>> index b9aeec4..1bd5d3a 100644
>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>> @@ -183,7 +183,20 @@
>>                       powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>;
>>                       reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>;
>>                       edid-emulation = <5>;
>> -                     panel = <&panel>;
>> +
>> +                     ports {
>> +                             port@0 {
>> +                                     bridge_out: endpoint {
>> +                                             remote-endpoint = <&panel_in>;
>> +                                     };
>> +                             };
>> +
>> +                             port@1 {
>> +                                     bridge_in: endpoint {
>> +                                             remote-endpoint = <&dp_out>;
>> +                                     };
>> +                             };
>> +                     };
>>               };
>>       };
>>
>> @@ -228,6 +241,12 @@
>>               compatible = "auo,b116xw03";
>>               power-supply = <&fet6>;
>>               backlight = <&backlight>;
>> +
>> +             port {
>> +                     panel_in: endpoint {
>> +                             remote-endpoint = <&bridge_out>;
>> +                     };
>> +             };
>>       };
>>  };
>>
>> @@ -242,7 +261,14 @@
>>       samsung,link-rate = <0x0a>;
>>       samsung,lane-count = <2>;
>>       samsung,hpd-gpio = <&gpx0 7 GPIO_ACTIVE_HIGH>;
>> -     bridge = <&ptn3460>;
>> +
>> +     ports {
>> +             port@0 {
>> +                     dp_out: endpoint {
>> +                             remote-endpoint = <&bridge_in>;
>> +                     };
>> +             };
>> +     };
>>  };
>>
>>  &ehci {
>> --
>> 1.7.9.5
>
> I'm fine on the DT changes in this series, shall I take 13/14 and 14/14 in
> Samsung tree?
Yes, please take these patches.

Thanks and regards,
Ajay Kumar
kgene@kernel.org Feb. 4, 2015, 3:30 p.m. UTC | #3
On 01/27/15 13:27, Ajay kumar wrote:
> Hi Kukjin,
> 
Hi,

[...]
>>
>> I'm fine on the DT changes in this series, shall I take 13/14 and 14/14 in
>> Samsung tree?
> Yes, please take these patches.
> 
Done, thanks.

- Kukjin
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
index b9aeec4..1bd5d3a 100644
--- a/arch/arm/boot/dts/exynos5250-snow.dts
+++ b/arch/arm/boot/dts/exynos5250-snow.dts
@@ -183,7 +183,20 @@ 
 			powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>;
 			reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>;
 			edid-emulation = <5>;
-			panel = <&panel>;
+
+			ports {
+				port@0 {
+					bridge_out: endpoint {
+						remote-endpoint = <&panel_in>;
+					};
+				};
+
+				port@1 {
+					bridge_in: endpoint {
+						remote-endpoint = <&dp_out>;
+					};
+				};
+			};
 		};
 	};
 
@@ -228,6 +241,12 @@ 
 		compatible = "auo,b116xw03";
 		power-supply = <&fet6>;
 		backlight = <&backlight>;
+
+		port {
+			panel_in: endpoint {
+				remote-endpoint = <&bridge_out>;
+			};
+		};
 	};
 };
 
@@ -242,7 +261,14 @@ 
 	samsung,link-rate = <0x0a>;
 	samsung,lane-count = <2>;
 	samsung,hpd-gpio = <&gpx0 7 GPIO_ACTIVE_HIGH>;
-	bridge = <&ptn3460>;
+
+	ports {
+		port@0 {
+			dp_out: endpoint {
+				remote-endpoint = <&bridge_in>;
+			};
+		};
+	};
 };
 
 &ehci {