| Message ID | 1520603500-15218-4-git-send-email-jacopo+renesas@jmondi.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
On Fri, Mar 09, 2018 at 02:51:40PM +0100, Jacopo Mondi wrote: > The R-Car V3M Eagle board includes a transparent LVDS decoder, connected > to the on-chip LVDS encoder output on one side and to HDMI encoder > ADV7511w on the other one. > > As the decoder does not need any configuration it has been so-far > omitted from DTS. Now that a driver for transparent LVDS decoder is > available, describe it in DT as well. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> Thanks, I have marked this as deferred pending acceptance of the bindings. Please repost or otherwise ping me once they have been accepted.
On 03/09/2018 04:51 PM, Jacopo Mondi wrote: > The R-Car V3M Eagle board includes a transparent LVDS decoder, connected > to the on-chip LVDS encoder output on one side and to HDMI encoder > ADV7511w on the other one. > > As the decoder does not need any configuration it has been so-far > omitted from DTS. Now that a driver for transparent LVDS decoder is > available, describe it in DT as well. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > --- > arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 31 ++++++++++++++++++++++++-- > 1 file changed, 29 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > index c0fd144..0a95c20 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > @@ -42,6 +42,33 @@ > }; > }; > }; > + > + lvds,decoder { I said hyphen (-), not comma (,)! :-) [...] MBR, Sergei
Hi Sergei, On Fri, Mar 09, 2018 at 08:30:36PM +0300, Sergei Shtylyov wrote: > On 03/09/2018 04:51 PM, Jacopo Mondi wrote: > > > The R-Car V3M Eagle board includes a transparent LVDS decoder, connected > > to the on-chip LVDS encoder output on one side and to HDMI encoder > > ADV7511w on the other one. > > > > As the decoder does not need any configuration it has been so-far > > omitted from DTS. Now that a driver for transparent LVDS decoder is > > available, describe it in DT as well. > > > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > > --- > > arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 31 ++++++++++++++++++++++++-- > > 1 file changed, 29 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > > index c0fd144..0a95c20 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > > +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > > @@ -42,6 +42,33 @@ > > }; > > }; > > }; > > + > > + lvds,decoder { > > I said hyphen (-), not comma (,)! :-) Ahah ok, my bad, I somehow confused the two :) It looked weird to me, but I thought you were pointing me to some convention I didn't know about! I'll change this in v3 Thanks j > > [...] > > MBR, Sergei
diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts index c0fd144..0a95c20 100644 --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts @@ -42,6 +42,33 @@ }; }; }; + + lvds,decoder { + compatible = "lvds-decoder"; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + + lvds_dec_in: endpoint { + remote-endpoint = <&lvds0_out>; + }; + }; + + port@1{ + reg = <1>; + + lvds_dec_out: endpoint { + remote-endpoint = <&adv7511_in>; + }; + + }; + + }; + }; }; &avb { @@ -98,7 +125,7 @@ port@0 { reg = <0>; adv7511_in: endpoint { - remote-endpoint = <&lvds0_out>; + remote-endpoint = <&lvds_dec_out>; }; }; @@ -153,7 +180,7 @@ ports { port@1 { endpoint { - remote-endpoint = <&adv7511_in>; + remote-endpoint = <&lvds_dec_in>; }; }; };
The R-Car V3M Eagle board includes a transparent LVDS decoder, connected to the on-chip LVDS encoder output on one side and to HDMI encoder ADV7511w on the other one. As the decoder does not need any configuration it has been so-far omitted from DTS. Now that a driver for transparent LVDS decoder is available, describe it in DT as well. Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> --- arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 31 ++++++++++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-)