[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <TY1PR01MB17708785623D4B6747418A9EC05F0@TY1PR01MB1770.jpnprd01.prod.outlook.com>
Date: Fri, 6 Dec 2019 15:40:42 +0000
From: Fabrizio Castro <fabrizio.castro@...renesas.com>
To: Laurent Pinchart <laurent.pinchart@...asonboard.com>
CC: Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Simon Horman <horms@...ge.net.au>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Magnus Damm <magnus.damm@...il.com>,
"linux-renesas-soc@...r.kernel.org"
<linux-renesas-soc@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Chris Paterson <Chris.Paterson2@...esas.com>,
Biju Das <biju.das@...renesas.com>,
Kieran Bingham <kieran.bingham+renesas@...asonboard.com>,
Jacopo Mondi <jacopo+renesas@...ndi.org>,
"sam@...nborg.org" <sam@...nborg.org>
Subject: RE: [PATCH v3 7/8] [HACK] arm64: dts: renesas: draak: Enable LVDS
Hello Laurent,
Thank you for the feedback!
> From: linux-renesas-soc-owner@...r.kernel.org <linux-renesas-soc-owner@...r.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@...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
Powered by blists - more mailing lists