From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from galahad.ideasonboard.com ([185.26.127.97]:42955 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751742AbdCCTT3 (ORCPT ); Fri, 3 Mar 2017 14:19:29 -0500 From: Laurent Pinchart To: Geert Uytterhoeven Cc: Geert Uytterhoeven , David Airlie , Rob Herring , Mark Rutland , DRI Development , Linux-Renesas , "devicetree@vger.kernel.org" Subject: Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties Date: Fri, 03 Mar 2017 21:17:58 +0200 Message-ID: <5304812.zzcGWdqkpi@avalon> In-Reply-To: References: <1488547835-25229-1-git-send-email-geert+renesas@glider.be> <2423895.QcDWT1fsiC@avalon> 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 Geert, On Friday 03 Mar 2017 20:03:09 Geert Uytterhoeven wrote: > On Fri, Mar 3, 2017 at 3:41 PM, Laurent Pinchart wrote: > > On Friday 03 Mar 2017 14:30:35 Geert Uytterhoeven wrote: > >> Document the optional properties for describing module resets, to > >> support resetting display channels and LVDS encoders on R-Car Gen2 and > >> Gen3. > >> > >> Signed-off-by: Geert Uytterhoeven > >> --- > >> > >> Documentation/devicetree/bindings/display/renesas,du.txt | 10 ++++++++++ > >> 1 file changed, 10 insertions(+) > >> > >> diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt > >> b/Documentation/devicetree/bindings/display/renesas,du.txt index > >> 1a02f099a0ff0a3a..7873d53764a297c2 100644 > >> --- a/Documentation/devicetree/bindings/display/renesas,du.txt > >> +++ b/Documentation/devicetree/bindings/display/renesas,du.txt > >> > >> @@ -36,6 +36,16 @@ Required Properties: > >> When supplied they must be named "dclkin.x" with "x" being the > >> input clock numerical index. > >> > >> +Optional properties: > >> + - resets: A list of phandles + reset-specifier pairs, one for each > >> entry in > > > > s/phandlers/phandle/ > > You're seeing typos that do not exist ;-) > > >> + the reset-names property. > > > > What's in the reset specifier ? > > That depends on the reset provider. > > See "[v2,1/4] dt-bindings: clock: renesas: cpg-mssr: Document reset control > support" (https://patchwork.kernel.org/patch/9536627/) for the Renesas > CPG/MSSR case. > > E.g. "resets = <&cpg 310>;" Thanks. I never know whether the specified is the combination of phandle + data, or is only the cells following the phandle. -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties Date: Fri, 03 Mar 2017 21:17:58 +0200 Message-ID: <5304812.zzcGWdqkpi@avalon> References: <1488547835-25229-1-git-send-email-geert+renesas@glider.be> <2423895.QcDWT1fsiC@avalon> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: 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" , Geert Uytterhoeven , DRI Development , Linux-Renesas , Rob Herring List-Id: devicetree@vger.kernel.org SGkgR2VlcnQsCgpPbiBGcmlkYXkgMDMgTWFyIDIwMTcgMjA6MDM6MDkgR2VlcnQgVXl0dGVyaG9l dmVuIHdyb3RlOgo+IE9uIEZyaSwgTWFyIDMsIDIwMTcgYXQgMzo0MSBQTSwgTGF1cmVudCBQaW5j aGFydCB3cm90ZToKPiA+IE9uIEZyaWRheSAwMyBNYXIgMjAxNyAxNDozMDozNSBHZWVydCBVeXR0 ZXJob2V2ZW4gd3JvdGU6Cj4gPj4gRG9jdW1lbnQgdGhlIG9wdGlvbmFsIHByb3BlcnRpZXMgZm9y IGRlc2NyaWJpbmcgbW9kdWxlIHJlc2V0cywgdG8KPiA+PiBzdXBwb3J0IHJlc2V0dGluZyBkaXNw bGF5IGNoYW5uZWxzIGFuZCBMVkRTIGVuY29kZXJzIG9uIFItQ2FyIEdlbjIgYW5kCj4gPj4gR2Vu My4KPiA+PiAKPiA+PiBTaWduZWQtb2ZmLWJ5OiBHZWVydCBVeXR0ZXJob2V2ZW4gPGdlZXJ0K3Jl bmVzYXNAZ2xpZGVyLmJlPgo+ID4+IC0tLQo+ID4+IAo+ID4+ICBEb2N1bWVudGF0aW9uL2Rldmlj ZXRyZWUvYmluZGluZ3MvZGlzcGxheS9yZW5lc2FzLGR1LnR4dCB8IDEwICsrKysrKysrKysKPiA+ PiAgMSBmaWxlIGNoYW5nZWQsIDEwIGluc2VydGlvbnMoKykKPiA+PiAKPiA+PiBkaWZmIC0tZ2l0 IGEvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvcmVuZXNhcyxkdS50 eHQKPiA+PiBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9kaXNwbGF5L3JlbmVz YXMsZHUudHh0IGluZGV4Cj4gPj4gMWEwMmYwOTlhMGZmMGEzYS4uNzg3M2Q1Mzc2NGEyOTdjMiAx MDA2NDQKPiA+PiAtLS0gYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvZGlzcGxh eS9yZW5lc2FzLGR1LnR4dAo+ID4+ICsrKyBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5k aW5ncy9kaXNwbGF5L3JlbmVzYXMsZHUudHh0Cj4gPj4gCj4gPj4gQEAgLTM2LDYgKzM2LDE2IEBA IFJlcXVpcmVkIFByb3BlcnRpZXM6Cj4gPj4gICAgICAgIFdoZW4gc3VwcGxpZWQgdGhleSBtdXN0 IGJlIG5hbWVkICJkY2xraW4ueCIgd2l0aCAieCIgYmVpbmcgdGhlCj4gPj4gICAgICAgIGlucHV0 IGNsb2NrIG51bWVyaWNhbCBpbmRleC4KPiA+PiAKPiA+PiArT3B0aW9uYWwgcHJvcGVydGllczoK PiA+PiArICAtIHJlc2V0czogQSBsaXN0IG9mIHBoYW5kbGVzICsgcmVzZXQtc3BlY2lmaWVyIHBh aXJzLCBvbmUgZm9yIGVhY2gKPiA+PiBlbnRyeSBpbgo+ID4gCj4gPiBzL3BoYW5kbGVycy9waGFu ZGxlLwo+IAo+IFlvdSdyZSBzZWVpbmcgdHlwb3MgdGhhdCBkbyBub3QgZXhpc3QgOy0pCj4gCj4g Pj4gKyAgICB0aGUgcmVzZXQtbmFtZXMgcHJvcGVydHkuCj4gPiAKPiA+IFdoYXQncyBpbiB0aGUg cmVzZXQgc3BlY2lmaWVyID8KPiAKPiBUaGF0IGRlcGVuZHMgb24gdGhlIHJlc2V0IHByb3ZpZGVy Lgo+IAo+IFNlZSAiW3YyLDEvNF0gZHQtYmluZGluZ3M6IGNsb2NrOiByZW5lc2FzOiBjcGctbXNz cjogRG9jdW1lbnQgcmVzZXQgY29udHJvbAo+IHN1cHBvcnQiIChodHRwczovL3BhdGNod29yay5r ZXJuZWwub3JnL3BhdGNoLzk1MzY2MjcvKSBmb3IgdGhlIFJlbmVzYXMKPiBDUEcvTVNTUiBjYXNl Lgo+IAo+IEUuZy4gInJlc2V0cyA9IDwmY3BnIDMxMD47IgoKVGhhbmtzLiBJIG5ldmVyIGtub3cg d2hldGhlciB0aGUgc3BlY2lmaWVkIGlzIHRoZSBjb21iaW5hdGlvbiBvZiBwaGFuZGxlICsgCmRh dGEsIG9yIGlzIG9ubHkgdGhlIGNlbGxzIGZvbGxvd2luZyB0aGUgcGhhbmRsZS4KCi0tIApSZWdh cmRzLAoKTGF1cmVudCBQaW5jaGFydAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJl ZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGlu Zm8vZHJpLWRldmVsCg==