* [PATCH] ARM: dts: exynos: Add the OF graph between fimd and dsi [not found] <CGME20170609025951epcas1p19f5ca8c241b8b79068a84eb239ed46ed@epcas1p1.samsung.com> @ 2017-06-09 2:59 ` Hoegeun Kwon 2017-06-09 7:27 ` Krzysztof Kozlowski 0 siblings, 1 reply; 3+ messages in thread From: Hoegeun Kwon @ 2017-06-09 2:59 UTC (permalink / raw) To: krzk, inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, robh+dt, mark.rutland, catalin.marinas, will.deacon Cc: devicetree, linux-samsung-soc, linux-kernel, dri-devel, javier, Hoegeun Kwon, linux-arm-kernel It should be connected to OF graph between fimd and dsi. Add the OF graph between fimd and dsi. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> --- arch/arm/boot/dts/exynos3250.dtsi | 26 ++++++++++++++++++++++++++ arch/arm/boot/dts/exynos4.dtsi | 26 ++++++++++++++++++++++++++ 2 files changed, 52 insertions(+) diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi index 590ee44..22d523f 100644 --- a/arch/arm/boot/dts/exynos3250.dtsi +++ b/arch/arm/boot/dts/exynos3250.dtsi @@ -311,6 +311,19 @@ iommus = <&sysmmu_fimd0>; samsung,sysreg = <&sys_reg>; status = "disabled"; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + fimd_to_dsi: endpoint { + remote-endpoint = + <&dsi_to_fimd>; + }; + }; + }; }; dsi_0: dsi@11C80000 { @@ -326,6 +339,19 @@ #address-cells = <1>; #size-cells = <0>; status = "disabled"; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + dsi_to_fimd: endpoint { + remote-endpoint = + <&fimd_to_dsi>; + }; + }; + }; }; sysmmu_fimd0: sysmmu@11E20000 { diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi index 497a947..773d392 100644 --- a/arch/arm/boot/dts/exynos4.dtsi +++ b/arch/arm/boot/dts/exynos4.dtsi @@ -193,6 +193,19 @@ status = "disabled"; #address-cells = <1>; #size-cells = <0>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + dsi_to_fimd: endpoint { + remote-endpoint = + <&fimd_to_dsi>; + }; + }; + }; }; camera { @@ -715,6 +728,19 @@ iommus = <&sysmmu_fimd0>; samsung,sysreg = <&sys_reg>; status = "disabled"; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + fimd_to_dsi: endpoint { + remote-endpoint = + <&dsi_to_fimd>; + }; + }; + }; }; tmu: tmu@100C0000 { -- 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] 3+ messages in thread
* Re: [PATCH] ARM: dts: exynos: Add the OF graph between fimd and dsi 2017-06-09 2:59 ` [PATCH] ARM: dts: exynos: Add the OF graph between fimd and dsi Hoegeun Kwon @ 2017-06-09 7:27 ` Krzysztof Kozlowski 2017-06-09 8:26 ` Hoegeun Kwon 0 siblings, 1 reply; 3+ messages in thread From: Krzysztof Kozlowski @ 2017-06-09 7:27 UTC (permalink / raw) To: Hoegeun Kwon Cc: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, robh+dt, mark.rutland, catalin.marinas, will.deacon, javier, dri-devel, linux-arm-kernel, linux-samsung-soc, linux-kernel, devicetree, a.hajda On Fri, Jun 09, 2017 at 11:59:33AM +0900, Hoegeun Kwon wrote: > It should be connected to OF graph between fimd and dsi. > Add the OF graph between fimd and dsi. I need more explanation why this is needed. What happens without this? What issue is fixed here? What about Exynos5250 and Exynos5420? I think we do not have a board in mainline using FIMD but maybe this should be added just for correct hardware description? Best regards, Krzysztof > > Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> > --- > arch/arm/boot/dts/exynos3250.dtsi | 26 ++++++++++++++++++++++++++ > arch/arm/boot/dts/exynos4.dtsi | 26 ++++++++++++++++++++++++++ > 2 files changed, 52 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi > index 590ee44..22d523f 100644 > --- a/arch/arm/boot/dts/exynos3250.dtsi > +++ b/arch/arm/boot/dts/exynos3250.dtsi > @@ -311,6 +311,19 @@ > iommus = <&sysmmu_fimd0>; > samsung,sysreg = <&sys_reg>; > status = "disabled"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + fimd_to_dsi: endpoint { > + remote-endpoint = > + <&dsi_to_fimd>; > + }; > + }; > + }; > }; > > dsi_0: dsi@11C80000 { > @@ -326,6 +339,19 @@ > #address-cells = <1>; > #size-cells = <0>; > status = "disabled"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + dsi_to_fimd: endpoint { > + remote-endpoint = > + <&fimd_to_dsi>; > + }; > + }; > + }; > }; > > sysmmu_fimd0: sysmmu@11E20000 { > diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi > index 497a947..773d392 100644 > --- a/arch/arm/boot/dts/exynos4.dtsi > +++ b/arch/arm/boot/dts/exynos4.dtsi > @@ -193,6 +193,19 @@ > status = "disabled"; > #address-cells = <1>; > #size-cells = <0>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + dsi_to_fimd: endpoint { > + remote-endpoint = > + <&fimd_to_dsi>; > + }; > + }; > + }; > }; > > camera { > @@ -715,6 +728,19 @@ > iommus = <&sysmmu_fimd0>; > samsung,sysreg = <&sys_reg>; > status = "disabled"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + fimd_to_dsi: endpoint { > + remote-endpoint = > + <&dsi_to_fimd>; > + }; > + }; > + }; > }; > > tmu: tmu@100C0000 { > -- > 1.9.1 > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: dts: exynos: Add the OF graph between fimd and dsi 2017-06-09 7:27 ` Krzysztof Kozlowski @ 2017-06-09 8:26 ` Hoegeun Kwon 0 siblings, 0 replies; 3+ messages in thread From: Hoegeun Kwon @ 2017-06-09 8:26 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: mark.rutland, devicetree, linux-samsung-soc, catalin.marinas, sw0312.kim, linux-kernel, robh+dt, javier, kgene, dri-devel, will.deacon, linux-arm-kernel On 06/09/2017 04:27 PM, Krzysztof Kozlowski wrote: > On Fri, Jun 09, 2017 at 11:59:33AM +0900, Hoegeun Kwon wrote: >> It should be connected to OF graph between fimd and dsi. >> Add the OF graph between fimd and dsi. > I need more explanation why this is needed. What happens without this? > What issue is fixed here? > > What about Exynos5250 and Exynos5420? I think we do not have a board in > mainline using FIMD but maybe this should be added just for correct > hardware description? Thank you for your review. I'm sorry, this patch is wrong. Please ignore this patch. :) Best regards, Hoegeun > > > Best regards, > Krzysztof > >> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> >> --- >> arch/arm/boot/dts/exynos3250.dtsi | 26 ++++++++++++++++++++++++++ >> arch/arm/boot/dts/exynos4.dtsi | 26 ++++++++++++++++++++++++++ >> 2 files changed, 52 insertions(+) >> >> diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi >> index 590ee44..22d523f 100644 >> --- a/arch/arm/boot/dts/exynos3250.dtsi >> +++ b/arch/arm/boot/dts/exynos3250.dtsi >> @@ -311,6 +311,19 @@ >> iommus = <&sysmmu_fimd0>; >> samsung,sysreg = <&sys_reg>; >> status = "disabled"; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + port@0 { >> + reg = <0>; >> + fimd_to_dsi: endpoint { >> + remote-endpoint = >> + <&dsi_to_fimd>; >> + }; >> + }; >> + }; >> }; >> >> dsi_0: dsi@11C80000 { >> @@ -326,6 +339,19 @@ >> #address-cells = <1>; >> #size-cells = <0>; >> status = "disabled"; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + port@0 { >> + reg = <0>; >> + dsi_to_fimd: endpoint { >> + remote-endpoint = >> + <&fimd_to_dsi>; >> + }; >> + }; >> + }; >> }; >> >> sysmmu_fimd0: sysmmu@11E20000 { >> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi >> index 497a947..773d392 100644 >> --- a/arch/arm/boot/dts/exynos4.dtsi >> +++ b/arch/arm/boot/dts/exynos4.dtsi >> @@ -193,6 +193,19 @@ >> status = "disabled"; >> #address-cells = <1>; >> #size-cells = <0>; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + port@0 { >> + reg = <0>; >> + dsi_to_fimd: endpoint { >> + remote-endpoint = >> + <&fimd_to_dsi>; >> + }; >> + }; >> + }; >> }; >> >> camera { >> @@ -715,6 +728,19 @@ >> iommus = <&sysmmu_fimd0>; >> samsung,sysreg = <&sys_reg>; >> status = "disabled"; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + port@0 { >> + reg = <0>; >> + fimd_to_dsi: endpoint { >> + remote-endpoint = >> + <&dsi_to_fimd>; >> + }; >> + }; >> + }; >> }; >> >> tmu: tmu@100C0000 { >> -- >> 1.9.1 >> > > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-06-09 8:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20170609025951epcas1p19f5ca8c241b8b79068a84eb239ed46ed@epcas1p1.samsung.com>
2017-06-09 2:59 ` [PATCH] ARM: dts: exynos: Add the OF graph between fimd and dsi Hoegeun Kwon
2017-06-09 7:27 ` Krzysztof Kozlowski
2017-06-09 8:26 ` Hoegeun Kwon
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).