[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
2015ë 12ì 08ì¼ 10:47ì Krzysztof Kozlowski ì´(ê°) ì´ ê¸: > On 08.12.2015 10:33, Javier Martinez Canillas wrote: >> Hello Krzysztof, >> >> On 12/07/2015 09:45 PM, Krzysztof Kozlowski wrote: >>> On 07.12.2015 21:52, Inki Dae wrote: From: Javier Martinez Canillas 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 Tested-by: Javier Martinez Canillas Reviewed-by: Inki Dae --- arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) >>> >>> Looks sensible: >>> Reviewed-by: Krzysztof Kozlowski >>> >>> Dependencies are not mentioned, does it depend on patch 1? >>> >> >> Yes, it depends on patch 1/4 so it should be merged through the Exynos DRM >> tree to maintain bisectability. Inki's patch maintains the DT ABI backward >> compatibility though so another option is to wait until the DRM change hit >> mainline and then pick $SUBJECT. > > Thanks. We could also use a tag with DRM changes for samsung-soc but > since I already flushed my queue for v4.5 I think it would be an > overkill. From my point of view it can safely go through exynos-drm tree. I will pick it up to exynos-drm tree. :) Thanks, Inki Dae > > Best regards, > Krzysztof > >
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
2015ë 12ì 08ì¼ 09:48ì Krzysztof Kozlowski ì´(ê°) ì´ ê¸: > On 08.12.2015 00:36, Inki Dae wrote: >> Hi Javier, >> >> 2015-12-07 22:41 GMT+09:00 Javier Martinez Canillas > osg.samsung.com>: >>> Hello Inki, >>> >>> On 12/07/2015 09:52 AM, Inki Dae wrote: From: Javier Martinez Canillas >>> >>> Thanks a lot for posting this patch. >>> 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 Tested-by: Javier Martinez Canillas >>> >>> This tag was not in my original patch, it's true that I tested >>> it but will someone believe me? ;) >> >> Oops. I confused you spread Reviewed-by and Tested-by here and there. >> Don't worry about that. Will remove it if you don't give me Tested-by. >> :) > > Actually authorship (the "From") in this case means Tested-by. Author > always tests the patch so it would look weird if we start adding > tested-by to our own patches, right? > > Dear Inki, > However the patch misses your SoB. You touched and sent it so please > extend the SoB chain-of-blame. Right. Missed. Thanks, Inki Dae > > Best regards, > Krzysztof > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > >
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
On 08.12.2015 10:33, Javier Martinez Canillas wrote: > Hello Krzysztof, > > On 12/07/2015 09:45 PM, Krzysztof Kozlowski wrote: >> On 07.12.2015 21:52, Inki Dae wrote: >>> From: Javier Martinez Canillas >>> >>> 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 >>> Tested-by: Javier Martinez Canillas >>> Reviewed-by: Inki Dae >>> --- >>> arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++- >>> 1 file changed, 14 insertions(+), 1 deletion(-) >>> >> >> Looks sensible: >> Reviewed-by: Krzysztof Kozlowski >> >> Dependencies are not mentioned, does it depend on patch 1? >> > > Yes, it depends on patch 1/4 so it should be merged through the Exynos DRM > tree to maintain bisectability. Inki's patch maintains the DT ABI backward > compatibility though so another option is to wait until the DRM change hit > mainline and then pick $SUBJECT. Thanks. We could also use a tag with DRM changes for samsung-soc but since I already flushed my queue for v4.5 I think it would be an overkill. From my point of view it can safely go through exynos-drm tree. Best regards, Krzysztof
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
On 08.12.2015 00:36, Inki Dae wrote: > Hi Javier, > > 2015-12-07 22:41 GMT+09:00 Javier Martinez Canillas osg.samsung.com>: >> Hello Inki, >> >> On 12/07/2015 09:52 AM, Inki Dae wrote: >>> From: Javier Martinez Canillas >>> >> >> Thanks a lot for posting this patch. >> >>> 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 >>> Tested-by: Javier Martinez Canillas >> >> This tag was not in my original patch, it's true that I tested >> it but will someone believe me? ;) > > Oops. I confused you spread Reviewed-by and Tested-by here and there. > Don't worry about that. Will remove it if you don't give me Tested-by. > :) Actually authorship (the "From") in this case means Tested-by. Author always tests the patch so it would look weird if we start adding tested-by to our own patches, right? Dear Inki, However the patch misses your SoB. You touched and sent it so please extend the SoB chain-of-blame. Best regards, Krzysztof
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
On 07.12.2015 21:52, Inki Dae wrote: > From: Javier Martinez Canillas > > 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 > Tested-by: Javier Martinez Canillas > Reviewed-by: Inki Dae > --- > arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++- > 1 file changed, 14 insertions(+), 1 deletion(-) > Looks sensible: Reviewed-by: Krzysztof Kozlowski Dependencies are not mentioned, does it depend on patch 1? Best regards, Krzysztof
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
Hi Javier, 2015-12-07 22:41 GMT+09:00 Javier Martinez Canillas : > Hello Inki, > > On 12/07/2015 09:52 AM, Inki Dae wrote: >> From: Javier Martinez Canillas >> > > Thanks a lot for posting this patch. > >> 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 >> Tested-by: Javier Martinez Canillas > > This tag was not in my original patch, it's true that I tested > it but will someone believe me? ;) Oops. I confused you spread Reviewed-by and Tested-by here and there. Don't worry about that. Will remove it if you don't give me Tested-by. :) Thanks, Inki Dae > >> Reviewed-by: Inki Dae > > Thanks for the review. > > Best regards, > -- > Javier Martinez Canillas > Open Source Group > Samsung Research America > ___ > dri-devel mailing list > dri-devel at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
Hello Krzysztof, On 12/07/2015 09:48 PM, Krzysztof Kozlowski wrote: > On 08.12.2015 00:36, Inki Dae wrote: >> Hi Javier, >> >> 2015-12-07 22:41 GMT+09:00 Javier Martinez Canillas > osg.samsung.com>: >>> Hello Inki, >>> >>> On 12/07/2015 09:52 AM, Inki Dae wrote: From: Javier Martinez Canillas >>> >>> Thanks a lot for posting this patch. >>> 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 Tested-by: Javier Martinez Canillas >>> >>> This tag was not in my original patch, it's true that I tested >>> it but will someone believe me? ;) >> >> Oops. I confused you spread Reviewed-by and Tested-by here and there. >> Don't worry about that. Will remove it if you don't give me Tested-by. >> :) > > Actually authorship (the "From") in this case means Tested-by. Author > always tests the patch so it would look weird if we start adding > tested-by to our own patches, right? > Exactly, that's what I tried to say. It's implied that the author tested her/his own patch in the best possible way. > Dear Inki, > However the patch misses your SoB. You touched and sent it so please > extend the SoB chain-of-blame. > Right, I missed that. > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
Hello Krzysztof, On 12/07/2015 09:45 PM, Krzysztof Kozlowski wrote: > On 07.12.2015 21:52, Inki Dae wrote: >> From: Javier Martinez Canillas >> >> 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 >> Tested-by: Javier Martinez Canillas >> Reviewed-by: Inki Dae >> --- >> arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++- >> 1 file changed, 14 insertions(+), 1 deletion(-) >> > > Looks sensible: > Reviewed-by: Krzysztof Kozlowski > > Dependencies are not mentioned, does it depend on patch 1? > Yes, it depends on patch 1/4 so it should be merged through the Exynos DRM tree to maintain bisectability. Inki's patch maintains the DT ABI backward compatibility though so another option is to wait until the DRM change hit mainline and then pick $SUBJECT. > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
From: Javier Martinez Canillas
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
Tested-by: Javier Martinez Canillas
Reviewed-by: Inki Dae
---
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 49a4f43..1cc2e95 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 {
+ dp_out: endpoint {
+ remote-endpoint = <&panel_in>;
+ };
+ };
+ };
};
&fimd {
--
1.9.1
[PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
Hello Inki, On 12/07/2015 09:52 AM, Inki Dae wrote: > From: Javier Martinez Canillas > Thanks a lot for posting this patch. > 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 > Tested-by: Javier Martinez Canillas This tag was not in my original patch, it's true that I tested it but will someone believe me? ;) > Reviewed-by: Inki Dae Thanks for the review. Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America
