Re: [PATCH v2 1/2] drm/exynos: dp: add of_graph dt binding support for panel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Inki,

On 12/02/2015 11:11 PM, Inki Dae wrote:
> Hi Javier,
> 
> 2015년 12월 03일 00:04에 Javier Martinez Canillas 이(가) 쓴 글:
>> Hello Inki,
>>
>> On 12/02/2015 08:57 AM, Inki Dae wrote:
>>> This patch adds of_graph dt binding support for panel device
>>> and also keeps the backward compatibility.
>>>
>>
>> You have to also update the DT binding doc which seems to be
>> outdated already:
>>
>> Documentation/devicetree/bindings/display/exynos/exynos_dp.txt
> 
> Right. It should be updated.
>

Great, I see you already posted that.

>>
>>> i.e.,
>>> The dts file for Exynos5800 based peach pi board
>>> has a panel property so we need to keep the backward compatibility.
>>>
>>
>> How did you test this patch?
> 
> I thought you will test it and give me tested-by because you commented like below,
> " Assuming you can make a distinction if the endpoint is a panel or a bridge,
> then yes, I agree with the idea of the patch. Please feel free to cc me if
> you post such a patch and I'll gladly test it on my Exynos5800 Peach Pi."
> 
> That is why I cced you. I really have no any Exynos5800 Peach Pi board.
>

Yes, but if you didn't test a patch, then it should be marked with a RFT
prefix in the subject line or at least mention that needs testing since
you lack the HW to test. I've no way to know if you have another board
with a similar design :)

But what I meant is how the patch is supposed to be tested since there
ins't a change in the Exynos5800 Peach Pi DTS? We can of course test
that doesn't break backward compatibility but we don't have a way to
test the actual change.

So I tested with the patch following patch [0] and things are working
correctly. Please include that patch in your series.

I've some comments on your patch though but I'll comment on your lastest
version.

> Thanks,
> Inki Dae
> 
>>  
>> Best regards,
>>

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

[0]:
>From 644bab7949ac17a8d42ca0cf36cd55d61bc88928 Mon Sep 17 00:00:00 2001
From: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
Date: Thu, 3 Dec 2015 09:32:17 -0300
Subject: [PATCH 1/1] ARM: dts: Use OF graph for DP to panel connection in
 exynos5800-peach-pi

The DT binding for the Exynos DRM Display Port (DP) driver isn't consistent
since it uses a phandle to describe the connection between the DP port and
the display panel but uses the OF graph ports and endpoints to describe the
connection betwen the DP port, a bridge chip and the panel.

The Exynos DP driver and the DT binding have been changed to allow also to
describe the DP port to panel connection using ports / endpoints (OF graph)
so this patch changes the Exynos5800 Peach Pi DT to make it consistent with
the Exynos5420 Peach Pit that has a eDP to LVDS chip and uses OF graph too.

Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
---
 arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
index 7b018e451880..9c6fd7314ee0 100644
--- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
+++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
@@ -122,6 +122,12 @@
 		compatible = "auo,b133htn01";
 		power-supply = <&tps65090_fet6>;
 		backlight = <&backlight>;
+
+		port {
+			panel_in: endpoint {
+				remote-endpoint = <&dp_out>;
+			};
+		};
 	};
 
 	mmc1_pwrseq: mmc1_pwrseq {
@@ -148,7 +154,14 @@
 	samsung,link-rate = <0x0a>;
 	samsung,lane-count = <2>;
 	samsung,hpd-gpio = <&gpx2 6 GPIO_ACTIVE_HIGH>;
-	panel = <&panel>;
+
+	ports {
+		port@0 {
+			dp_out: endpoint {
+				remote-endpoint = <&panel_in>;
+			};
+		};
+	};
 };
 
 &fimd {
-- 
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux