From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [v4,02/13] dt-bindings: usb: add documentation for typec port controller(TCPCI) From: Mats Karrman Message-Id: Date: Mon, 30 Apr 2018 09:41:07 +0200 To: Li Jun Cc: robh+dt@kernel.org, gregkh@linuxfoundation.org, heikki.krogerus@linux.intel.com, linux@roeck-us.net, a.hajda@samsung.com, shufan_lee@richtek.com, peter.chen@nxp.com, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-imx@nxp.com, devel@driverdev.osuosl.org List-ID: SGkgTGkgSnVuLAoKQXJlIHlvdSB3b3JraW5nIG9uIGFuIHVwZGF0ZWQgdmVyc2lvbiBvZiB0aGlz IHBhdGNoIHNlcmllcz8KSSdtIHBvbmRlcmluZyBvdGhlciBjaGFuZ2VzIHRoYXQgYnVpbGRzIG9u IHRoZXNlIHBhdGNoZXMgKHRoZSBkb2N1bWVudGF0aW9uCmFuZCB0aGUgZndub2RlIGFkZGVkIHRv IHRoZSB0Y3BjX2RldiBhbmQgdGNwbSBwcmltYXJpbHkpLgoKQnR3LCB0aGVyZSBpcyBhIHNlbWkt Y29sb24gbWlzc2luZyBpbiB5b3VyIGV4YW1wbGUgYmVsb3cuCgpCUiAvLyBNYXRzCgpPbiAyMDE4 LTAzLTI4IDE4OjA2LCBMaSBKdW4gd3JvdGU6Cgo+IFRDUENJIHN0YW5kcyBmb3IgdHlwZWMgcG9y dCBjb250cm9sbGVyIGludGVyZmFjZSwgaXRzIGltcGxlbWVudGF0aW9uCj4gaGFzIGZ1bGwgdHlw ZWMgcG9ydCBjb250cm9sIHdpdGggcG93ZXIgZGVsaXZlcnkgc3VwcG9ydCwgaXQncyBhCj4gc3Rh bmRhcmQgaTJjIHNsYXZlIHdpdGggR1BJTyBpbnB1dCBhcyBpcnEgaW50ZXJmYWNlLCBkZXRhaWwg c2VlIHNwZWMKPiAiVW5pdmVyc2FsIFNlcmlhbCBCdXMgVHlwZS1DIFBvcnQgQ29udHJvbGxlciBJ bnRlcmZhY2UgU3BlY2lmaWNhdGlvbgo+IFJldmlzaW9uIDEuMCwgVmVyc2lvbiAxLjEiCj4KPiBT aWduZWQtb2ZmLWJ5OiBMaSBKdW4gPGp1bi5saUBueHAuY29tPgo+IC0tLQo+ICAgLi4uL2Rldmlj ZXRyZWUvYmluZGluZ3MvdXNiL3R5cGVjLXRjcGNpLnR4dCAgICAgICAgfCAzMyArKysrKysrKysr KysrKysrKysrKysrCj4gICAxIGZpbGUgY2hhbmdlZCwgMzMgaW5zZXJ0aW9ucygrKQo+Cj4gZGlm ZiAtLWdpdCBhL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy91c2IvdHlwZWMtdGNw Y2kudHh0IGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3VzYi90eXBlYy10Y3Bj aS50eHQKPiBuZXcgZmlsZSBtb2RlIDEwMDY0NAo+IGluZGV4IDAwMDAwMDAuLjdhN2E4ZTAKPiAt LS0gL2Rldi9udWxsCj4gKysrIGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3Vz Yi90eXBlYy10Y3BjaS50eHQKPiBAQCAtMCwwICsxLDMzIEBACj4gK1RDUENJKFR5cGVjIHBvcnQg Y290cm9sbGVyIGludGVyZmFjZSkgYmluZGluZwo+ICstLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0KPiArCj4gK1JlcXVpcmVkIHByb3BlcnRpZXM6Cj4gKy0gY29t cGF0aWJsZTogICAgICAgc2hvdWxkIGJlICJ1c2ItdGNwY2ksY2hpcC1zcGVjaWZpYy1zdHJpbmci Lgo+ICstIHJlZzogICAgICAgICAgICAgIHRoZSBpMmMgc2xhdmUgYWRkcmVzcyBvZiB0eXBlYyBw b3J0IGNvbnRyb2xsZXIgZGV2aWNlLgo+ICstIGludGVycnVwdC1wYXJlbnQ6IHRoZSBwaGFuZGxl IHRvIHRoZSBpbnRlcnJ1cHQgY29udHJvbGxlciB3aGljaCBwcm92aWRlcwo+ICsgICAgICAgICAg ICAgICAgICAgIHRoZSBpbnRlcnJ1cHQuCj4gKy0gaW50ZXJydXB0czogICAgICAgaW50ZXJydXB0 IHNwZWNpZmljYXRpb24gZm9yIHRjcGNpIGFsZXJ0Lgo+ICsKPiArUmVxdWlyZWQgc3ViLW5vZGU6 Cj4gKy0gY29ubmVjdG9yOiBUaGUgInVzYi1jLWNvbm5lY3RvciIgYXR0YWNoZWQgdG8gdGhlIHRj cGNpIGNoaXAsIHRoZSBiaW5kaW5ncwo+ICsgIG9mIGNvbm5lY3RvciBub2RlIGFyZSBzcGVjaWZp ZWQgaW4KPiArICBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvY29ubmVjdG9yL3Vz Yi1jb25uZWN0b3IudHh0Cj4gKwo+ICtFeGFtcGxlOgo+ICsKPiArcHRuNTExMEA1MCB7Cj4gKwlj b21wYXRpYmxlID0gInVzYi10Y3BjaSxwdG41MTEwIjsKPiArCXJlZyA9IDwweDUwPjsKPiArCWlu dGVycnVwdC1wYXJlbnQgPSA8JmdwaW8zPjsKPiArCWludGVycnVwdHMgPSA8MyBJUlFfVFlQRV9M RVZFTF9MT1c+Owo+ICsKPiArCXVzYl9jb246IGNvbm5lY3RvciB7Cj4gKwkJY29tcGF0aWJsZSA9 ICJ1c2ItYy1jb25uZWN0b3IiOwo+ICsJCWxhYmVsID0gIlVTQi1DIjsKPiArCQlwb3J0LXR5cGUg PSAiZHVhbCI7Cj4gKwkJdHJ5LXBvd2VyLXJvbGUgPSAic2luayIKCkhlcmUhCgo+ICsJCXNvdXJj ZS1wZG9zID0gPDB4MzgwMTkwYzg+Owo+ICsJCXNpbmstcGRvcyA9IDwweDM4MDE5MGM4IDB4Mzgw MmQwYzg+Owo+ICsJCW9wLXNpbmstbWljcm93YXR0LWhvdXJzID0gPDkwMDAwMDA+Owo+ICsJfTsK PiArfTsKPgotLS0KVG8gdW5zdWJzY3JpYmUgZnJvbSB0aGlzIGxpc3Q6IHNlbmQgdGhlIGxpbmUg InVuc3Vic2NyaWJlIGxpbnV4LXVzYiIgaW4KdGhlIGJvZHkgb2YgYSBtZXNzYWdlIHRvIG1ham9y ZG9tb0B2Z2VyLmtlcm5lbC5vcmcKTW9yZSBtYWpvcmRvbW8gaW5mbyBhdCAgaHR0cDovL3ZnZXIu a2VybmVsLm9yZy9tYWpvcmRvbW8taW5mby5odG1sCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mats Karrman Subject: Re: [PATCH v4 02/13] dt-bindings: usb: add documentation for typec port controller(TCPCI) Date: Mon, 30 Apr 2018 09:41:07 +0200 Message-ID: References: <1522253178-32414-1-git-send-email-jun.li@nxp.com> <1522253178-32414-3-git-send-email-jun.li@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1522253178-32414-3-git-send-email-jun.li@nxp.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: Li Jun Cc: devel@driverdev.osuosl.org, peter.chen@nxp.com, heikki.krogerus@linux.intel.com, devicetree@vger.kernel.org, gregkh@linuxfoundation.org, linux-usb@vger.kernel.org, a.hajda@samsung.com, robh+dt@kernel.org, linux-imx@nxp.com, linux@roeck-us.net, shufan_lee@richtek.com List-Id: devicetree@vger.kernel.org Hi Li Jun, Are you working on an updated version of this patch series? I'm pondering other changes that builds on these patches (the documentation and the fwnode added to the tcpc_dev and tcpm primarily). Btw, there is a semi-colon missing in your example below. BR // Mats On 2018-03-28 18:06, Li Jun wrote: > TCPCI stands for typec port controller interface, its implementation > has full typec port control with power delivery support, it's a > standard i2c slave with GPIO input as irq interface, detail see spec > "Universal Serial Bus Type-C Port Controller Interface Specification > Revision 1.0, Version 1.1" > > Signed-off-by: Li Jun > --- > .../devicetree/bindings/usb/typec-tcpci.txt | 33 ++++++++++++++++++++++ > 1 file changed, 33 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/typec-tcpci.txt b/Documentation/devicetree/bindings/usb/typec-tcpci.txt > new file mode 100644 > index 0000000..7a7a8e0 > --- /dev/null > +++ b/Documentation/devicetree/bindings/usb/typec-tcpci.txt > @@ -0,0 +1,33 @@ > +TCPCI(Typec port cotroller interface) binding > +--------------------------------------------- > + > +Required properties: > +- compatible: should be "usb-tcpci,chip-specific-string". > +- reg: the i2c slave address of typec port controller device. > +- interrupt-parent: the phandle to the interrupt controller which provides > + the interrupt. > +- interrupts: interrupt specification for tcpci alert. > + > +Required sub-node: > +- connector: The "usb-c-connector" attached to the tcpci chip, the bindings > + of connector node are specified in > + Documentation/devicetree/bindings/connector/usb-connector.txt > + > +Example: > + > +ptn5110@50 { > + compatible = "usb-tcpci,ptn5110"; > + reg = <0x50>; > + interrupt-parent = <&gpio3>; > + interrupts = <3 IRQ_TYPE_LEVEL_LOW>; > + > + usb_con: connector { > + compatible = "usb-c-connector"; > + label = "USB-C"; > + port-type = "dual"; > + try-power-role = "sink" Here! > + source-pdos = <0x380190c8>; > + sink-pdos = <0x380190c8 0x3802d0c8>; > + op-sink-microwatt-hours = <9000000>; > + }; > +}; >