| Message ID | 1567017402-5895-8-git-send-email-fabrizio.castro@bp.renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | Add dual-LVDS panel support to EK874 | expand |
Hi Fabrizio, Thank you for the patch. The subject is wrong, it should be [HACK] arm64: dts: renesas: ebisu: Enable LVDS dual-link operation On Wed, Aug 28, 2019 at 07:36:41PM +0100, Fabrizio Castro wrote: > Enable and connect the second LVDS encoder to the second LVDS input of > the THC63LVD1024 for dual-link LVDS operation. This requires changing > the default settings of SW45 and SW47 to OFF and ON respectively. > > This patch is based on Laurent's dual-LVDS work: > https://patchwork.kernel.org/patch/10965045/ > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > --- > v2->v3: > * new patch > > Geert, > > no need to review this patch unless they like the idea behind this > series. > > Thanks, > Fab > > --- > arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts | 21 +++++++++++++++++++-- > 1 file changed, 19 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts > index b38f9d4..38b9c5a 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts > @@ -87,11 +87,20 @@ > > port@0 { > reg = <0>; > - thc63lvd1024_in: endpoint { > + dual-lvds-even-pixels; > + thc63lvd1024_in0: endpoint { > remote-endpoint = <&lvds0_out>; > }; > }; > > + port@1 { > + reg = <1>; > + dual-lvds-odd-pixels; > + thc63lvd1024_in1: endpoint { > + remote-endpoint = <&lvds1_out>; > + }; > + }; > + > port@2 { > reg = <2>; > thc63lvd1024_out: endpoint { > @@ -489,7 +498,7 @@ > ports { > port@1 { > lvds0_out: endpoint { > - remote-endpoint = <&thc63lvd1024_in>; > + remote-endpoint = <&thc63lvd1024_in0>; > }; > }; > }; > @@ -507,6 +516,14 @@ > <&x13_clk>, > <&extal_clk>; > clock-names = "fck", "dclkin.0", "extal"; > + > + ports { > + port@1 { > + lvds1_out: endpoint { > + remote-endpoint = <&thc63lvd1024_in1>; > + }; > + }; > + }; > }; > > &ohci0 { > -- > 2.7.4 >
Hello Laurent, Thank you for the feedback! > From: linux-renesas-soc-owner@vger.kernel.org <linux-renesas-soc-owner@vger.kernel.org> On Behalf Of Laurent Pinchart > Sent: 07 November 2019 19:58 > Subject: Re: [PATCH v3 7/8] [HACK] arm64: dts: renesas: draak: Enable LVDS > > Hi Fabrizio, > > Thank you for the patch. > > The subject is wrong, it should be > > [HACK] arm64: dts: renesas: ebisu: Enable LVDS dual-link operation This sounds like a copy and paste gone really wrong! Sorry about that. I'll be dropping this patch in v4. Thanks, Fab > > On Wed, Aug 28, 2019 at 07:36:41PM +0100, Fabrizio Castro wrote: > > Enable and connect the second LVDS encoder to the second LVDS input of > > the THC63LVD1024 for dual-link LVDS operation. This requires changing > > the default settings of SW45 and SW47 to OFF and ON respectively. > > > > This patch is based on Laurent's dual-LVDS work: > > https://patchwork.kernel.org/patch/10965045/ > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > --- > > v2->v3: > > * new patch > > > > Geert, > > > > no need to review this patch unless they like the idea behind this > > series. > > > > Thanks, > > Fab > > > > --- > > arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts | 21 +++++++++++++++++++-- > > 1 file changed, 19 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts > > index b38f9d4..38b9c5a 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts > > +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts > > @@ -87,11 +87,20 @@ > > > > port@0 { > > reg = <0>; > > - thc63lvd1024_in: endpoint { > > + dual-lvds-even-pixels; > > + thc63lvd1024_in0: endpoint { > > remote-endpoint = <&lvds0_out>; > > }; > > }; > > > > + port@1 { > > + reg = <1>; > > + dual-lvds-odd-pixels; > > + thc63lvd1024_in1: endpoint { > > + remote-endpoint = <&lvds1_out>; > > + }; > > + }; > > + > > port@2 { > > reg = <2>; > > thc63lvd1024_out: endpoint { > > @@ -489,7 +498,7 @@ > > ports { > > port@1 { > > lvds0_out: endpoint { > > - remote-endpoint = <&thc63lvd1024_in>; > > + remote-endpoint = <&thc63lvd1024_in0>; > > }; > > }; > > }; > > @@ -507,6 +516,14 @@ > > <&x13_clk>, > > <&extal_clk>; > > clock-names = "fck", "dclkin.0", "extal"; > > + > > + ports { > > + port@1 { > > + lvds1_out: endpoint { > > + remote-endpoint = <&thc63lvd1024_in1>; > > + }; > > + }; > > + }; > > }; > > > > &ohci0 { > > -- > > 2.7.4 > > > > -- > Regards, > > Laurent Pinchart
diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts index b38f9d4..38b9c5a 100644 --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts @@ -87,11 +87,20 @@ port@0 { reg = <0>; - thc63lvd1024_in: endpoint { + dual-lvds-even-pixels; + thc63lvd1024_in0: endpoint { remote-endpoint = <&lvds0_out>; }; }; + port@1 { + reg = <1>; + dual-lvds-odd-pixels; + thc63lvd1024_in1: endpoint { + remote-endpoint = <&lvds1_out>; + }; + }; + port@2 { reg = <2>; thc63lvd1024_out: endpoint { @@ -489,7 +498,7 @@ ports { port@1 { lvds0_out: endpoint { - remote-endpoint = <&thc63lvd1024_in>; + remote-endpoint = <&thc63lvd1024_in0>; }; }; }; @@ -507,6 +516,14 @@ <&x13_clk>, <&extal_clk>; clock-names = "fck", "dclkin.0", "extal"; + + ports { + port@1 { + lvds1_out: endpoint { + remote-endpoint = <&thc63lvd1024_in1>; + }; + }; + }; }; &ohci0 {
Enable and connect the second LVDS encoder to the second LVDS input of the THC63LVD1024 for dual-link LVDS operation. This requires changing the default settings of SW45 and SW47 to OFF and ON respectively. This patch is based on Laurent's dual-LVDS work: https://patchwork.kernel.org/patch/10965045/ Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> --- v2->v3: * new patch Geert, no need to review this patch unless they like the idea behind this series. Thanks, Fab --- arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts | 21 +++++++++++++++++++-- 1 file changed, 19 insertions(+), 2 deletions(-)