From mboxrd@z Thu Jan 1 00:00:00 1970 From: jacopo mondi Subject: Re: [PATCH 7/7] arm64: dts: renesas: eagle: Add ADV7511W and HDMI output Date: Fri, 6 Apr 2018 16:16:32 +0200 Message-ID: <20180406141632.GO20945@w540> References: <1523020092-25540-1-git-send-email-jacopo+renesas@jmondi.org> <1523020092-25540-8-git-send-email-jacopo+renesas@jmondi.org> <2235860.8H9ojNphtF@avalon> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1333342509==" Return-path: In-Reply-To: <2235860.8H9ojNphtF@avalon> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Laurent Pinchart Cc: devicetree@vger.kernel.org, Niklas =?utf-8?Q?S=C3=B6derlund?= , sergei.shtylyov@cogentembedded.com, magnus.damm@gmail.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, horms@verge.net.au, Jacopo Mondi , niklas.soderlund@ragnatech.se, geert@linux-m68k.org List-Id: devicetree@vger.kernel.org --===============1333342509== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="R0KbGFzrRdaLlFh/" Content-Disposition: inline --R0KbGFzrRdaLlFh/ Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Laurent, On Fri, Apr 06, 2018 at 04:51:11PM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > Thank you for the patch. > > On Friday, 6 April 2018 16:08:12 EEST Jacopo Mondi wrote: > > From: Niklas S=C3=B6derlund > > > > Enable HDMI output adding the HDMI connector and the ADV7511W, connected > > to THC63LVD1024 LVDS decoder output. > > > > Signed-off-by: Niklas S=C3=B6derlund > > Signed-off-by: Jacopo Mondi > > --- > > arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 51 ++++++++++++++++++= ++++- > > 1 file changed, 50 insertions(+), 1 deletion(-) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > > b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts index 9d0e65d..e9f7b83 > > 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > > +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > > @@ -32,6 +32,17 @@ > > reg =3D <0x0 0x48000000 0x0 0x38000000>; > > }; > > > > + hdmi-out { > > + compatible =3D "hdmi-connector"; > > + type =3D "a"; > > + > > + port { > > + hdmi_con_out: endpoint { > > + remote-endpoint =3D <&adv7511_out>; > > + }; > > + }; > > + }; > > + > > thc63lvd1024: lvds-decoder { > > compatible =3D "thine,thc63lvd1024"; > > > > @@ -41,11 +52,17 @@ > > > > port@0 { > > reg =3D <0>; > > - > > This is unrelated, if you don't want a blank line here remove it from pat= ch > 6/7 :-) No, you're right, this is a leftover from me splitting a single a patch in 3. According to your comments on other patches in the series I shouldn't have done that to begin with :) Thanks j > > > thc63lvd1024_in: endpoint { > > remote-endpoint =3D <&lvds0_out>; > > }; > > }; > > + > > + port@2 { > > + reg =3D <2>; > > + thc63lvd1024_out: endpoint { > > + remote-endpoint =3D <&adv7511_in>; > > + }; > > + }; > > }; > > }; > > }; > > @@ -85,6 +102,38 @@ > > gpio-controller; > > #gpio-cells =3D <2>; > > }; > > + > > + hdmi@39 { > > + compatible =3D "adi,adv7511w"; > > + reg =3D <0x39>; > > + interrupt-parent =3D <&gpio1>; > > + interrupts =3D <20 IRQ_TYPE_LEVEL_LOW>; > > + > > + adi,input-depth =3D <8>; > > + adi,input-colorspace =3D "rgb"; > > + adi,input-clock =3D "1x"; > > + adi,input-style =3D <1>; > > + adi,input-justification =3D "evenly"; > > + > > + ports { > > + #address-cells =3D <1>; > > + #size-cells =3D <0>; > > + > > + port@0 { > > + reg =3D <0>; > > + adv7511_in: endpoint { > > + remote-endpoint =3D <&thc63lvd1024_out>; > > + }; > > + }; > > + > > + port@1 { > > + reg =3D <1>; > > + adv7511_out: endpoint { > > + remote-endpoint =3D <&hdmi_con_out>; > > + }; > > + }; > > + }; > > + }; > > }; > > > > &pfc { > > With patches 5/7, 6/7 and 7/7 merged together and the pinmux removed, > > Reviewed-by: Laurent Pinchart > > -- > Regards, > > Laurent Pinchart > > > --R0KbGFzrRdaLlFh/ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJax4FAAAoJEHI0Bo8WoVY8yBIP/0LBBYLyRQhzgOQ6hD31wVZs EmbPi+Yj+chIKUl1acF6klnPCO4dV46Ilz96yDS6CzeyPH/MUkmy2LJxEnj9Kim2 FE+8C61WSA9+8wMbAWvGToEdQNlnnLbY5GwypZqZXuZ+/2i3yiq4zWUza1aP3Dzu NEGCU98eGD0uHs9+9yuT1OOr3CyVY8oTCe7TgtJ+w8roH3ldx+0fdBkw3Mh859CV AGDPV+WwMxUQMRRRVqWtyIuWSg9mxT/fuuM9wzOUntY8ib17hKjVUugg+BIjx/bP RbUCfQHf/dydLXg8iBBXmebiJJUQH89dpAVQ7aGlzQac98yKr0rYP6LV72Q/sikn jiQnnoGgT+XauerEvbOBkE1wx3ha/2YPkpznzNpRXyqKdMQAwxZ8WgizgaPnGw9D BHeG055XsIuTcV9n26q39xs8Yb79qawkCtK2cWdrUcHBLLIV3EBOldayNtsRus9X H3S1pD2AnrFk+oCxu13xmmDoGNHDdjphHdlZdCQih/SV4jTw0sTvxWi/MksgVSqM l9dEEz0GAnkRUWrAKi3OhXNhTsDyRbpDG4YnJwytTD2/poWok6XgqfIWLzaN9Qm/ 6CHjfFbSCjuTylIHZTFMy13cQlHOtswjNUFdgsVbMtXYDk4TtrDhdSY0GKPj/R6M sGs0PBnZKrd15wGKoZBX =ydXF -----END PGP SIGNATURE----- --R0KbGFzrRdaLlFh/-- --===============1333342509== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1333342509==--