* [PATCH v3 0/7] Fix the parse_dt of exynos dsi and remove the OF graph [not found] <CGME20170302102112epcas5p476b30508bbaffbf1fe9781b941cd418f@epcas5p4.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon [not found] ` <CGME20170302102113epcas5p4992ba2da58333ee0e3fe988334aad9a9@epcas5p4.samsung.com> ` (7 more replies) 0 siblings, 8 replies; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel Hi All, The dsi + panel is a parental relationship, so OF grpah is not needed. Therefore, the current dsi_parse_dt function will throw an error, because there is no linked OF graph for case such as fimd + dsi + panel. So the 1/5 patch parse the Pll, burst and esc clock frequency properties in dsi_parse_dt and modified to create a bridge_node only if there is an OF graph associated with dsi. Also fixed the dts, which depend on the 1/5 patch. So removed the ports node and move burst and esc clock frequency properties to the parent (DSI node). Changes for V3: - Split the patches considering the bisectability problem. Changes for V2: - Added the clear explanation for commit. (1/5 patch) - Fixed it to the same subject as the actual work. (2/5 ~ 5/5 patches) Best Regards, Hoegeun Hoegeun Kwon (7): arm64: dts: exynos: Add the burst and esc clock frequency properties for exynos5433 dts arm: dts: Add the burst and esc clock frequency properties for exynos3250 dts arm: dts: Add the burst and esc clock frequency properties for exynos4412 dts arm: dts: Add the burst and esc clock frequency properties for exynos4210 dts drm/exynos: dsi: Fix the parse_dt function arm64: dts: exynos: Remove the OF graph from DSI node arm: dts: Remove the OF graph from DSI node arch/arm/boot/dts/exynos3250-rinato.dts | 23 ++-------------- arch/arm/boot/dts/exynos4210-trats.dts | 23 ++-------------- arch/arm/boot/dts/exynos4412-trats2.dts | 23 ++-------------- .../boot/dts/exynos/exynos5433-tm2-common.dtsi | 16 ++--------- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 32 ++++++---------------- 5 files changed, 16 insertions(+), 101 deletions(-) -- 1.9.1 ^ permalink raw reply [flat|nested] 11+ messages in thread
[parent not found: <CGME20170302102113epcas5p4992ba2da58333ee0e3fe988334aad9a9@epcas5p4.samsung.com>]
* [PATCH v3 1/7] arm64: dts: exynos: Add the burst and esc clock frequency properties for exynos5433 dts [not found] ` <CGME20170302102113epcas5p4992ba2da58333ee0e3fe988334aad9a9@epcas5p4.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon 0 siblings, 0 replies; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel The OF graph is not needed because the panel is a child of dsi. So added the burst and esc clock frequency properties to the parent (DSI node), taking into account the bisectability problem so that remove the OF graph from DSI node. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> --- arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi index 6ce93a3..e31e20c 100644 --- a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi +++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi @@ -298,6 +298,8 @@ status = "okay"; vddcore-supply = <&ldo6_reg>; vddio-supply = <&ldo7_reg>; + samsung,burst-clock-frequency = <512000000>; + samsung,esc-clock-frequency = <16000000>; samsung,pll-clock-frequency = <24000000>; pinctrl-names = "default"; pinctrl-0 = <&te_irq>; -- 1.9.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 11+ messages in thread
[parent not found: <CGME20170302102113epcas1p343e9da20f05be0a2359bf32834626c4a@epcas1p3.samsung.com>]
* [PATCH v3 2/7] arm: dts: Add the burst and esc clock frequency properties for exynos3250 dts [not found] ` <CGME20170302102113epcas1p343e9da20f05be0a2359bf32834626c4a@epcas1p3.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon 2017-03-03 0:26 ` Andi Shyti 0 siblings, 1 reply; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel The OF graph is not needed because the panel is a child of dsi. So added the burst and esc clock frequency properties to the parent (DSI node), taking into account the bisectability problem so that remove the OF graph from DSI node. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> --- arch/arm/boot/dts/exynos3250-rinato.dts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts index 548413e..c9f191c 100644 --- a/arch/arm/boot/dts/exynos3250-rinato.dts +++ b/arch/arm/boot/dts/exynos3250-rinato.dts @@ -215,6 +215,8 @@ &dsi_0 { vddcore-supply = <&ldo6_reg>; vddio-supply = <&ldo6_reg>; + samsung,burst-clock-frequency = <250000000>; + samsung,esc-clock-frequency = <20000000>; samsung,pll-clock-frequency = <24000000>; status = "okay"; -- 1.9.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v3 2/7] arm: dts: Add the burst and esc clock frequency properties for exynos3250 dts 2017-03-02 10:20 ` [PATCH v3 2/7] arm: dts: Add the burst and esc clock frequency properties for exynos3250 dts Hoegeun Kwon @ 2017-03-03 0:26 ` Andi Shyti 0 siblings, 0 replies; 11+ messages in thread From: Andi Shyti @ 2017-03-03 0:26 UTC (permalink / raw) To: Hoegeun Kwon Cc: mark.rutland, devicetree, linux-samsung-soc, jy0922.shim, javier, airlied, catalin.marinas, sw0312.kim, linux-kernel, krzk, inki.dae, robh+dt, kgene, dri-devel, will.deacon, linux-arm-kernel Hi Hoegeun, On Thu, Mar 02, 2017 at 07:20:14PM +0900, Hoegeun Kwon wrote: > The OF graph is not needed because the panel is a child of dsi. So > added the burst and esc clock frequency properties to the parent (DSI > node), taking into account the bisectability problem so that remove > the OF graph from DSI node. nitpick: 1. bisectability is not a problem (and for bisectability you should not mention it in the commit log, you would confuse bisecting people) 2. you should use the imperative form, not "added the burst... " but "add the burst..." Same for the other patches. Andi ^ permalink raw reply [flat|nested] 11+ messages in thread
[parent not found: <CGME20170302102113epcas5p4c93024f46ed53ac962b332b559553cd5@epcas5p4.samsung.com>]
* [PATCH v3 3/7] arm: dts: Add the burst and esc clock frequency properties for exynos4412 dts [not found] ` <CGME20170302102113epcas5p4c93024f46ed53ac962b332b559553cd5@epcas5p4.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon 0 siblings, 0 replies; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel The OF graph is not needed because the panel is a child of dsi. So added the burst and esc clock frequency properties to the parent (DSI node), taking into account the bisectability problem so that remove the OF graph from DSI node. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> --- arch/arm/boot/dts/exynos4412-trats2.dts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts index 41ecd6d..82221a0 100644 --- a/arch/arm/boot/dts/exynos4412-trats2.dts +++ b/arch/arm/boot/dts/exynos4412-trats2.dts @@ -385,6 +385,8 @@ &dsi_0 { vddcore-supply = <&ldo8_reg>; vddio-supply = <&ldo10_reg>; + samsung,burst-clock-frequency = <500000000>; + samsung,esc-clock-frequency = <20000000>; samsung,pll-clock-frequency = <24000000>; status = "okay"; -- 1.9.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 11+ messages in thread
[parent not found: <CGME20170302102113epcas5p491d4220b8ae7a037d1cd7ece49bb2d3e@epcas5p4.samsung.com>]
* [PATCH v3 4/7] arm: dts: Add the burst and esc clock frequency properties for exynos4210 dts [not found] ` <CGME20170302102113epcas5p491d4220b8ae7a037d1cd7ece49bb2d3e@epcas5p4.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon 0 siblings, 0 replies; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel The OF graph is not needed because the panel is a child of dsi. So added the burst and esc clock frequency properties to the parent (DSI node), taking into account the bisectability problem so that remove the OF graph from DSI node. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> --- arch/arm/boot/dts/exynos4210-trats.dts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/boot/dts/exynos4210-trats.dts b/arch/arm/boot/dts/exynos4210-trats.dts index 0ca1b4d..1743ca8 100644 --- a/arch/arm/boot/dts/exynos4210-trats.dts +++ b/arch/arm/boot/dts/exynos4210-trats.dts @@ -197,6 +197,8 @@ &dsi_0 { vddcore-supply = <&vusb_reg>; vddio-supply = <&vmipi_reg>; + samsung,burst-clock-frequency = <500000000>; + samsung,esc-clock-frequency = <20000000>; samsung,pll-clock-frequency = <24000000>; status = "okay"; -- 1.9.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 11+ messages in thread
[parent not found: <CGME20170302102113epcas1p306e85959e4bbfe1ecb0e5af8fcd18b7d@epcas1p3.samsung.com>]
* [PATCH v3 5/7] drm/exynos: dsi: Fix the parse_dt function [not found] ` <CGME20170302102113epcas1p306e85959e4bbfe1ecb0e5af8fcd18b7d@epcas1p3.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon 0 siblings, 0 replies; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel The dsi + panel is a parental relationship, so OF grpah is not needed. Therefore, the current dsi_parse_dt function will throw an error, because there is no linked OF graph for case such as fimd + dsi + panel. So this patch parse the Pll, burst and esc clock frequency properties in dsi_parse_dt and modified to create a bridge_node only if there is an OF graph associated with dsi. So I think the ABI breakage is needed. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> --- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 32 ++++++++------------------------ 1 file changed, 8 insertions(+), 24 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c index f5c04d0..2d4e118 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c @@ -1652,39 +1652,23 @@ static int exynos_dsi_parse_dt(struct exynos_dsi *dsi) if (ret < 0) return ret; - ep = of_graph_get_endpoint_by_regs(node, DSI_PORT_OUT, 0); - if (!ep) { - dev_err(dev, "no output port with endpoint specified\n"); - return -EINVAL; - } - - ret = exynos_dsi_of_read_u32(ep, "samsung,burst-clock-frequency", + ret = exynos_dsi_of_read_u32(node, "samsung,burst-clock-frequency", &dsi->burst_clk_rate); if (ret < 0) - goto end; + return ret; - ret = exynos_dsi_of_read_u32(ep, "samsung,esc-clock-frequency", + ret = exynos_dsi_of_read_u32(node, "samsung,esc-clock-frequency", &dsi->esc_clk_rate); if (ret < 0) - goto end; - - of_node_put(ep); + return ret; ep = of_graph_get_next_endpoint(node, NULL); - if (!ep) { - ret = -EINVAL; - goto end; - } - - dsi->bridge_node = of_graph_get_remote_port_parent(ep); - if (!dsi->bridge_node) { - ret = -EINVAL; - goto end; + if (ep) { + dsi->bridge_node = of_graph_get_remote_port_parent(ep); + of_node_put(ep); } -end: - of_node_put(ep); - return ret; + return 0; } static int exynos_dsi_bind(struct device *dev, struct device *master, -- 1.9.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 11+ messages in thread
[parent not found: <CGME20170302102113epcas5p466a64e5f55c008ceec957b1432a30173@epcas5p4.samsung.com>]
* [PATCH v3 6/7] arm64: dts: exynos: Remove the OF graph from DSI node [not found] ` <CGME20170302102113epcas5p466a64e5f55c008ceec957b1432a30173@epcas5p4.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon 0 siblings, 0 replies; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel The OF graph is not needed because the panel is a child of dsi. So removed the ports node in DSI node. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> --- arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi index e31e20c..77ba238 100644 --- a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi +++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi @@ -303,20 +303,6 @@ samsung,pll-clock-frequency = <24000000>; pinctrl-names = "default"; pinctrl-0 = <&te_irq>; - - ports { - #address-cells = <1>; - #size-cells = <0>; - - port@1 { - reg = <1>; - - dsi_out: endpoint { - samsung,burst-clock-frequency = <512000000>; - samsung,esc-clock-frequency = <16000000>; - }; - }; - }; }; &hdmi { -- 1.9.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 11+ messages in thread
[parent not found: <CGME20170302102113epcas1p30d42204d23a4e0def2a406086fea9cef@epcas1p3.samsung.com>]
* [PATCH v3 7/7] arm: dts: Remove the OF graph from DSI node [not found] ` <CGME20170302102113epcas1p30d42204d23a4e0def2a406086fea9cef@epcas1p3.samsung.com> @ 2017-03-02 10:20 ` Hoegeun Kwon 0 siblings, 0 replies; 11+ messages in thread From: Hoegeun Kwon @ 2017-03-02 10:20 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel The OF graph is not needed because the panel is a child of dsi. So removed the ports node in DSI node, and port node in panel node. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> --- arch/arm/boot/dts/exynos3250-rinato.dts | 21 --------------------- arch/arm/boot/dts/exynos4210-trats.dts | 21 --------------------- arch/arm/boot/dts/exynos4412-trats2.dts | 21 --------------------- 3 files changed, 63 deletions(-) diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts index c9f191c..82e676a 100644 --- a/arch/arm/boot/dts/exynos3250-rinato.dts +++ b/arch/arm/boot/dts/exynos3250-rinato.dts @@ -220,21 +220,6 @@ samsung,pll-clock-frequency = <24000000>; status = "okay"; - ports { - #address-cells = <1>; - #size-cells = <0>; - - port@1 { - reg = <1>; - - dsi_out: endpoint { - remote-endpoint = <&dsi_in>; - samsung,burst-clock-frequency = <250000000>; - samsung,esc-clock-frequency = <20000000>; - }; - }; - }; - panel@0 { compatible = "samsung,s6e63j0x03"; reg = <0>; @@ -264,12 +249,6 @@ vsync-len = <2>; }; }; - - port { - dsi_in: endpoint { - remote-endpoint = <&dsi_out>; - }; - }; }; }; diff --git a/arch/arm/boot/dts/exynos4210-trats.dts b/arch/arm/boot/dts/exynos4210-trats.dts index 1743ca8..9452bed 100644 --- a/arch/arm/boot/dts/exynos4210-trats.dts +++ b/arch/arm/boot/dts/exynos4210-trats.dts @@ -202,21 +202,6 @@ samsung,pll-clock-frequency = <24000000>; status = "okay"; - ports { - #address-cells = <1>; - #size-cells = <0>; - - port@1 { - reg = <1>; - - dsi_out: endpoint { - remote-endpoint = <&dsi_in>; - samsung,burst-clock-frequency = <500000000>; - samsung,esc-clock-frequency = <20000000>; - }; - }; - }; - panel@0 { reg = <0>; compatible = "samsung,s6e8aa0"; @@ -244,12 +229,6 @@ vsync-len = <2>; }; }; - - port { - dsi_in: endpoint { - remote-endpoint = <&dsi_out>; - }; - }; }; }; diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts index 82221a0..86ce5e5 100644 --- a/arch/arm/boot/dts/exynos4412-trats2.dts +++ b/arch/arm/boot/dts/exynos4412-trats2.dts @@ -390,21 +390,6 @@ samsung,pll-clock-frequency = <24000000>; status = "okay"; - ports { - #address-cells = <1>; - #size-cells = <0>; - - port@1 { - reg = <1>; - - dsi_out: endpoint { - remote-endpoint = <&dsi_in>; - samsung,burst-clock-frequency = <500000000>; - samsung,esc-clock-frequency = <20000000>; - }; - }; - }; - panel@0 { compatible = "samsung,s6e8aa0"; reg = <0>; @@ -432,12 +417,6 @@ vsync-len = <2>; }; }; - - port { - dsi_in: endpoint { - remote-endpoint = <&dsi_out>; - }; - }; }; }; -- 1.9.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v3 0/7] Fix the parse_dt of exynos dsi and remove the OF graph 2017-03-02 10:20 ` [PATCH v3 0/7] Fix the parse_dt of exynos dsi and remove the OF graph Hoegeun Kwon ` (6 preceding siblings ...) [not found] ` <CGME20170302102113epcas1p30d42204d23a4e0def2a406086fea9cef@epcas1p3.samsung.com> @ 2017-03-03 0:22 ` Andi Shyti 2017-03-04 8:23 ` Krzysztof Kozlowski 7 siblings, 1 reply; 11+ messages in thread From: Andi Shyti @ 2017-03-03 0:22 UTC (permalink / raw) To: Hoegeun Kwon Cc: mark.rutland, devicetree, linux-samsung-soc, jy0922.shim, javier, airlied, catalin.marinas, sw0312.kim, linux-kernel, krzk, inki.dae, robh+dt, kgene, dri-devel, will.deacon, linux-arm-kernel Hi Hoegeun, > Hoegeun Kwon (7): > arm64: dts: exynos: Add the burst and esc clock frequency properties > for exynos5433 dts > arm: dts: Add the burst and esc clock frequency properties for > exynos3250 dts > arm: dts: Add the burst and esc clock frequency properties for > exynos4412 dts > arm: dts: Add the burst and esc clock frequency properties for > exynos4210 dts > drm/exynos: dsi: Fix the parse_dt function > arm64: dts: exynos: Remove the OF graph from DSI node > arm: dts: Remove the OF graph from DSI node for all of them: Reviewed-by: Andi Shyti <andi.shyti@samsung.com> although I would have squashed patch 2, 3 and 4, but no need to resend, unless someone else agrees. Andi ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v3 0/7] Fix the parse_dt of exynos dsi and remove the OF graph 2017-03-03 0:22 ` [PATCH v3 0/7] Fix the parse_dt of exynos dsi and remove the OF graph Andi Shyti @ 2017-03-04 8:23 ` Krzysztof Kozlowski 0 siblings, 0 replies; 11+ messages in thread From: Krzysztof Kozlowski @ 2017-03-04 8:23 UTC (permalink / raw) To: Andi Shyti Cc: mark.rutland, devicetree, linux-samsung-soc, jy0922.shim, airlied, catalin.marinas, sw0312.kim, linux-kernel, robh+dt, inki.dae, javier, kgene, dri-devel, will.deacon, Hoegeun Kwon, linux-arm-kernel On Fri, Mar 03, 2017 at 09:22:06AM +0900, Andi Shyti wrote: > Hi Hoegeun, > > > Hoegeun Kwon (7): > > arm64: dts: exynos: Add the burst and esc clock frequency properties > > for exynos5433 dts > > arm: dts: Add the burst and esc clock frequency properties for > > exynos3250 dts > > arm: dts: Add the burst and esc clock frequency properties for > > exynos4412 dts > > arm: dts: Add the burst and esc clock frequency properties for > > exynos4210 dts > > drm/exynos: dsi: Fix the parse_dt function > > arm64: dts: exynos: Remove the OF graph from DSI node > > arm: dts: Remove the OF graph from DSI node > > for all of them: > > Reviewed-by: Andi Shyti <andi.shyti@samsung.com> Andi, Thanks for review. It is much welcomed. I am trying not loose such review tags but it might happen because: 1. Patchwork does not store them, 2. Mail does not sit in inbox for long (inbox zero), so if it is possible the most convenient for me is to reply with review to each email. > > although I would have squashed patch 2, 3 and 4, but no need to > resend, unless someone else agrees. Yes, these are small additions to the same arch so they could be indeed squashed. I don't mind keeping them separated though. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2017-03-04 8:23 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <CGME20170302102112epcas5p476b30508bbaffbf1fe9781b941cd418f@epcas5p4.samsung.com> 2017-03-02 10:20 ` [PATCH v3 0/7] Fix the parse_dt of exynos dsi and remove the OF graph Hoegeun Kwon [not found] ` <CGME20170302102113epcas5p4992ba2da58333ee0e3fe988334aad9a9@epcas5p4.samsung.com> 2017-03-02 10:20 ` [PATCH v3 1/7] arm64: dts: exynos: Add the burst and esc clock frequency properties for exynos5433 dts Hoegeun Kwon [not found] ` <CGME20170302102113epcas1p343e9da20f05be0a2359bf32834626c4a@epcas1p3.samsung.com> 2017-03-02 10:20 ` [PATCH v3 2/7] arm: dts: Add the burst and esc clock frequency properties for exynos3250 dts Hoegeun Kwon 2017-03-03 0:26 ` Andi Shyti [not found] ` <CGME20170302102113epcas5p4c93024f46ed53ac962b332b559553cd5@epcas5p4.samsung.com> 2017-03-02 10:20 ` [PATCH v3 3/7] arm: dts: Add the burst and esc clock frequency properties for exynos4412 dts Hoegeun Kwon [not found] ` <CGME20170302102113epcas5p491d4220b8ae7a037d1cd7ece49bb2d3e@epcas5p4.samsung.com> 2017-03-02 10:20 ` [PATCH v3 4/7] arm: dts: Add the burst and esc clock frequency properties for exynos4210 dts Hoegeun Kwon [not found] ` <CGME20170302102113epcas1p306e85959e4bbfe1ecb0e5af8fcd18b7d@epcas1p3.samsung.com> 2017-03-02 10:20 ` [PATCH v3 5/7] drm/exynos: dsi: Fix the parse_dt function Hoegeun Kwon [not found] ` <CGME20170302102113epcas5p466a64e5f55c008ceec957b1432a30173@epcas5p4.samsung.com> 2017-03-02 10:20 ` [PATCH v3 6/7] arm64: dts: exynos: Remove the OF graph from DSI node Hoegeun Kwon [not found] ` <CGME20170302102113epcas1p30d42204d23a4e0def2a406086fea9cef@epcas1p3.samsung.com> 2017-03-02 10:20 ` [PATCH v3 7/7] arm: dts: " Hoegeun Kwon 2017-03-03 0:22 ` [PATCH v3 0/7] Fix the parse_dt of exynos dsi and remove the OF graph Andi Shyti 2017-03-04 8:23 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).