From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from galahad.ideasonboard.com ([185.26.127.97]:33895 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753212AbeCTNAw (ORCPT ); Tue, 20 Mar 2018 09:00:52 -0400 From: Laurent Pinchart To: Jacopo Mondi Cc: architt@codeaurora.org, a.hajda@samsung.com, airlied@linux.ie, horms@verge.net.au, magnus.damm@gmail.com, geert@linux-m68k.org, niklas.soderlund@ragnatech.se, sergei.shtylyov@cogentembedded.com, robh+dt@kernel.org, mark.rutland@arm.com, dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 3/3] arm64: dts: renesas: Add LVDS decoder to R-Car V3M Eagle Date: Tue, 20 Mar 2018 15:01:57 +0200 Message-ID: <3127791.hHqCBynxiV@avalon> In-Reply-To: <1521213399-31947-4-git-send-email-jacopo+renesas@jmondi.org> References: <1521213399-31947-1-git-send-email-jacopo+renesas@jmondi.org> <1521213399-31947-4-git-send-email-jacopo+renesas@jmondi.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hi Jacopo, Thank you for the patch. On Friday, 16 March 2018 17:16:39 EET Jacopo Mondi wrote: > The R-Car V3M Eagle board includes a transparent THC63LVD1024 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 is available, describe it in DT > as well. > > Signed-off-by: Jacopo Mondi > Reviewed-by: Andrzej Hajda The patch looks OK to me, but I think it should be squashed with Niklas' patch that added display HDMI output support to the V3M Eagle DT. > --- > > List of patch dependencies, as of renesas-drivers-2018-03-13-v4.16-rc5: > > - [PATCH v2 0/5] arm64: dts: renesas: r8a77970: enable HDMI output > which includes DU, LVDS and FCPD enablement from: > [PATCH v2 0/5] Add R8A77970/V3MSK LVDS/HDMI support > - [PATCH v4] v4l: vsp1: Fix video output on R8A77970 > > Patches to be applied on top of > "arm64: dts: renesas: eagle: add HDMI output using the ADV7511W" > > Thanks > j > --- > arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 33 ++++++++++++++++++++--- > 1 file changed, 30 insertions(+), 3 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts index c0fd144..69f43b8 > 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > @@ -42,6 +42,33 @@ > }; > }; > }; > + > + thc63lvd1024: lvds-decoder { > + compatible = "thine,thc63lvd1024"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + thc63lvd1024_in_0: endpoint { > + remote-endpoint = <&lvds0_out>; > + }; > + }; > + > + port@2{ > + reg = <2>; > + > + thc63lvd1024_out_2: endpoint { > + remote-endpoint = <&adv7511_in>; > + }; > + > + }; > + > + }; > + }; > }; > > &avb { > @@ -98,7 +125,7 @@ > port@0 { > reg = <0>; > adv7511_in: endpoint { > - remote-endpoint = <&lvds0_out>; > + remote-endpoint = <&thc63lvd1024_out_2>; > }; > }; > > @@ -152,8 +179,8 @@ > > ports { > port@1 { > - endpoint { > - remote-endpoint = <&adv7511_in>; > + lvds0_out: endpoint { > + remote-endpoint = <&thc63lvd1024_in_0>; > }; > }; > }; -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH v6 3/3] arm64: dts: renesas: Add LVDS decoder to R-Car V3M Eagle Date: Tue, 20 Mar 2018 15:01:57 +0200 Message-ID: <3127791.hHqCBynxiV@avalon> References: <1521213399-31947-1-git-send-email-jacopo+renesas@jmondi.org> <1521213399-31947-4-git-send-email-jacopo+renesas@jmondi.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1521213399-31947-4-git-send-email-jacopo+renesas@jmondi.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Jacopo Mondi Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, sergei.shtylyov@cogentembedded.com, airlied@linux.ie, magnus.damm@gmail.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, linux-renesas-soc@vger.kernel.org, horms@verge.net.au, geert@linux-m68k.org, dri-devel@lists.freedesktop.org, niklas.soderlund@ragnatech.se List-Id: devicetree@vger.kernel.org SGkgSmFjb3BvLAoKVGhhbmsgeW91IGZvciB0aGUgcGF0Y2guCgpPbiBGcmlkYXksIDE2IE1hcmNo IDIwMTggMTc6MTY6MzkgRUVUIEphY29wbyBNb25kaSB3cm90ZToKPiBUaGUgUi1DYXIgVjNNIEVh Z2xlIGJvYXJkIGluY2x1ZGVzIGEgdHJhbnNwYXJlbnQgVEhDNjNMVkQxMDI0IExWRFMKPiBkZWNv ZGVyLCBjb25uZWN0ZWQgdG8gdGhlIG9uLWNoaXAgTFZEUyBlbmNvZGVyIG91dHB1dCBvbiBvbmUg c2lkZQo+IGFuZCB0byBIRE1JIGVuY29kZXIgQURWNzUxMXcgb24gdGhlIG90aGVyIG9uZS4KPiAK PiBBcyB0aGUgZGVjb2RlciBkb2VzIG5vdCBuZWVkIGFueSBjb25maWd1cmF0aW9uIGl0IGhhcyBi ZWVuIHNvLWZhcgo+IG9taXR0ZWQgZnJvbSBEVFMuIE5vdyB0aGF0IGEgZHJpdmVyIGlzIGF2YWls YWJsZSwgZGVzY3JpYmUgaXQgaW4gRFQKPiBhcyB3ZWxsLgo+IAo+IFNpZ25lZC1vZmYtYnk6IEph Y29wbyBNb25kaSA8amFjb3BvK3JlbmVzYXNAam1vbmRpLm9yZz4KPiBSZXZpZXdlZC1ieTogQW5k cnplaiBIYWpkYSA8YS5oYWpkYUBzYW1zdW5nLmNvbT4KClRoZSBwYXRjaCBsb29rcyBPSyB0byBt ZSwgYnV0IEkgdGhpbmsgaXQgc2hvdWxkIGJlIHNxdWFzaGVkIHdpdGggTmlrbGFzJyBwYXRjaCAK dGhhdCBhZGRlZCBkaXNwbGF5IEhETUkgb3V0cHV0IHN1cHBvcnQgdG8gdGhlIFYzTSBFYWdsZSBE VC4KCj4gLS0tCj4gCj4gTGlzdCBvZiBwYXRjaCBkZXBlbmRlbmNpZXMsIGFzIG9mIHJlbmVzYXMt ZHJpdmVycy0yMDE4LTAzLTEzLXY0LjE2LXJjNToKPiAKPiAtIFtQQVRDSCB2MiAwLzVdIGFybTY0 OiBkdHM6IHJlbmVzYXM6IHI4YTc3OTcwOiBlbmFibGUgSERNSSBvdXRwdXQKPiAgICB3aGljaCBp bmNsdWRlcyBEVSwgTFZEUyBhbmQgRkNQRCBlbmFibGVtZW50IGZyb206Cj4gICBbUEFUQ0ggdjIg MC81XSBBZGQgUjhBNzc5NzAvVjNNU0sgTFZEUy9IRE1JIHN1cHBvcnQKPiAtIFtQQVRDSCB2NF0g djRsOiB2c3AxOiBGaXggdmlkZW8gb3V0cHV0IG9uIFI4QTc3OTcwCj4gCj4gUGF0Y2hlcyB0byBi ZSBhcHBsaWVkIG9uIHRvcCBvZgo+ICJhcm02NDogZHRzOiByZW5lc2FzOiBlYWdsZTogYWRkIEhE TUkgb3V0cHV0IHVzaW5nIHRoZSBBRFY3NTExVyIKPiAKPiBUaGFua3MKPiAgICBqCj4gLS0tCj4g IGFyY2gvYXJtNjQvYm9vdC9kdHMvcmVuZXNhcy9yOGE3Nzk3MC1lYWdsZS5kdHMgfCAzMyArKysr KysrKysrKysrKysrKysrKy0tLQo+ICAxIGZpbGUgY2hhbmdlZCwgMzAgaW5zZXJ0aW9ucygrKSwg MyBkZWxldGlvbnMoLSkKPiAKPiBkaWZmIC0tZ2l0IGEvYXJjaC9hcm02NC9ib290L2R0cy9yZW5l c2FzL3I4YTc3OTcwLWVhZ2xlLmR0cwo+IGIvYXJjaC9hcm02NC9ib290L2R0cy9yZW5lc2FzL3I4 YTc3OTcwLWVhZ2xlLmR0cyBpbmRleCBjMGZkMTQ0Li42OWY0M2I4Cj4gMTAwNjQ0Cj4gLS0tIGEv YXJjaC9hcm02NC9ib290L2R0cy9yZW5lc2FzL3I4YTc3OTcwLWVhZ2xlLmR0cwo+ICsrKyBiL2Fy Y2gvYXJtNjQvYm9vdC9kdHMvcmVuZXNhcy9yOGE3Nzk3MC1lYWdsZS5kdHMKPiBAQCAtNDIsNiAr NDIsMzMgQEAKPiAgCQkJfTsKPiAgCQl9Owo+ICAJfTsKPiArCj4gKwl0aGM2M2x2ZDEwMjQ6IGx2 ZHMtZGVjb2RlciB7Cj4gKwkJY29tcGF0aWJsZSA9ICJ0aGluZSx0aGM2M2x2ZDEwMjQiOwo+ICsK PiArCQlwb3J0cyB7Cj4gKwkJCSNhZGRyZXNzLWNlbGxzID0gPDE+Owo+ICsJCQkjc2l6ZS1jZWxs cyA9IDwwPjsKPiArCj4gKwkJCXBvcnRAMCB7Cj4gKwkJCQlyZWcgPSA8MD47Cj4gKwo+ICsJCQkJ dGhjNjNsdmQxMDI0X2luXzA6IGVuZHBvaW50IHsKPiArCQkJCQlyZW1vdGUtZW5kcG9pbnQgPSA8 Jmx2ZHMwX291dD47Cj4gKwkJCQl9Owo+ICsJCQl9Owo+ICsKPiArCQkJcG9ydEAyewo+ICsJCQkJ cmVnID0gPDI+Owo+ICsKPiArCQkJCXRoYzYzbHZkMTAyNF9vdXRfMjogZW5kcG9pbnQgewo+ICsJ CQkJCXJlbW90ZS1lbmRwb2ludCA9IDwmYWR2NzUxMV9pbj47Cj4gKwkJCQl9Owo+ICsKPiArCQkJ fTsKPiArCj4gKwkJfTsKPiArCX07Cj4gIH07Cj4gCj4gICZhdmIgewo+IEBAIC05OCw3ICsxMjUs NyBAQAo+ICAJCQlwb3J0QDAgewo+ICAJCQkJcmVnID0gPDA+Owo+ICAJCQkJYWR2NzUxMV9pbjog ZW5kcG9pbnQgewo+IC0JCQkJCXJlbW90ZS1lbmRwb2ludCA9IDwmbHZkczBfb3V0PjsKPiArCQkJ CQlyZW1vdGUtZW5kcG9pbnQgPSA8JnRoYzYzbHZkMTAyNF9vdXRfMj47Cj4gIAkJCQl9Owo+ICAJ CQl9Owo+IAo+IEBAIC0xNTIsOCArMTc5LDggQEAKPiAKPiAgCXBvcnRzIHsKPiAgCQlwb3J0QDEg ewo+IC0JCQllbmRwb2ludCB7Cj4gLQkJCQlyZW1vdGUtZW5kcG9pbnQgPSA8JmFkdjc1MTFfaW4+ Owo+ICsJCQlsdmRzMF9vdXQ6IGVuZHBvaW50IHsKPiArCQkJCXJlbW90ZS1lbmRwb2ludCA9IDwm dGhjNjNsdmQxMDI0X2luXzA+Owo+ICAJCQl9Owo+ICAJCX07Cj4gIAl9OwoKLS0gClJlZ2FyZHMs CgpMYXVyZW50IFBpbmNoYXJ0CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVz a3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9k cmktZGV2ZWwK