* [v3,02/12] dt-bindings: usb: add documentation for typec port controller(TCPCI)
@ 2018-03-20 2:28 Jun Li
0 siblings, 0 replies; 4+ messages in thread
From: Jun Li @ 2018-03-20 2:28 UTC (permalink / raw)
To: Mats Karrman, robh+dt, mark.rutland, gregkh, heikki.krogerus
Cc: a.hajda, linux, yueyao, shufan_lee, o_leveque, linux-usb, dl-linux-imx
SGkNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogTWF0cyBLYXJybWFuIFtt
YWlsdG86bWF0cy5kZXYubGlzdEBnbWFpbC5jb21dDQo+IFNlbnQ6IDIwMTjlubQz5pyIMTXml6Ug
MjM6NTQNCj4gVG86IEp1biBMaSA8anVuLmxpQG54cC5jb20+OyByb2JoK2R0QGtlcm5lbC5vcmc7
IG1hcmsucnV0bGFuZEBhcm0uY29tOw0KPiBncmVna2hAbGludXhmb3VuZGF0aW9uLm9yZzsgaGVp
a2tpLmtyb2dlcnVzQGxpbnV4LmludGVsLmNvbQ0KPiBDYzogYS5oYWpkYUBzYW1zdW5nLmNvbTsg
bGludXhAcm9lY2stdXMubmV0OyB5dWV5YW9AZ29vZ2xlLmNvbTsNCj4gc2h1ZmFuX2xlZUByaWNo
dGVrLmNvbTsgb19sZXZlcXVlQG9yYW5nZS5mcjsgbGludXgtdXNiQHZnZXIua2VybmVsLm9yZzsN
Cj4gZGwtbGludXgtaW14IDxsaW51eC1pbXhAbnhwLmNvbT4NCj4gU3ViamVjdDogUmU6IFtQQVRD
SCB2MyAwMi8xMl0gZHQtYmluZGluZ3M6IHVzYjogYWRkIGRvY3VtZW50YXRpb24gZm9yIHR5cGVj
DQo+IHBvcnQgY29udHJvbGxlcihUQ1BDSSkNCj4gDQo+IEhpLA0KPiANCj4gT24gMjAxOC0wMy0x
MyAxMDozNCwgTGkgSnVuIHdyb3RlOg0KPiANCj4gPiBUQ1BDSSBzdGFuZHMgZm9yIHR5cGVjIHBv
cnQgY29udHJvbGxlciBpbnRlcmZhY2UsIGl0cyBpbXBsZW1lbnRhdGlvbg0KPiA+IGhhcyBmdWxs
IHR5cGVjIHBvcnQgY29udHJvbCB3aXRoIHBvd2VyIGRlbGl2ZXJ5IHN1cHBvcnQsIGl0J3MgYQ0K
PiA+IHN0YW5kYXJkIGkyYyBzbGF2ZSB3aXRoIEdQSU8gaW5wdXQgYXMgaXJxIGludGVyZmFjZSwg
ZGV0YWlsIHNlZSBzcGVjDQo+ID4gIlVuaXZlcnNhbCBTZXJpYWwgQnVzIFR5cGUtQyBQb3J0IENv
bnRyb2xsZXIgSW50ZXJmYWNlIFNwZWNpZmljYXRpb24NCj4gPiBSZXZpc2lvbiAxLjAsIFZlcnNp
b24gMS4xIg0KPiA+DQo+ID4gU2lnbmVkLW9mZi1ieTogTGkgSnVuIDxqdW4ubGlAbnhwLmNvbT4N
Cj4gPiAtLS0NCj4gPiBjaGFuZ2UgZm9yIHYzOg0KPiA+IC0gY2hhbmdlIGNvbXBhdGlibGUgc3Ry
aW5nIGZyb20gInVzYix0Y3BjaSIgdG8gYmUNCj4gPiAgICAidXNiLXRjcGNpLGNoaXAtc3BlY2lm
aWMtc3RyaW5nIiwgdXBkYXRlIGV4YW1wbGUgb2YgaXQuDQo+ID4NCj4gPiAgIC4uLi9kZXZpY2V0
cmVlL2JpbmRpbmdzL3VzYi90eXBlYy10Y3BjaS50eHQgICAgICAgIHwgMzYNCj4gKysrKysrKysr
KysrKysrKysrKysrKw0KPiA+ICAgMSBmaWxlIGNoYW5nZWQsIDM2IGluc2VydGlvbnMoKykNCj4g
Pg0KPiA+IGRpZmYgLS1naXQgYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvdXNi
L3R5cGVjLXRjcGNpLnR4dA0KPiA+IGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdz
L3VzYi90eXBlYy10Y3BjaS50eHQNCj4gPiBuZXcgZmlsZSBtb2RlIDEwMDY0NA0KPiA+IGluZGV4
IDAwMDAwMDAuLmZlN2QyZmYNCj4gPiAtLS0gL2Rldi9udWxsDQo+ID4gKysrIGIvRG9jdW1lbnRh
dGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3VzYi90eXBlYy10Y3BjaS50eHQNCj4gPiBAQCAtMCww
ICsxLDM2IEBADQo+ID4gK1RDUENJKFR5cGVjIHBvcnQgY290cm9sbGVyIGludGVyZmFjZSkgYmlu
ZGluZw0KPiA+ICstLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0N
Cj4gPiArDQo+ID4gK1JlcXVpcmVkIHByb3BlcnRpZXM6DQo+ID4gKy0gY29tcGF0aWJsZTogICAg
ICAgc2hvdWxkIGJlICJ1c2ItdGNwY2ksY2hpcC1zcGVjaWZpYy1zdHJpbmciLg0KPiA+ICstIHJl
ZzogICAgICAgICAgICAgIHRoZSBpMmMgc2xhdmUgYWRkcmVzcyBvZiB0eXBlYyBwb3J0IGNvbnRy
b2xsZXIgZGV2aWNlLg0KPiA+ICstIGludGVycnVwdC1wYXJlbnQ6IHRoZSBwaGFuZGxlIHRvIHRo
ZSBpbnRlcnJ1cHQgY29udHJvbGxlciB3aGljaCBwcm92aWRlcw0KPiA+ICsgICAgICAgICAgICAg
ICAgICAgIHRoZSBpbnRlcnJ1cHQuDQo+ID4gKy0gaW50ZXJydXB0czogICAgICAgaW50ZXJydXB0
IHNwZWNpZmljYXRpb24gZm9yIHRjcGNpIGFsZXJ0Lg0KPiA+ICsNCj4gPiArUmVxdWlyZWQgc3Vi
LW5vZGU6DQo+ID4gKy0gY29ubmVjdG9yOiBUaGUgInVzYi1jLWNvbm5lY3RvciIgYXR0YWNoZWQg
dG8gdGhlIHRjcGNpIGNoaXAsIHRoZQ0KPiA+ICtiaW5kaW5ncw0KPiA+ICsgIG9mIGNvbm5lY3Rv
ciBub2RlIGFyZSBzcGVjaWZpZWQgaW4NCj4gPiArICBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUv
YmluZGluZ3MvY29ubmVjdG9yL3VzYi1jb25uZWN0b3IudHh0DQo+ID4gKw0KPiA+ICtFeGFtcGxl
Og0KPiA+ICsNCj4gPiArcHRuNTExMEA1MCB7DQo+ID4gKwljb21wYXRpYmxlID0gInVzYi10Y3Bj
aSxwdG41MTEwIjsNCj4gPiArCXJlZyA9IDwweDUwPjsNCj4gPiArCWludGVycnVwdC1wYXJlbnQg
PSA8JmdwaW8zPjsNCj4gPiArCWludGVycnVwdHMgPSA8MyBJUlFfVFlQRV9MRVZFTF9MT1c+Ow0K
PiA+ICsNCj4gPiArCXVzYl9jb246IGNvbm5lY3RvciB7DQo+ID4gKwkJY29tcGF0aWJsZSA9ICJ1
c2ItYy1jb25uZWN0b3IiOw0KPiA+ICsJCWxhYmVsID0gIlVTQi1DIjsNCj4gPiArCQlwb3J0LXR5
cGUgPSAiZHVhbCI7DQo+ID4gKwkJdHJ5LXBvd2VyLXJvbGUgPSAic2luayINCj4gPiArCQlzb3Vy
Y2UtcGRvcyA9IDwweDM4MDE5MGM4PjsNCj4gPiArCQlzaW5rLXBkb3MgPSA8MHgzODAxOTBjOCAw
eDM4MDJkMGM4PjsNCj4gPiArCQltYXgtc2luay1taWNyb3ZvbHQgPSA8OTAwMD47DQo+ID4gKwkJ
bWF4LXNpbmstbWljcm9hbXAgPSA8MjAwMD47DQo+ID4gKwkJbWF4LXNpbmstbWljcm93YXR0LWhv
dXJzID0gPDE4MDAwPjsNCj4gPiArCQlvcC1zaW5rLW1pY3Jvd2F0dC1ob3VycyA9IDw5MDAwPjsN
Cj4gDQo+IFZhbHVlcyBzaG91bGQgY2hhbmdlIHdpdGggc3VmZml4LCBlLmcuIDkwMDBtViA9IDkw
MDAwMDB1Vg0KPiANCg0KWWVzLCB0aGFua3MgZm9yIHBvaW50aW5nIHRoaXMgb3V0LCBJIHdpbGwg
dXBkYXRlIGluIHY0Lg0KDQpKdW4NCg0KPiAvLyBNYXRzDQo+IA0KPiA+ICsJfTsNCj4gPiArfTsN
Cj4gPg0K
---
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* [v3,02/12] dt-bindings: usb: add documentation for typec port controller(TCPCI)
@ 2018-03-20 2:36 Jun Li
0 siblings, 0 replies; 4+ messages in thread
From: Jun Li @ 2018-03-20 2:36 UTC (permalink / raw)
To: Mats Karrman, robh+dt, mark.rutland, gregkh, heikki.krogerus
Cc: a.hajda, linux, yueyao, shufan_lee, o_leveque, linux-usb, dl-linux-imx
> -----Original Message-----
> From: Jun Li
> Sent: 2018年3月20日 10:29
> To: Mats Karrman <mats.dev.list@gmail.com>; robh+dt@kernel.org;
> mark.rutland@arm.com; gregkh@linuxfoundation.org;
> heikki.krogerus@linux.intel.com
> Cc: a.hajda@samsung.com; linux@roeck-us.net; yueyao@google.com;
> shufan_lee@richtek.com; o_leveque@orange.fr; linux-usb@vger.kernel.org;
> dl-linux-imx <linux-imx@nxp.com>
> Subject: RE: [PATCH v3 02/12] dt-bindings: usb: add documentation for typec
> port controller(TCPCI)
>
> Hi
> > -----Original Message-----
> > From: Mats Karrman [mailto:mats.dev.list@gmail.com]
> > Sent: 2018年3月15日 23:54
> > To: Jun Li <jun.li@nxp.com>; robh+dt@kernel.org; mark.rutland@arm.com;
> > gregkh@linuxfoundation.org; heikki.krogerus@linux.intel.com
> > Cc: a.hajda@samsung.com; linux@roeck-us.net; yueyao@google.com;
> > shufan_lee@richtek.com; o_leveque@orange.fr;
> > linux-usb@vger.kernel.org; dl-linux-imx <linux-imx@nxp.com>
> > Subject: Re: [PATCH v3 02/12] dt-bindings: usb: add documentation for
> > typec port controller(TCPCI)
> >
> > Hi,
> >
> > On 2018-03-13 10:34, 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 <jun.li@nxp.com>
> > > ---
> > > change for v3:
> > > - change compatible string from "usb,tcpci" to be
> > > "usb-tcpci,chip-specific-string", update example of it.
> > >
> > > .../devicetree/bindings/usb/typec-tcpci.txt | 36
> > ++++++++++++++++++++++
> > > 1 file changed, 36 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..fe7d2ff
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/usb/typec-tcpci.txt
> > > @@ -0,0 +1,36 @@
> > > +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"
> > > + source-pdos = <0x380190c8>;
> > > + sink-pdos = <0x380190c8 0x3802d0c8>;
> > > + max-sink-microvolt = <9000>;
> > > + max-sink-microamp = <2000>;
> > > + max-sink-microwatt-hours = <18000>;
> > > + op-sink-microwatt-hours = <9000>;
> >
> > Values should change with suffix, e.g. 9000mV = 9000000uV
> >
>
> Yes, thanks for pointing this out, I will update in v4.
Just realized we are going to remove those properties, see[1]:
[1] https://www.spinics.net/lists/linux-usb/msg166986.html
anyway thank you for the comments.
Jun
>
> Jun
>
> > // Mats
> >
> > > + };
> > > +};
> > >
^ permalink raw reply [flat|nested] 4+ messages in thread
* [v3,02/12] dt-bindings: usb: add documentation for typec port controller(TCPCI)
@ 2018-03-15 15:53 Mats Karrman
0 siblings, 0 replies; 4+ messages in thread
From: Mats Karrman @ 2018-03-15 15:53 UTC (permalink / raw)
To: Li Jun, robh+dt, mark.rutland, gregkh, heikki.krogerus
Cc: a.hajda, linux, yueyao, shufan_lee, o_leveque, linux-usb, linux-imx
Hi,
On 2018-03-13 10:34, 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 <jun.li@nxp.com>
> ---
> change for v3:
> - change compatible string from "usb,tcpci" to be
> "usb-tcpci,chip-specific-string", update example of it.
>
> .../devicetree/bindings/usb/typec-tcpci.txt | 36 ++++++++++++++++++++++
> 1 file changed, 36 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..fe7d2ff
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/typec-tcpci.txt
> @@ -0,0 +1,36 @@
> +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"
> + source-pdos = <0x380190c8>;
> + sink-pdos = <0x380190c8 0x3802d0c8>;
> + max-sink-microvolt = <9000>;
> + max-sink-microamp = <2000>;
> + max-sink-microwatt-hours = <18000>;
> + op-sink-microwatt-hours = <9000>;
Values should change with suffix, e.g. 9000mV = 9000000uV
// Mats
> + };
> +};
>
---
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* [v3,02/12] dt-bindings: usb: add documentation for typec port controller(TCPCI)
@ 2018-03-13 9:34 Jun Li
0 siblings, 0 replies; 4+ messages in thread
From: Jun Li @ 2018-03-13 9:34 UTC (permalink / raw)
To: robh+dt, mark.rutland, gregkh, heikki.krogerus
Cc: a.hajda, jun.li, linux, yueyao, shufan_lee, o_leveque, linux-usb,
linux-imx
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 <jun.li@nxp.com>
---
change for v3:
- change compatible string from "usb,tcpci" to be
"usb-tcpci,chip-specific-string", update example of it.
.../devicetree/bindings/usb/typec-tcpci.txt | 36 ++++++++++++++++++++++
1 file changed, 36 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..fe7d2ff
--- /dev/null
+++ b/Documentation/devicetree/bindings/usb/typec-tcpci.txt
@@ -0,0 +1,36 @@
+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"
+ source-pdos = <0x380190c8>;
+ sink-pdos = <0x380190c8 0x3802d0c8>;
+ max-sink-microvolt = <9000>;
+ max-sink-microamp = <2000>;
+ max-sink-microwatt-hours = <18000>;
+ op-sink-microwatt-hours = <9000>;
+ };
+};
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-03-20 2:36 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-20 2:28 [v3,02/12] dt-bindings: usb: add documentation for typec port controller(TCPCI) Jun Li
-- strict thread matches above, loose matches on Subject: below --
2018-03-20 2:36 Jun Li
2018-03-15 15:53 Mats Karrman
2018-03-13 9:34 Jun Li
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.