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: [v5,01/14] dt-bindings: connector: add properties for typec From: Oliver Neukum Message-Id: <1525339063.12348.4.camel@suse.com> Date: Thu, 03 May 2018 11:17:43 +0200 To: Jun Li , "robh+dt@kernel.org" , "heikki.krogerus@linux.intel.com" , "gregkh@linuxfoundation.org" , "linux@roeck-us.net" Cc: "gsomlo@gmail.com" , dl-linux-imx , Peter Chen , "shufan_lee@richtek.com" , "a.hajda@samsung.com" , "cw00.choi@samsung.com" , "devicetree@vger.kernel.org" , "linux-usb@vger.kernel.org" List-ID: QW0gRG9ubmVyc3RhZywgZGVuIDAzLjA1LjIwMTgsIDA4OjM1ICswMDAwIHNjaHJpZWIgSnVuIExp Ogo+IEhpCj4gPiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQo+ID4gRnJvbTogT2xpdmVyIE5l dWt1bSBbbWFpbHRvOm9uZXVrdW1Ac3VzZS5jb21dCj4gPiBTZW50OiAyMDE45bm0NeaciDPml6Ug MTU6MjcKPiA+IFRvOiBKdW4gTGkgPGp1bi5saUBueHAuY29tPjsgcm9iaCtkdEBrZXJuZWwub3Jn Owo+ID4gaGVpa2tpLmtyb2dlcnVzQGxpbnV4LmludGVsLmNvbTsgZ3JlZ2toQGxpbnV4Zm91bmRh dGlvbi5vcmc7Cj4gPiBsaW51eEByb2Vjay11cy5uZXQKPiA+IENjOiBnc29tbG9AZ21haWwuY29t OyBkbC1saW51eC1pbXggPGxpbnV4LWlteEBueHAuY29tPjsgUGV0ZXIgQ2hlbgo+ID4gPHBldGVy LmNoZW5AbnhwLmNvbT47IHNodWZhbl9sZWVAcmljaHRlay5jb207IGEuaGFqZGFAc2Ftc3VuZy5j b207Cj4gPiBjdzAwLmNob2lAc2Ftc3VuZy5jb207IGRldmljZXRyZWVAdmdlci5rZXJuZWwub3Jn Owo+ID4gbGludXgtdXNiQHZnZXIua2VybmVsLm9yZwo+ID4gU3ViamVjdDogUmU6IFtQQVRDSCB2 NSAwMS8xNF0gZHQtYmluZGluZ3M6IGNvbm5lY3RvcjogYWRkIHByb3BlcnRpZXMgZm9yIHR5cGVj Cj4gPiAKPiA+IEFtIERvbm5lcnN0YWcsIGRlbiAwMy4wNS4yMDE4LCAwODoyNCArMDgwMCBzY2hy aWViIExpIEp1bjoKPiA+ID4gK09wdGlvbmFsIHByb3BlcnRpZXMgZm9yIHVzYi1jLWNvbm5lY3Rv cjoKPiA+ID4gKy0gcG93ZXItcm9sZTogc2hvdWxkIGJlIG9uZSBvZiAic291cmNlIiwgInNpbmsi IG9yICJkdWFsIihEUlApIGlmCj4gPiA+ICt0eXBlYwo+ID4gPiArwqAgY29ubmVjdG9yIGhhcyBw b3dlciBzdXBwb3J0Lgo+ID4gPiArLSB0cnktcG93ZXItcm9sZTogcHJlZmVycmVkIHBvd2VyIHJv bGUgaWYgImR1YWwiKERSUCkgY2FuIHN1cHBvcnQKPiA+ID4gK1RyeS5TTksKPiA+ID4gK8KgIG9y IFRyeS5TUkMsIHNob3VsZCBiZSAic2luayIgZm9yIFRyeS5TTksgb3IgInNvdXJjZSIgZm9yIFRy eS5TUkMuCj4gPiA+ICstIGRhdGEtcm9sZTogc2hvdWxkIGJlIG9uZSBvZiAiaG9zdCIsICJkZXZp Y2UiLCAiZHVhbCIoRFJEKSBpZiB0eXBlYwo+ID4gPiArwqAgY29ubmVjdG9yIHN1cHBvcnRzIFVT QiBkYXRhLgo+ID4gCj4gPiBIaSwKPiA+IAo+ID4gaXMgdGhpcyByZWFsbHkgY29ycmVjdD8KPiA+ IAo+ID4gQ2FuIG9uZSBpbXBsZW1lbnQgYSBkZXZpY2UgdGhhdCBjYW4gb3BlcmF0ZSBhcyBlaXRo ZXIgREZQIG9yIFVGUCwgYnV0IG5vdAo+ID4gaW1wbGVtZW50cyB0aGUgZHluYW1pYyByb2xlIHN3 aXRjaCB0aGF0IGEgRFJQIG11c3Qgc3VwcG9ydD8KPiAKPiBZb3UgbWVhbiBhIHBvcnQgd2l0aCBE UkQgb24gZGF0YSBidXQgbm90IERSUCBvbiBwb3dlcj8KPiAKPiBUaGUgZGF0YS1yb2xlIGlzIG5l d2x5IGFkZGVkIGFzIHRoZSBkYXRhIHJvbGUgaXMgbm90IGNvdXBsZWQgd2l0aCBwb3dlcgoKTm8s IEkgbWVhbnQgZGF0YSByb2xlLiBBcyBmYXIgYXMgSSBjYW4gdGVsbCBmb3IgYSBEUlAgeW91IG5l ZWQgdG8KaW1wbGVtZW50IHRoZSBkZXRlY3Rpb24gbG9naWMgZGVzY3JpYmVkIGluIGNoYXB0ZXIg NCBvZiB0aGUgc3BlYy4KSSBjYW4gc2VlIG5vIHJlYXNvbiB3aHkgeW91IGNvdWxkbid0IGJ1aWxk IGEgcG9ydCB0aGF0IGNhbiBiZSBzd2l0Y2hlZApiZXR3ZWVuIHRoZSBkYXRhIHJvbGVzIGJ1dCBu b3QgaW1wbGVtZW50IHRoYXQgbG9naWMuCgoJUmVnYXJkcwoJCU9saXZlcgotLS0KVG8gdW5zdWJz Y3JpYmUgZnJvbSB0aGlzIGxpc3Q6IHNlbmQgdGhlIGxpbmUgInVuc3Vic2NyaWJlIGxpbnV4LXVz YiIgaW4KdGhlIGJvZHkgb2YgYSBtZXNzYWdlIHRvIG1ham9yZG9tb0B2Z2VyLmtlcm5lbC5vcmcK TW9yZSBtYWpvcmRvbW8gaW5mbyBhdCAgaHR0cDovL3ZnZXIua2VybmVsLm9yZy9tYWpvcmRvbW8t aW5mby5odG1sCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1525339063.12348.4.camel@suse.com> Subject: Re: [PATCH v5 01/14] dt-bindings: connector: add properties for typec From: Oliver Neukum Date: Thu, 03 May 2018 11:17:43 +0200 In-Reply-To: References: <1525307094-27402-1-git-send-email-jun.li@nxp.com> <1525307094-27402-2-git-send-email-jun.li@nxp.com> <1525332431.12348.1.camel@suse.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit To: Jun Li , "robh+dt@kernel.org" , "heikki.krogerus@linux.intel.com" , "gregkh@linuxfoundation.org" , "linux@roeck-us.net" Cc: "gsomlo@gmail.com" , dl-linux-imx , Peter Chen , "shufan_lee@richtek.com" , "a.hajda@samsung.com" , "cw00.choi@samsung.com" , "devicetree@vger.kernel.org" , "linux-usb@vger.kernel.org" List-ID: Am Donnerstag, den 03.05.2018, 08:35 +0000 schrieb Jun Li: > Hi > > -----Original Message----- > > From: Oliver Neukum [mailto:oneukum@suse.com] > > Sent: 2018年5月3日 15:27 > > To: Jun Li ; robh+dt@kernel.org; > > heikki.krogerus@linux.intel.com; gregkh@linuxfoundation.org; > > linux@roeck-us.net > > Cc: gsomlo@gmail.com; dl-linux-imx ; Peter Chen > > ; shufan_lee@richtek.com; a.hajda@samsung.com; > > cw00.choi@samsung.com; devicetree@vger.kernel.org; > > linux-usb@vger.kernel.org > > Subject: Re: [PATCH v5 01/14] dt-bindings: connector: add properties for typec > > > > Am Donnerstag, den 03.05.2018, 08:24 +0800 schrieb Li Jun: > > > +Optional properties for usb-c-connector: > > > +- power-role: should be one of "source", "sink" or "dual"(DRP) if > > > +typec > > > +  connector has power support. > > > +- try-power-role: preferred power role if "dual"(DRP) can support > > > +Try.SNK > > > +  or Try.SRC, should be "sink" for Try.SNK or "source" for Try.SRC. > > > +- data-role: should be one of "host", "device", "dual"(DRD) if typec > > > +  connector supports USB data. > > > > Hi, > > > > is this really correct? > > > > Can one implement a device that can operate as either DFP or UFP, but not > > implements the dynamic role switch that a DRP must support? > > You mean a port with DRD on data but not DRP on power? > > The data-role is newly added as the data role is not coupled with power No, I meant data role. As far as I can tell for a DRP you need to implement the detection logic described in chapter 4 of the spec. I can see no reason why you couldn't build a port that can be switched between the data roles but not implement that logic. Regards Oliver