From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH 2/2] drm: bridge: add sil902x DT bindings doc Date: Wed, 6 Jan 2016 14:24:57 +0100 Message-ID: <20160106142457.28c8af15@bbrezillon> References: <1452079551-30914-1-git-send-email-boris.brezillon@free-electrons.com> <1452079551-30914-2-git-send-email-boris.brezillon@free-electrons.com> <20160106131959.GA18121@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20160106131959.GA18121@rob-hp-laptop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Rob Herring Cc: Mark Rutland , devicetree@vger.kernel.org, Pawel Moll , Ian Campbell , Nicolas Ferre , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Alexandre Belloni , Kumar Gala , Jean-Christophe Plagniol-Villard List-Id: devicetree@vger.kernel.org SGkgUm9iLAoKT24gV2VkLCA2IEphbiAyMDE2IDA3OjE5OjU5IC0wNjAwClJvYiBIZXJyaW5nIDxy b2JoQGtlcm5lbC5vcmc+IHdyb3RlOgoKPiBPbiBXZWQsIEphbiAwNiwgMjAxNiBhdCAxMjoyNTo1 MVBNICswMTAwLCBCb3JpcyBCcmV6aWxsb24gd3JvdGU6Cj4gPiBBZGQgU2lsOTAyMiBEVCBiaW5k aW5ncyBkZXNjcmlwdGlvbi4KPiA+IAo+ID4gU2lnbmVkLW9mZi1ieTogQm9yaXMgQnJlemlsbG9u IDxib3Jpcy5icmV6aWxsb25AZnJlZS1lbGVjdHJvbnMuY29tPgo+ID4gLS0tCj4gPiAgLi4uL2Rl dmljZXRyZWUvYmluZGluZ3MvZGlzcGxheS9icmlkZ2Uvc2lsOTAyeC50eHQgfCAzMSArKysrKysr KysrKysrKysrKysrKysrCj4gPiAgMSBmaWxlIGNoYW5nZWQsIDMxIGluc2VydGlvbnMoKykKPiA+ ICBjcmVhdGUgbW9kZSAxMDA2NDQgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rp c3BsYXkvYnJpZGdlL3NpbDkwMngudHh0Cj4gPiAKPiA+IGRpZmYgLS1naXQgYS9Eb2N1bWVudGF0 aW9uL2RldmljZXRyZWUvYmluZGluZ3MvZGlzcGxheS9icmlkZ2Uvc2lsOTAyeC50eHQgYi9Eb2N1 bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvZGlzcGxheS9icmlkZ2Uvc2lsOTAyeC50eHQK PiA+IG5ldyBmaWxlIG1vZGUgMTAwNjQ0Cj4gPiBpbmRleCAwMDAwMDAwLi43ZjEzMzlmCj4gPiAt LS0gL2Rldi9udWxsCj4gPiArKysgYi9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3Mv ZGlzcGxheS9icmlkZ2Uvc2lsOTAyeC50eHQKPiA+IEBAIC0wLDAgKzEsMzEgQEAKPiA+ICtzaWw5 MDJ4IEhETUkgYnJpZGdlIGJpbmRpbmdzCj4gPiArCj4gPiArUmVxdWlyZWQgcHJvcGVydGllczoK PiA+ICsJLSBjb21wYXRpYmxlOiAic2lsLHNpbDkwMjIiCj4gPiArCS0gcmVnOiBpMmMgYWRkcmVz cyBvZiB0aGUgYnJpZGdlCj4gPiArCS0gcmVzZXQtZ3Bpb3M6IE9GIGRldmljZS10cmVlIGdwaW8g c3BlY2lmaWNhdGlvbiBmb3IgUlNUX04gcGluLgo+ID4gKwo+ID4gK09wdGlvbmFsIHByb3BlcnRp ZXM6Cj4gPiArCS0gaW50ZXJydXB0cy1leHRlbmRlZCBvciBpbnRlcnJ1cHQtcGFyZW50ICsgaW50 ZXJydXB0czogZGVzY3JpYmUKPiA+ICsJICB0aGUgaW50ZXJydXB0IGxpbmUgdXNlZCB0byBpbmZv cm0gdGhlIGhvc3QgYWJvdXQgaG90cGx1ZyBldmVudHMuCj4gPiArCj4gPiArT3B0aW9uYWwgc3Vi bm9kZXM6Cj4gPiArCS0gdmlkZW8gaW5wdXQ6IERldmljZSBub2RlIGNhbiBjb250YWluIHZpZGVv IGlucHV0IHBvcnQgbm9kZSB0bwo+ID4gKwkgIGNvbm5lY3QgdGhlIGJyaWRnZSB0byBhIGRpc3Bs YXkgY29udHJvbGxlciBvdXRwdXQgKFNlZSB0aGlzCj4gPiArCSAgZG9jdW1lbnRhdGlvbiBbMV0p Lgo+ID4gKwo+ID4gK1sxXTogRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL21lZGlh L3ZpZGVvLWludGVyZmFjZXMudHh0Cj4gPiArCj4gPiArRXhhbXBsZToKPiA+ICsJaGRtaS1icmlk Z2VAMzkgewo+ID4gKwkJY29tcGF0aWJsZSA9ICJzaWwsc2lsOTAyMiI7Cj4gPiArCQlyZWcgPSA8 MHgzOT47Cj4gPiArCQlyZXNldC1ncGlvcyA9IDwmZ3B4MSA1IDEgMCAwPjsKPiAKPiA0IEdQSU8g Y2VsbHM/IFZhbGlkLCBidXQgdW51c3VhbC4KCkhlaGUsIEkgYmxpbmRseSBjb3BpZWQgbnhwLHB0 bjM0NjAgZG9jLCB3aGljaCBpcyBkZWZpbmluZyBzdWNoIEdQSU8KZGVzY3JpcHRvcnMgOi0pLiBJ IGNhbiBjaGFuZ2UgdGhhdCBpZiB5b3Ugd2FudC4KCj4gCj4gPiArCQlwb3J0cyB7Cj4gPiArCQkJ cG9ydEAwIHsKPiAKPiBFaXRoZXIgbmVlZCBhIHJlZyBwcm9wZXJ0eSBoZXJlIG9yIGRyb3AgdGhl IHVuaXQgYWRkcmVzcy4gWW91IGNvdWxkIAo+IHJlbW92ZSBwb3J0cyBhcyB3ZWxsLgoKQWN0dWFs bHkgd2UnbGwgbGlrZWx5IGRlZmluZSBhIDJuZCBwb3J0IGZvciB0aGUgYXVkaW8gaW5wdXQsIHNv IEknZApwcmVmZXIga2VlcGluZyB0aGUgcG9ydHMgYW5kIEAwIHN1ZmZpeCBhbmQgYWRkaW5nIGEg cmVnIHByb3BlcnR5IHJhdGhlcgp0aGFuIGp1c3QgZGVmaW5pbmcgcG9ydCB7IC4uLiB9OwoKQmVz dCBSZWdhcmRzLAoKQm9yaXMKCi0tIApCb3JpcyBCcmV6aWxsb24sIEZyZWUgRWxlY3Ryb25zCkVt YmVkZGVkIExpbnV4IGFuZCBLZXJuZWwgZW5naW5lZXJpbmcKaHR0cDovL2ZyZWUtZWxlY3Ryb25z LmNvbQpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmkt ZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cDov L2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752585AbcAFNZF (ORCPT ); Wed, 6 Jan 2016 08:25:05 -0500 Received: from down.free-electrons.com ([37.187.137.238]:53928 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751930AbcAFNZA (ORCPT ); Wed, 6 Jan 2016 08:25:00 -0500 Date: Wed, 6 Jan 2016 14:24:57 +0100 From: Boris Brezillon To: Rob Herring Cc: David Airlie , Daniel Vetter , dri-devel@lists.freedesktop.org, Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree@vger.kernel.org, Nicolas Ferre , Jean-Christophe Plagniol-Villard , Alexandre Belloni , linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] drm: bridge: add sil902x DT bindings doc Message-ID: <20160106142457.28c8af15@bbrezillon> In-Reply-To: <20160106131959.GA18121@rob-hp-laptop> References: <1452079551-30914-1-git-send-email-boris.brezillon@free-electrons.com> <1452079551-30914-2-git-send-email-boris.brezillon@free-electrons.com> <20160106131959.GA18121@rob-hp-laptop> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.27; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On Wed, 6 Jan 2016 07:19:59 -0600 Rob Herring wrote: > On Wed, Jan 06, 2016 at 12:25:51PM +0100, Boris Brezillon wrote: > > Add Sil9022 DT bindings description. > > > > Signed-off-by: Boris Brezillon > > --- > > .../devicetree/bindings/display/bridge/sil902x.txt | 31 ++++++++++++++++++++++ > > 1 file changed, 31 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/display/bridge/sil902x.txt > > > > diff --git a/Documentation/devicetree/bindings/display/bridge/sil902x.txt b/Documentation/devicetree/bindings/display/bridge/sil902x.txt > > new file mode 100644 > > index 0000000..7f1339f > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/display/bridge/sil902x.txt > > @@ -0,0 +1,31 @@ > > +sil902x HDMI bridge bindings > > + > > +Required properties: > > + - compatible: "sil,sil9022" > > + - reg: i2c address of the bridge > > + - reset-gpios: OF device-tree gpio specification for RST_N pin. > > + > > +Optional properties: > > + - interrupts-extended or interrupt-parent + interrupts: describe > > + the interrupt line used to inform the host about hotplug events. > > + > > +Optional subnodes: > > + - video input: Device node can contain video input port node to > > + connect the bridge to a display controller output (See this > > + documentation [1]). > > + > > +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt > > + > > +Example: > > + hdmi-bridge@39 { > > + compatible = "sil,sil9022"; > > + reg = <0x39>; > > + reset-gpios = <&gpx1 5 1 0 0>; > > 4 GPIO cells? Valid, but unusual. Hehe, I blindly copied nxp,ptn3460 doc, which is defining such GPIO descriptors :-). I can change that if you want. > > > + ports { > > + port@0 { > > Either need a reg property here or drop the unit address. You could > remove ports as well. Actually we'll likely define a 2nd port for the audio input, so I'd prefer keeping the ports and @0 suffix and adding a reg property rather than just defining port { ... }; Best Regards, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com