This patch series considers a port node outbound for panel device node, including dt binding for it. And also it fixes a wrong error type.
Changelog v2: - add a patch from Javier, which allows dp to connect panel using of graph. - remove unnecessary properties and numbering pointed out by Rob and Javier. - update description about eDP device.
Thanks, Inki Dae
Inki Dae (3): drm/exynos: dp: add of_graph dt binding support for panel drm/exynos: dp: fix wrong return type dt-bindings: exynos-dp: update ports node binding for panel
Javier Martinez Canillas (1): ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
.../bindings/display/exynos/exynos_dp.txt | 41 +++++++++++++++++++--- arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 +++++++- drivers/gpu/drm/exynos/exynos_dp_core.c | 27 ++++++++++++-- 3 files changed, 75 insertions(+), 8 deletions(-)
This patch adds of_graph dt binding support for panel device and also keeps the backward compatibility.
i.e., The dts file for Exynos5800 based peach pi board has a panel property so we need to keep the backward compatibility.
Changelog v3: - bind only one of two nodes outbound - panel or bridge.
Changelog v2: - return -EINVAL if getting a port node failed.
Signed-off-by: Inki Dae inki.dae@samsung.com Reviewed-by: Javier Martinez Canillas javier@osg.samsung.com --- drivers/gpu/drm/exynos/exynos_dp_core.c | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c b/drivers/gpu/drm/exynos/exynos_dp_core.c index 94f02a0..60260a0 100644 --- a/drivers/gpu/drm/exynos/exynos_dp_core.c +++ b/drivers/gpu/drm/exynos/exynos_dp_core.c @@ -1392,7 +1392,7 @@ static const struct component_ops exynos_dp_ops = { static int exynos_dp_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; - struct device_node *panel_node, *bridge_node, *endpoint; + struct device_node *panel_node = NULL, *bridge_node, *endpoint = NULL; struct exynos_dp_device *dp; int ret;
@@ -1403,14 +1403,32 @@ static int exynos_dp_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, dp);
+ /* This is for the backward compatibility. */ panel_node = of_parse_phandle(dev->of_node, "panel", 0); if (panel_node) { dp->panel = of_drm_find_panel(panel_node); of_node_put(panel_node); if (!dp->panel) return -EPROBE_DEFER; + } else { + endpoint = of_graph_get_next_endpoint(dev->of_node, NULL); + if (endpoint) { + panel_node = of_graph_get_remote_port_parent(endpoint); + if (panel_node) { + dp->panel = of_drm_find_panel(panel_node); + of_node_put(panel_node); + if (!dp->panel) + return -EPROBE_DEFER; + } else { + DRM_ERROR("no port node for panel device.\n"); + return -EINVAL; + } + } }
+ if (endpoint) + goto out; + endpoint = of_graph_get_next_endpoint(dev->of_node, NULL); if (endpoint) { bridge_node = of_graph_get_remote_port_parent(endpoint); @@ -1423,6 +1441,7 @@ static int exynos_dp_probe(struct platform_device *pdev) return -EPROBE_DEFER; }
+out: pm_runtime_enable(dev);
ret = component_add(&pdev->dev, &exynos_dp_ops);
This patch fixes wrong return type when dt binding of bridge device failed.
If a board has a bridge device then of_graph_get_remote_port_parent function shouldn't be NULL. So this patch will return a proper error type so that the deferred probe isn't triggered.
Changelog v2: - return -EINVAL if getting a port node failed.
Signed-off-by: Inki Dae inki.dae@samsung.com Reviewed-by: Javier Martinez Canillas javier@osg.samsung.com --- drivers/gpu/drm/exynos/exynos_dp_core.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c b/drivers/gpu/drm/exynos/exynos_dp_core.c index 60260a0..aeee60a 100644 --- a/drivers/gpu/drm/exynos/exynos_dp_core.c +++ b/drivers/gpu/drm/exynos/exynos_dp_core.c @@ -1437,8 +1437,10 @@ static int exynos_dp_probe(struct platform_device *pdev) of_node_put(bridge_node); if (!dp->ptn_bridge) return -EPROBE_DEFER; - } else - return -EPROBE_DEFER; + } else { + DRM_ERROR("no port node for bridge device.\n"); + return -EINVAL; + } }
out:
This patch updates a ports node binding for panel.
With this, dp node can have a ports node which describes a remote endpoint node that can be connected to panel or bridge node.
Changelog v2: - remove unnecessary properties and numbering. - update description about eDP device.
Signed-off-by: Inki Dae inki.dae@samsung.com Reviewed-by: Javier Martinez Canillas javier@osg.samsung.com --- .../bindings/display/exynos/exynos_dp.txt | 41 +++++++++++++++++++--- 1 file changed, 37 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt b/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt index 64693f2..22efeba 100644 --- a/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt +++ b/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt @@ -1,3 +1,20 @@ +Device-Tree bindings for Samsung Exynos Embedded DisplayPort Transmitter(eDP) + +DisplayPort is industry standard to accommodate the growing board adoption +of digital display technology within the PC and CE industries. +It consolidates the internal and external connection methods to reduce device +complexity and cost. It also supports necessary features for important cross +industry applications and provides performance scalability to enable the next +generation of displays that feature higher color depths, refresh rates, and +display resolutions. + +eDP (embedded display port) device is compliant with Embedded DisplayPort +standard as follows, +- DisplayPort standard 1.1a for Exynos5250 and Exynos5260. +- DisplayPort standard 1.3 for Exynos5422s and Exynos5800. + +eDP resides between FIMD and panel or FIMD and bridge such as LVDS. + The Exynos display port interface should be configured based on the type of panel connected to it.
@@ -66,8 +83,15 @@ Optional properties for dp-controller: Hotplug detect GPIO. Indicates which GPIO should be used for hotplug detection - -video interfaces: Device node can contain video interface port - nodes according to [1]. +Video interfaces: + Device node can contain video interface port nodes according to [1]. + The following are properties specific to those nodes: + + endpoint node connected to bridge or panel node: + - remote-endpoint: specifies the endpoint in panel or bridge node. + This node is required in all kinds of exynos dp + to represent the connection between dp and bridge + or dp and panel.
[1]: Documentation/devicetree/bindings/media/video-interfaces.txt
@@ -111,9 +135,18 @@ Board Specific portion: };
ports { - port@0 { + port { dp_out: endpoint { - remote-endpoint = <&bridge_in>; + remote-endpoint = <&dp_in>; + }; + }; + }; + + panel { + ... + port { + dp_in: endpoint { + remote-endpoint = <&dp_out>; }; }; };
On Mon, Dec 07, 2015 at 09:52:37PM +0900, Inki Dae wrote:
This patch updates a ports node binding for panel.
With this, dp node can have a ports node which describes a remote endpoint node that can be connected to panel or bridge node.
Changelog v2:
- remove unnecessary properties and numbering.
- update description about eDP device.
Signed-off-by: Inki Dae inki.dae@samsung.com Reviewed-by: Javier Martinez Canillas javier@osg.samsung.com
Acked-by: Rob Herring robh@kernel.org
.../bindings/display/exynos/exynos_dp.txt | 41 +++++++++++++++++++--- 1 file changed, 37 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt b/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt index 64693f2..22efeba 100644 --- a/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt +++ b/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt @@ -1,3 +1,20 @@ +Device-Tree bindings for Samsung Exynos Embedded DisplayPort Transmitter(eDP)
+DisplayPort is industry standard to accommodate the growing board adoption +of digital display technology within the PC and CE industries. +It consolidates the internal and external connection methods to reduce device +complexity and cost. It also supports necessary features for important cross +industry applications and provides performance scalability to enable the next +generation of displays that feature higher color depths, refresh rates, and +display resolutions.
+eDP (embedded display port) device is compliant with Embedded DisplayPort +standard as follows, +- DisplayPort standard 1.1a for Exynos5250 and Exynos5260. +- DisplayPort standard 1.3 for Exynos5422s and Exynos5800.
+eDP resides between FIMD and panel or FIMD and bridge such as LVDS.
The Exynos display port interface should be configured based on the type of panel connected to it.
@@ -66,8 +83,15 @@ Optional properties for dp-controller: Hotplug detect GPIO. Indicates which GPIO should be used for hotplug detection
- -video interfaces: Device node can contain video interface port
nodes according to [1].+Video interfaces:
- Device node can contain video interface port nodes according to [1].
- The following are properties specific to those nodes:
- endpoint node connected to bridge or panel node:
- remote-endpoint: specifies the endpoint in panel or bridge node.
This node is required in all kinds of exynos dpto represent the connection between dp and bridgeor dp and panel.@@ -111,9 +135,18 @@ Board Specific portion: };
ports {
port@0 {
port { dp_out: endpoint {
remote-endpoint = <&bridge_in>;
remote-endpoint = <&dp_in>;};};};panel {...port {dp_in: endpoint {remote-endpoint = <&dp_out>; }; };-- 1.9.1
From: Javier Martinez Canillas javier@osg.samsung.com
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@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com Reviewed-by: Inki Dae inki.dae@samsung.com --- 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 {
Hello Inki,
On 12/07/2015 09:52 AM, Inki Dae wrote:
From: Javier Martinez Canillas javier@osg.samsung.com
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 javier@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com
This tag was not in my original patch, it's true that I tested it but will someone believe me? ;)
Reviewed-by: Inki Dae inki.dae@samsung.com
Thanks for the review.
Best regards,
Hi Javier,
2015-12-07 22:41 GMT+09:00 Javier Martinez Canillas javier@osg.samsung.com:
Hello Inki,
On 12/07/2015 09:52 AM, Inki Dae wrote:
From: Javier Martinez Canillas javier@osg.samsung.com
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 javier@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com
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 inki.dae@samsung.com
Thanks for the review.
Best regards,
Javier Martinez Canillas Open Source Group Samsung Research America _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
On 08.12.2015 00:36, Inki Dae wrote:
Hi Javier,
2015-12-07 22:41 GMT+09:00 Javier Martinez Canillas javier@osg.samsung.com:
Hello Inki,
On 12/07/2015 09:52 AM, Inki Dae wrote:
From: Javier Martinez Canillas javier@osg.samsung.com
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 javier@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com
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
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 javier@osg.samsung.com:
Hello Inki,
On 12/07/2015 09:52 AM, Inki Dae wrote:
From: Javier Martinez Canillas javier@osg.samsung.com
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 javier@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com
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,
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 javier@osg.samsung.com:
Hello Inki,
On 12/07/2015 09:52 AM, Inki Dae wrote:
From: Javier Martinez Canillas javier@osg.samsung.com
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 javier@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com
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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 07.12.2015 21:52, Inki Dae wrote:
From: Javier Martinez Canillas javier@osg.samsung.com
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@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com Reviewed-by: Inki Dae inki.dae@samsung.com
arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)
Looks sensible: Reviewed-by: Krzysztof Kozlowski k.kozlowski@samsung.com
Dependencies are not mentioned, does it depend on patch 1?
Best regards, Krzysztof
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 javier@osg.samsung.com
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@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com Reviewed-by: Inki Dae inki.dae@samsung.com
arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)
Looks sensible: Reviewed-by: Krzysztof Kozlowski k.kozlowski@samsung.com
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,
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 javier@osg.samsung.com
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@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com Reviewed-by: Inki Dae inki.dae@samsung.com
arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)
Looks sensible: Reviewed-by: Krzysztof Kozlowski k.kozlowski@samsung.com
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
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 javier@osg.samsung.com
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@osg.samsung.com Tested-by: Javier Martinez Canillas javier@osg.samsung.com Reviewed-by: Inki Dae inki.dae@samsung.com
arch/arm/boot/dts/exynos5800-peach-pi.dts | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)
Looks sensible: Reviewed-by: Krzysztof Kozlowski k.kozlowski@samsung.com
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
dri-devel@lists.freedesktop.org
-
Inki Dae -
Inki Dae -
Javier Martinez Canillas -
Krzysztof Kozlowski -
Rob Herring