From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3C5A1FA372C for ; Thu, 7 Nov 2019 02:29:41 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 0752321882 for ; Thu, 7 Nov 2019 02:29:40 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="vd58pCKj" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732382AbfKGC3k (ORCPT ); Wed, 6 Nov 2019 21:29:40 -0500 Received: from perceval.ideasonboard.com ([213.167.242.64]:47364 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727328AbfKGC3k (ORCPT ); Wed, 6 Nov 2019 21:29:40 -0500 Received: from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 39F532D1; Thu, 7 Nov 2019 03:29:38 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1573093778; bh=10cPkHoiL4VtxxYfO8BfkPKU8pAwjaQbgE/SdPHPfQA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=vd58pCKjdA2EL4jUIFi6vRgerVlSxLI8b8ge/+SngBcPDCbAyprNVTwSyaWBJQx32 DBAhLPmZXi2i9p1819pI5XLJeaWlhTrHSm/Bopy43pmwOaFYA2mFu8iAhdZBGnS+gc /Db91FIIItFZU+nnIi2FPkrmr7Ve0yhrj8DmzkOo= Date: Thu, 7 Nov 2019 04:29:29 +0200 From: Laurent Pinchart To: Geert Uytterhoeven Cc: Kieran Bingham , David Airlie , Daniel Vetter , Rob Herring , Mark Rutland , Magnus Damm , dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH 1/3] dt-bindings: display: renesas: du: Add vendor prefix to vsps property Message-ID: <20191107022929.GK4878@pendragon.ideasonboard.com> References: <20191105183504.21447-1-geert+renesas@glider.be> <20191105183504.21447-2-geert+renesas@glider.be> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20191105183504.21447-2-geert+renesas@glider.be> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-renesas-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org Hi Geert, Thank you for the patch. On Tue, Nov 05, 2019 at 07:35:02PM +0100, Geert Uytterhoeven wrote: > The Renesas-specific "vsps" property lacks a vendor prefix. > Add a "renesas," prefix to comply with DT best practises. > > Move "renesas,vsps" below "renesas,cmms" to preserve alphabetical sort > order. > > Fixes: 06711e6385a4ab4c ("drm: rcar-du: Document the vsps property in the DT bindings") > Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart and taken in my tree. > --- > .../devicetree/bindings/display/renesas,du.txt | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt b/Documentation/devicetree/bindings/display/renesas,du.txt > index f6fdaa67c257a046..eb4ae41fe41f83c7 100644 > --- a/Documentation/devicetree/bindings/display/renesas,du.txt > +++ b/Documentation/devicetree/bindings/display/renesas,du.txt > @@ -41,15 +41,15 @@ Required Properties: > supplied they must be named "dclkin.x" with "x" being the input clock > numerical index. > > - - vsps: A list of phandle and channel index tuples to the VSPs that handle > - the memory interfaces for the DU channels. The phandle identifies the VSP > - instance that serves the DU channel, and the channel index identifies the > - LIF instance in that VSP. > - > - renesas,cmms: A list of phandles to the CMM instances present in the SoC, > one for each available DU channel. The property shall not be specified for > SoCs that do not provide any CMM (such as V3M and V3H). > > + - renesas,vsps: A list of phandle and channel index tuples to the VSPs that > + handle the memory interfaces for the DU channels. The phandle identifies the > + VSP instance that serves the DU channel, and the channel index identifies > + the LIF instance in that VSP. > + > Required nodes: > > The connections to the DU output video ports are modeled using the OF graph > @@ -96,8 +96,8 @@ Example: R8A7795 (R-Car H3) ES2.0 DU > <&cpg CPG_MOD 722>, > <&cpg CPG_MOD 721>; > clock-names = "du.0", "du.1", "du.2", "du.3"; > - vsps = <&vspd0 0>, <&vspd1 0>, <&vspd2 0>, <&vspd0 1>; > renesas,cmms = <&cmm0>, <&cmm1>, <&cmm2>, <&cmm3>; > + renesas,vsps = <&vspd0 0>, <&vspd1 0>, <&vspd2 0>, <&vspd0 1>; > > ports { > #address-cells = <1>; -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 1/3] dt-bindings: display: renesas: du: Add vendor prefix to vsps property Date: Thu, 7 Nov 2019 04:29:29 +0200 Message-ID: <20191107022929.GK4878@pendragon.ideasonboard.com> References: <20191105183504.21447-1-geert+renesas@glider.be> <20191105183504.21447-2-geert+renesas@glider.be> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1C5DB6EE70 for ; Thu, 7 Nov 2019 02:29:40 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20191105183504.21447-2-geert+renesas@glider.be> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Geert Uytterhoeven Cc: Mark Rutland , devicetree@vger.kernel.org, David Airlie , Kieran Bingham , Magnus Damm , dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, Rob Herring List-Id: dri-devel@lists.freedesktop.org SGkgR2VlcnQsCgpUaGFuayB5b3UgZm9yIHRoZSBwYXRjaC4KCk9uIFR1ZSwgTm92IDA1LCAyMDE5 IGF0IDA3OjM1OjAyUE0gKzAxMDAsIEdlZXJ0IFV5dHRlcmhvZXZlbiB3cm90ZToKPiBUaGUgUmVu ZXNhcy1zcGVjaWZpYyAidnNwcyIgcHJvcGVydHkgbGFja3MgYSB2ZW5kb3IgcHJlZml4Lgo+IEFk ZCBhICJyZW5lc2FzLCIgcHJlZml4IHRvIGNvbXBseSB3aXRoIERUIGJlc3QgcHJhY3Rpc2VzLgo+ IAo+IE1vdmUgInJlbmVzYXMsdnNwcyIgYmVsb3cgInJlbmVzYXMsY21tcyIgdG8gcHJlc2VydmUg YWxwaGFiZXRpY2FsIHNvcnQKPiBvcmRlci4KPiAKPiBGaXhlczogMDY3MTFlNjM4NWE0YWI0YyAo ImRybTogcmNhci1kdTogRG9jdW1lbnQgdGhlIHZzcHMgcHJvcGVydHkgaW4gdGhlIERUIGJpbmRp bmdzIikKPiBTaWduZWQtb2ZmLWJ5OiBHZWVydCBVeXR0ZXJob2V2ZW4gPGdlZXJ0K3JlbmVzYXNA Z2xpZGVyLmJlPgoKUmV2aWV3ZWQtYnk6IExhdXJlbnQgUGluY2hhcnQgPGxhdXJlbnQucGluY2hh cnRAaWRlYXNvbmJvYXJkLmNvbT4KCmFuZCB0YWtlbiBpbiBteSB0cmVlLgoKPiAtLS0KPiAgLi4u L2RldmljZXRyZWUvYmluZGluZ3MvZGlzcGxheS9yZW5lc2FzLGR1LnR4dCAgICAgICB8IDEyICsr KysrKy0tLS0tLQo+ICAxIGZpbGUgY2hhbmdlZCwgNiBpbnNlcnRpb25zKCspLCA2IGRlbGV0aW9u cygtKQo+IAo+IGRpZmYgLS1naXQgYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3Mv ZGlzcGxheS9yZW5lc2FzLGR1LnR4dCBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5n cy9kaXNwbGF5L3JlbmVzYXMsZHUudHh0Cj4gaW5kZXggZjZmZGFhNjdjMjU3YTA0Ni4uZWI0YWU0 MWZlNDFmODNjNyAxMDA2NDQKPiAtLS0gYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGlu Z3MvZGlzcGxheS9yZW5lc2FzLGR1LnR4dAo+ICsrKyBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJl ZS9iaW5kaW5ncy9kaXNwbGF5L3JlbmVzYXMsZHUudHh0Cj4gQEAgLTQxLDE1ICs0MSwxNSBAQCBS ZXF1aXJlZCBQcm9wZXJ0aWVzOgo+ICAgICAgICBzdXBwbGllZCB0aGV5IG11c3QgYmUgbmFtZWQg ImRjbGtpbi54IiB3aXRoICJ4IiBiZWluZyB0aGUgaW5wdXQgY2xvY2sKPiAgICAgICAgbnVtZXJp Y2FsIGluZGV4Lgo+ICAKPiAtICAtIHZzcHM6IEEgbGlzdCBvZiBwaGFuZGxlIGFuZCBjaGFubmVs IGluZGV4IHR1cGxlcyB0byB0aGUgVlNQcyB0aGF0IGhhbmRsZQo+IC0gICAgdGhlIG1lbW9yeSBp bnRlcmZhY2VzIGZvciB0aGUgRFUgY2hhbm5lbHMuIFRoZSBwaGFuZGxlIGlkZW50aWZpZXMgdGhl IFZTUAo+IC0gICAgaW5zdGFuY2UgdGhhdCBzZXJ2ZXMgdGhlIERVIGNoYW5uZWwsIGFuZCB0aGUg Y2hhbm5lbCBpbmRleCBpZGVudGlmaWVzIHRoZQo+IC0gICAgTElGIGluc3RhbmNlIGluIHRoYXQg VlNQLgo+IC0KPiAgICAtIHJlbmVzYXMsY21tczogQSBsaXN0IG9mIHBoYW5kbGVzIHRvIHRoZSBD TU0gaW5zdGFuY2VzIHByZXNlbnQgaW4gdGhlIFNvQywKPiAgICAgIG9uZSBmb3IgZWFjaCBhdmFp bGFibGUgRFUgY2hhbm5lbC4gVGhlIHByb3BlcnR5IHNoYWxsIG5vdCBiZSBzcGVjaWZpZWQgZm9y Cj4gICAgICBTb0NzIHRoYXQgZG8gbm90IHByb3ZpZGUgYW55IENNTSAoc3VjaCBhcyBWM00gYW5k IFYzSCkuCj4gIAo+ICsgIC0gcmVuZXNhcyx2c3BzOiBBIGxpc3Qgb2YgcGhhbmRsZSBhbmQgY2hh bm5lbCBpbmRleCB0dXBsZXMgdG8gdGhlIFZTUHMgdGhhdAo+ICsgICAgaGFuZGxlIHRoZSBtZW1v cnkgaW50ZXJmYWNlcyBmb3IgdGhlIERVIGNoYW5uZWxzLiBUaGUgcGhhbmRsZSBpZGVudGlmaWVz IHRoZQo+ICsgICAgVlNQIGluc3RhbmNlIHRoYXQgc2VydmVzIHRoZSBEVSBjaGFubmVsLCBhbmQg dGhlIGNoYW5uZWwgaW5kZXggaWRlbnRpZmllcwo+ICsgICAgdGhlIExJRiBpbnN0YW5jZSBpbiB0 aGF0IFZTUC4KPiArCj4gIFJlcXVpcmVkIG5vZGVzOgo+ICAKPiAgVGhlIGNvbm5lY3Rpb25zIHRv IHRoZSBEVSBvdXRwdXQgdmlkZW8gcG9ydHMgYXJlIG1vZGVsZWQgdXNpbmcgdGhlIE9GIGdyYXBo Cj4gQEAgLTk2LDggKzk2LDggQEAgRXhhbXBsZTogUjhBNzc5NSAoUi1DYXIgSDMpIEVTMi4wIERV Cj4gIAkJCSA8JmNwZyBDUEdfTU9EIDcyMj4sCj4gIAkJCSA8JmNwZyBDUEdfTU9EIDcyMT47Cj4g IAkJY2xvY2stbmFtZXMgPSAiZHUuMCIsICJkdS4xIiwgImR1LjIiLCAiZHUuMyI7Cj4gLQkJdnNw cyA9IDwmdnNwZDAgMD4sIDwmdnNwZDEgMD4sIDwmdnNwZDIgMD4sIDwmdnNwZDAgMT47Cj4gIAkJ cmVuZXNhcyxjbW1zID0gPCZjbW0wPiwgPCZjbW0xPiwgPCZjbW0yPiwgPCZjbW0zPjsKPiArCQly ZW5lc2FzLHZzcHMgPSA8JnZzcGQwIDA+LCA8JnZzcGQxIDA+LCA8JnZzcGQyIDA+LCA8JnZzcGQw IDE+Owo+ICAKPiAgCQlwb3J0cyB7Cj4gIAkJCSNhZGRyZXNzLWNlbGxzID0gPDE+OwoKLS0gClJl Z2FyZHMsCgpMYXVyZW50IFBpbmNoYXJ0Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZy ZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3Rp bmZvL2RyaS1kZXZlbA==