From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chanwoo Choi Subject: Re: [v5 PATCH 1/5] extcon: Add Type-C and DP support Date: Thu, 14 Jul 2016 09:49:58 +0900 Message-ID: <5786E1B6.2000805@samsung.com> References: <1468336188-565-1-git-send-email-zyw@rock-chips.com> <1468336188-565-2-git-send-email-zyw@rock-chips.com> <57859532.1010002@samsung.com> <57859BDA.7080309@rock-chips.com> <5785A1F3.1020703@samsung.com> <5785AD61.3070307@rock-chips.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-reply-to: <5785AD61.3070307-TNX95d0MmH7DzftRWevZcw@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+glpar-linux-rockchip=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Chris Zhong , dianders-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org, tfiga-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org, heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org, yzq-TNX95d0MmH7DzftRWevZcw@public.gmane.org, groeck-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org, myungjoo.ham-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org, wulf-TNX95d0MmH7DzftRWevZcw@public.gmane.org, marcheu-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rockchip.vger.kernel.org SGkgQ2hyaXMsCgpPbiAyMDE264WEIDA37JuUIDEz7J28IDExOjU0LCBDaHJpcyBaaG9uZyB3cm90 ZToKPiBIaSBDaGFud29vIENob2kKPiAKPiBPbiAwNy8xMy8yMDE2IDEwOjA1IEFNLCBDaGFud29v IENob2kgd3JvdGU6Cj4+IEhpIENocmlzLAo+Pgo+PiBPbiAyMDE264WEIDA37JuUIDEz7J28IDEw OjM5LCBDaHJpcyBaaG9uZyB3cm90ZToKPj4+IEhpIENoYW53b28gQ2hvaQo+Pj4KPj4+Cj4+PiBP biAwNy8xMy8yMDE2IDA5OjExIEFNLCBDaGFud29vIENob2kgd3JvdGU6Cj4+Pj4gSGkgQ2hyaXMs Cj4+Pj4KPj4+PiBJJ20gbm93IGRldmVsb3BpbmcgdGhlIGV4dGNvbiBwcm9wZXJ0eSBvbiBleHRj b24tdGVzdCBicmFuY2guCj4+Pj4gQnV0LCBpdCBoYXMgbm90IGJlZW4gY29tcGxldGVkLgo+Pj4+ Cj4+Pj4gT24gbmV4dCB2ZXJzaW9uLCBJJ2xsIHJlbW92ZSB0aGUgbm90aWZpY2F0aW9uIGFib3V0 IGV4dGNvbiBwcm9wZXJ0eQo+Pj4+IGFuZCBvbmx5IHN1cHBvcnQgdGhlIGZvbGxvd2luZyB0d28g ZnVuY3Rpb25zLgo+Pj4+IC0gZXh0Y29uX3NldF9jYWJsZV9wcm9wZXJ0eSgpCj4+Pj4gLSBleHRj b25fZ2V0X2NhYmxlX3Byb3BlcnR5KCkKPj4+Pgo+Pj4+IEJlY2F1c2UgdGhlIG51bWJlciBvZiBw cm9wZXJ0aWVzIHdvdWxkIGJlIHJpc2VuIGFuZCB0aGUgYWxsIHByb3BlcnRpZXMKPj4+PiBkZXBl bmQgb24gdGhlIHNwZWNpZmljIGV4dGVybmFsIGNvbm5lY3RvcihlLmcuLCBFWFRDT05fUFJPUF9V U0JfVkJVUwo+Pj4+IGRlcGVuZCBvbiB0aGUgRVhUQ09OX1RZUEVfVVNCIHR5cGUpLiBXaGVuIHRo ZSBzcGVjaWZpYyBleHRlcm5hbCBjb25uZWN0b3IKPj4+PiBpcyBkZXRhY2hlZCwgZXh0Y29uIGZy YW1ld29yayBzaG91bGQgbWFrZSB0aGUgcHJvcGVydHkgc3RhdGUgYXMgZGVmYXVsdCBzdGF0ZS4K Pj4+IFllcywgSSB0aGluayBnZXR0aW5nIHRoZSBub3RpZmljYXRpb24gZnJvbSBjYWJsZSBzdGF0 ZSBpcyBlbm91Z2gsIGFjdHVhbGx5IEkgYW0gdXNpbmcgaXQgbGlrZSB5b3Ugc2FpZC4KPj4gT0su Cj4+Cj4+Pj4gSXQgbWF5IHNlbmQgdGhlIHRvbyBtYW55IG5vdGlmaWNhdGlvbiBmb3IgZXh0Y29u IHByb3BlcnR5Lgo+Pj4+IEZvciBleGFtcGxlLCBBc3N1bWUgdGhhdCBFWFRDT05fVFlQRV9VU0Ig aGFzIHRoZSBvdmVyIDIwIHByb3BlcnRpZXMsCj4+Pj4gd2hlbiBFWFRDT05fVVNCIG9yIEVYVENP Tl9VU0JfSE9TVCBpcyBkZXRhY2hlZCwgZXh0Y29uIHNob3VsZCBzZW5kCj4+Pj4gdGhlIG5vdGlm aWNhdGlvbiBmb3IgdGhlIG92ZXIgMjAgcHJvcGVydGllcyBhbmQgb25lIG1vcmUgbm90aWZpY2Fp dG9uCj4+Pj4gZm9yIHN0YXRlIG9mIGV4dGVybmFsIGNvbm5lY3Rvci4KPj4+Pgo+Pj4+IFNvLCBJ J2xsIHNlbmQgdGhlIFJGQyBwYXRjaHNldCB3aXRob3V0IHRoZSBub3RpZmljYXRpb24gb2YgcHJv ZXJ0eS4KPj4+Pgo+Pj4+IExhc3RseSwKPj4+PiBJIGhhdmUgYSBjb21tZW50IG9uIGJlbG93Lgo+ Pj4+Cj4+Pj4gVGhhbmtzLAo+Pj4+IENoYW53b28gQ2hvaQo+Pj4+Cj4+Pj4gT24gMjAxNuuFhCAw N+yblCAxM+ydvCAwMDowOSwgQ2hyaXMgWmhvbmcgd3JvdGU6Cj4+Pj4+IEFkZCBFWFRDT05fRElT UF9EUCBmb3IgdGhlIERpc3BsYXkgZXh0ZXJuYWwgY29ubmVjdG9yLiBGb3IgVHlwZS1DCj4+Pj4+ IGNvbm5lY3RvciB0aGUgRGlzcGxheVBvcnQgY2FuIHdvcmsgYXMgYW4gQWx0ZXJuYXRlIE1vZGUo VkVTQSBEaXNwbGF5UG9ydAo+Pj4+PiBBbHQgTW9kZSBvbiBVU0IgVHlwZS1DIFN0YW5kYXJkKS4g VGhlIFR5cGUtQyBzdXBwb3J0IGJvdGggbm9ybWFsIGFuZAo+Pj4+PiBmbGlwcGVkIG9yaWVudGF0 aW9uLCBzbyBhZGQgYSBwcm9wZXJ0eSB0byBleHRjb24uCj4+Pj4+Cj4+Pj4+IFNpZ25lLW9mZi1i eTogQ2hyaXMgWmhvbmcgPHp5d0Byb2NrLWNoaXBzLmNvbT4KPj4+Pj4KPj4+Pj4gU2lnbmVkLW9m Zi1ieTogQ2hyaXMgWmhvbmcgPHp5d0Byb2NrLWNoaXBzLmNvbT4KPj4+Pj4gLS0tCj4+Pj4+Cj4+ Pj4+IENoYW5nZXMgaW4gdjU6Cj4+Pj4+IC0gc3VwcG9ydCBnZXQgcHJvcGVydHkKPj4+Pj4KPj4+ Pj4gQ2hhbmdlcyBpbiB2NDogTm9uZQo+Pj4+PiBDaGFuZ2VzIGluIHYzOiBOb25lCj4+Pj4+IENo YW5nZXMgaW4gdjI6IE5vbmUKPj4+Pj4gQ2hhbmdlcyBpbiB2MTogTm9uZQo+Pj4+Pgo+Pj4+PiAg ICBkcml2ZXJzL2V4dGNvbi9leHRjb24uYyB8IDI4ICsrKysrKysrKysrKysrKysrKysrKysrKysr KysKPj4+Pj4gICAgaW5jbHVkZS9saW51eC9leHRjb24uaCAgfCAxMyArKysrKysrKysrKysrCj4+ Pj4+ICAgIDIgZmlsZXMgY2hhbmdlZCwgNDEgaW5zZXJ0aW9ucygrKQo+Pj4+Pgo+Pj4+PiBkaWZm IC0tZ2l0IGEvZHJpdmVycy9leHRjb24vZXh0Y29uLmMgYi9kcml2ZXJzL2V4dGNvbi9leHRjb24u Ywo+Pj4+PiBpbmRleCBhMTExN2RiLi4yNTkxYjI4IDEwMDY0NAo+Pj4+PiAtLS0gYS9kcml2ZXJz L2V4dGNvbi9leHRjb24uYwo+Pj4+PiArKysgYi9kcml2ZXJzL2V4dGNvbi9leHRjb24uYwo+Pj4+ PiBAQCAtMTU3LDYgKzE1NywxMSBAQCBzdHJ1Y3QgX19leHRjb25faW5mbyB7Cj4+Pj4+ICAgICAg ICAgICAgLmlkID0gRVhUQ09OX0RJU1BfVkdBLAo+Pj4+PiAgICAgICAgICAgIC5uYW1lID0gIlZH QSIsCj4+Pj4+ICAgICAgICB9LAo+Pj4+PiArICAgIFtFWFRDT05fRElTUF9EUF0gPSB7Cj4+Pj4+ ICsgICAgICAgIC50eXBlID0gRVhUQ09OX1RZUEVfRElTUCwKPj4+Pj4gKyAgICAgICAgLmlkID0g RVhUQ09OX0RJU1BfRFAsCj4+Pj4+ICsgICAgICAgIC5uYW1lID0gIkRQIiwKPj4+Pj4gKyAgICB9 LAo+Pj4+PiAgICAgICAgICAvKiBNaXNjZWxsYW5lb3VzIGV4dGVybmFsIGNvbm5lY3RvciAqLwo+ Pj4+PiAgICAgICAgW0VYVENPTl9ET0NLXSA9IHsKPj4+Pj4gQEAgLTI3MCw2ICsyNzUsNyBAQCBz dGF0aWMgYm9vbCBpc19leHRjb25fcHJvcGVydHlfc3VwcG9ydGVkKHVuc2lnbmVkIGludCBpZCwK Pj4+Pj4gICAgICAgICAgICBzd2l0Y2ggKHByb3ApIHsKPj4+Pj4gICAgICAgICAgICBjYXNlIEVY VENPTl9QUk9QX1VTQl9JRDoKPj4+Pj4gICAgICAgICAgICBjYXNlIEVYVENPTl9QUk9QX1VTQl9W QlVTOgo+Pj4+PiArICAgICAgICBjYXNlIEVYVENPTl9QUk9QX1RZUEVDX1BPTEFSSVRZOgo+Pj4+ PiAgICAgICAgICAgICAgICByZXR1cm4gdHJ1ZTsKPj4+Pj4gICAgICAgICAgICBkZWZhdWx0Ogo+ Pj4+PiAgICAgICAgICAgICAgICBicmVhazsKPj4+Pj4gQEAgLTI4Niw2ICsyOTIsOCBAQCBzdGF0 aWMgYm9vbCBpc19leHRjb25fcHJvcGVydHlfc3VwcG9ydGVkKHVuc2lnbmVkIGludCBpZCwKPj4+ Pj4gICAgICAgICAgICB9Cj4+Pj4+ICAgICAgICBjYXNlIEVYVENPTl9UWVBFX0RJU1A6Cj4+Pj4+ ICAgICAgICAgICAgc3dpdGNoIChwcm9wKSB7Cj4+Pj4+ICsgICAgICAgIGNhc2UgRVhUQ09OX1BS T1BfVFlQRUNfUE9MQVJJVFk6Cj4+Pj4gU2hvdWxkIEVYVENPTl9QUk9QX1RZUEVDX1BPTEFSSVRZ IHByb3BlcnR5IGFkZCB0byBib3RoIEVYVENPTl9UWVBFX1VTQiBhbmQgRVhUQ09OX1RZUF9ESVNQ Pwo+Pj4+IEVYVENPTl9QUk9QX1RZUEVDX1BPTEFSSVRZIGlzIHRoZSBwcm9wZXJ0eSBvZiBVU0Ig Qy10eXBlPwo+Pj4gaXQgaXMgZm9yIFVTQiBUeXBlLUMsIEJ1dCBhdCBEaXNwbGF5IFBvcnQgYWx0 IG1vZGUsIGJvdGggRVhUQ09OX1VTQiBhbmQgRVhUQ09OX1VTQl9IT1NUIG1heSBiZSBkZXRhY2hl ZC4gRG9lcyBpdCBzdXBwb3J0IHNldCB0aGUgcHJvcGVydHkgdG8gYSBkZXRhY2hlZCBjYWJsZSwg aWYgc28sIEkgdGhpbmsgbW92ZSB0aGlzIGNhc2UgdG8gRVhUQ09OX1VTQiBpcyBmaW5lLgo+PiBP bmUgZXh0ZXJuYWwgY29ubmVjdG9yIGNhbiBzZXQgdGhlIHN0YXRlIG9mIG9uZSBtb3JlIGV4dGVy bmFsIGNvbm5lY3Rvcgo+PiBpZiB0aGUgb25lIGNvbm5lY3RvciBzdXBwb3J0IHRoZSB2YXJpb3Vz IGZ1bmN0aW9ucy4KPj4gRm9yIGV4YW1wbGUsIEVYVENPTl9VU0IgYW5kIEVYVENPTl9DSEdfVVNC X1NEUAo+PiBUaGUgZXhpc3RpbmcgZXh0Y29uIGRyaXZlclsxXShlLmcuLCBtYXgxNDU3Ny9tYXg3 NzY5MyBldGMuKSBzZXQgdGhlIHN0YXRlIG9mIGJvdGggRVhUQ09OX1VTQiBhbmQgRVhUQ09OX0NI R19VU0JfU0RQIGNvbm5lY3RvciBhdCB0aGUgc2FtZSB0aW1lCj4+IHdoZW4gdXNiIGNhYmxlIGlz IGF0dGFjaGVkLiBCZWNhdXNlIGluIHRoaXMgY2FzZSwgdGhlIHVzYiBjb25uZWN0b3IgdXNlcyBh cyBib3RoIHBvd2VyIHN1cHBseShFWFRDT05fQ0hHX1VTQl9TRFApIGFuZCBkYXRhIHRyYW5zZmVy KEVYVENPTl9VU0IpLgo+PiBbMV0gaHR0cHM6Ly9naXQua2VybmVsLm9yZy9jZ2l0L2xpbnV4L2tl cm5lbC9naXQvY2hhbndvby9leHRjb24uZ2l0L2NvbW1pdC8/aD1leHRjb24tbmV4dCZpZD04YjQ1 YjZhMDc0MTY3ODkwMjgxMGQ3YmU5NWU2MzVjMjEwZmJiMTk4Cj4+Cj4+IFNvLCBEUCBBbHQgbW9k ZSB1c2VzIHRoZSBVU0IgVHlwZS1DLiBTbywgV2hlbiBVU0IgQy10eXBlIGNvbm5lY3RvciBpcyBh dHRhY2hlZCBmb3IgRFAgQWx0IG1vZGUsCj4+IE1heWJlLCB5b3UgY2FuIHNldCB0aGUgZm9sbG93 aW5nIHR3byBzdGF0ZSBvZiBjb25uZWN0b3IgYW5kIG9uZSBwcm9wZXJ0eToKPj4gLSBleHRjb25f c2V0X2NhYmxlX3N0YXRlKGVkZXYsIFtFWFRDT05fVVNCIG9yIEVYVENPTl9VU0JfSE9TVF0sIDEp Owo+PiAtIGV4dGNvbl9zZXRfY2FibGVfc3RhdGUoZWRldiwgRVhUQ09OX0RJU1BfRFAsIDEpOwo+ PiAtIGV4dGNvbl9zZXRfY2FibGVfc3RhdGUoZWRldiwgW0VYVENPTl9VU0Igb3IgRVhUQ09OX1VT Ql9IT1NUXSwgRVhUQ09OX1BST1BfVFlQRUNfUE9MQVJJVFksIDAgb3IgMSk7Cj4+Cj4+IFRoYW5r cywKPj4gQ2hhbndvbyBDaG9pCj4gCj4gVGhlcmUgYXJlIDQgbW9kZXMgZm9yIFR5cGUtQyBEUCBh bHQgbW9kZToKPiAxKSBVU0IgaG9zdCBvbmx5ICA6Cj4gCj4gZXh0Y29uX3NldF9jYWJsZV9zdGF0 ZShlZGV2LCBFWFRDT05fVVNCX0hPU1QsIDEpOwo+IGV4dGNvbl9zZXRfY2FibGVfc3RhdGUoZWRl diwgRVhUQ09OX1VTQiwgMCk7Cj4gZXh0Y29uX3NldF9jYWJsZV9zdGF0ZShlZGV2LCBFWFRDT05f RElTUF9EUCwgMCk7Cj4gCj4gMikgVVNCIGRldmljZSBvbmx5Cj4gCj4gZXh0Y29uX3NldF9jYWJs ZV9zdGF0ZShlZGV2LCBFWFRDT05fVVNCX0hPU1QsIDApOwo+IGV4dGNvbl9zZXRfY2FibGVfc3Rh dGUoZWRldiwgRVhUQ09OX1VTQiwgMSk7Cj4gZXh0Y29uX3NldF9jYWJsZV9zdGF0ZShlZGV2LCBF WFRDT05fRElTUF9EUCwgMCk7Cj4gCj4gMykgRFAgb25seQo+IAo+IGV4dGNvbl9zZXRfY2FibGVf c3RhdGUoZWRldiwgRVhUQ09OX1VTQl9IT1NULCAwKTsKPiBleHRjb25fc2V0X2NhYmxlX3N0YXRl KGVkZXYsIEVYVENPTl9VU0IsIDApOwo+IGV4dGNvbl9zZXRfY2FibGVfc3RhdGUoZWRldiwgRVhU Q09OX0RJU1BfRFAsIDEpOwo+IAo+IDQpIFVTQiArIERQCj4gCj4gZXh0Y29uX3NldF9jYWJsZV9z dGF0ZShlZGV2LCBFWFRDT05fVVNCX0hPU1QsIDEpOwo+IGV4dGNvbl9zZXRfY2FibGVfc3RhdGUo ZWRldiwgRVhUQ09OX1VTQiwgMCk7Cj4gZXh0Y29uX3NldF9jYWJsZV9zdGF0ZShlZGV2LCBFWFRD T05fRElTUF9EUCwgMSk7Cj4gCj4gCj4gZm9yIDNyZCBtb2RlOiBEUCBvbmx5LCB0aGVyZSBpcyBv bmx5IEVYVENPTl9ESVNQX0RQIGlzIGF0dGFjaGVkLCB0aGUgRVhUQ09OX1VTQl9IT1NUCj4gYW5k IEVYVENPTl9VU0IgYXJlIGRldGFjaGVkLCBDYW4gSSBzZXQgdGhlIHByb3BlcnR5IGludG8gdGhl c2UgMiBkZXRhY2hlZCBjYWJsZT8KPiBvciBqdXN0IGNhbGwgZXh0Y29uX3NldF9jYWJsZV9zdGF0 ZShlZGV2LCBFWFRDT05fRElTUF9EUCwgRVhUQ09OX1BST1BfVFlQRUNfUE9MQVJJVFksIDAgb3Ig MSk7CgpJJ20gdGhpbmtpbmcgdG8gc29sdmUgdGhpcyBpc3N1ZS4gSW4gcmVzdWx0LCB3ZSBjYW4g YWRkIG9uZSBtb3JlIHR5cGUgdG8gc3BlY2lmaWMgY29ubmVjdG9yLgpGb3IgRVhUQ09OX0RJU1Bf RFAsIHdlIGNhbiBhZGQgdGhlIHR3byB0eXBlIGFzIGZvbGxvd2luZy4gQW5kIEVYVENPTl9QUk9Q X1RZUEVDX1BPTEFSSVRZIHByb3BlcnR5CnNob3VsZCBiZSBhZGRlZCB0byBFWFRDT05fVFlQRV9V U0IuCgorCVtFWFRDT05fRElTUF9EUF0gPSB7CisJCS50eXBlID0gRVhUQ09OX1RZUEVfRElTUCB8 IEVYVENPTl9UWVBFX1VTQiwKKwkJLmlkID0gRVhUQ09OX0RJU1BfRFAsCisJCS5uYW1lID0gIkRQ IiwKKwl9LAoKU28sIGFzIHlvdSBtZW50aW9uZWQsIEVYVENPTl9ESVNQX0RQIGNhbiBzZXQgdGhl IEVYVENPTl9QUk9QX1RZUEVDX1BPTEFSSVRZIHByb3BlcnR5IGFzIGZvbGxvd2luZzoKLSBleHRj b25fc2V0X2NhYmxlX3N0YXRlKGVkZXYsIEVYVENPTl9ESVNQX0RQLCBFWFRDT05fUFJPUF9UWVBF Q19QT0xBUklUWSwgMCBvciAxKTsKCkknbGwgYWdhaW4gZGV2ZWxvcGluZyB0aGUgZXh0Y29uIHBy b3BlcnR5LgoKVGhhbmtzLApDaGFud29vIENob2kKCgpfX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fXwpMaW51eC1yb2NrY2hpcCBtYWlsaW5nIGxpc3QKTGludXgt cm9ja2NoaXBAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2xpbnV4LXJvY2tjaGlwCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751665AbcGNAuL (ORCPT ); Wed, 13 Jul 2016 20:50:11 -0400 Received: from mailout1.samsung.com ([203.254.224.24]:54256 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751526AbcGNAuB (ORCPT ); Wed, 13 Jul 2016 20:50:01 -0400 MIME-version: 1.0 Content-type: text/plain; charset=utf-8 X-AuditID: cbfee68d-f79876d000001436-55-5786e1b6c51b Content-transfer-encoding: 8BIT Message-id: <5786E1B6.2000805@samsung.com> Date: Thu, 14 Jul 2016 09:49:58 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: Chris Zhong , dianders@chromium.org, tfiga@chromium.org, heiko@sntech.de, yzq@rock-chips.com, groeck@chromium.org, myungjoo.ham@samsung.com, wulf@rock-chips.com, marcheu@chromium.org Cc: linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [v5 PATCH 1/5] extcon: Add Type-C and DP support References: <1468336188-565-1-git-send-email-zyw@rock-chips.com> <1468336188-565-2-git-send-email-zyw@rock-chips.com> <57859532.1010002@samsung.com> <57859BDA.7080309@rock-chips.com> <5785A1F3.1020703@samsung.com> <5785AD61.3070307@rock-chips.com> In-reply-to: <5785AD61.3070307@rock-chips.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrOIsWRmVeSWpSXmKPExsWyRsSkUHfbw7Zwg2V7rS3OLjvIZnHq1TJm i/+PXrNaXN41h83i04P/zBbzDv1mtLjduILN4nPrPzaLVWuPM1vMXbeEzWL+gx3MDtwesxsu snhsXlLv8XfWfhaPvi2rGD22X5vH7PF5k1wAWxSXTUpqTmZZapG+XQJXxq19C9gLXhlXzH9+ iqWB8btmFyMnh4SAicTpc3sZIWwxiQv31rN1MXJxCAmsYJTY+nEhO0xR/4q9TBCJWYwSi9/d YwNJ8AoISvyYfI+li5GDg1lAXuLIpWwIU11iypRciPIHjBKnFt1khCjXklixsxWslUVAVWLR 7vXMIDYbUHz/ixtsIL2iAhES3ScqQXpFBC4xSix92MIOMdNFYspHbhBTWMBW4ui7MojxF5kk rp+7ATaeU0BPYtmSmYwgCQmBr+wS/9u7mCF2CUh8m3wI7EwJAVmJTQeYId6SlDi44gbLBEax WUiemYXwzCyEZxYwMq9iFE0tSC4oTkovMtQrTswtLs1L10vOz93ECIzP0/+e9e5gvH3A+hCj AAejEg9vRG9buBBrYllxZe4hRlOgGyYyS4km5wOTQF5JvKGxmZGFqYmpsZG5pZmSOK+i1M9g IYH0xJLU7NTUgtSi+KLSnNTiQ4xMHJxSDYymb6uKlTVrrrJev8B3/KMMb/U8n9J+0TWNltUJ P1tmJa0QC7u1MnqT7YrbS6YtOlJm/Gwhr8Nqu63qrz+x3pkmPtdc8e+h3ol/Q946nJz0wzzR zf1KoozaoX3Tz80VmrFW4Wpq6GaZ74e/q01fnRO40vXc8iVfFbi9rNrXJCzhyhcvOpxz84CD EktxRqKhFnNRcSIAwRuOkMoCAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmphleLIzCtJLcpLzFFi42I5/e+xoO62h23hBq+vmFicXXaQzeLUq2XM Fv8fvWa1uLxrDpvFpwf/mS3mHfrNaHG7cQWbxefWf2wWq9YeZ7aYu24Jm8X8BzuYHbg9Zjdc ZPHYvKTe4++s/SwefVtWMXpsvzaP2ePzJrkAtqgGRpuM1MSU1CKF1Lzk/JTMvHRbJe/geOd4 UzMDQ11DSwtzJYW8xNxUWyUXnwBdt8wcoPuUFMoSc0qBQgGJxcVK+naYJoSGuOlawDRG6PqG BMH1GBmggYQ1jBm39i1gL3hlXDH/+SmWBsbvml2MnBwSAiYS/Sv2MkHYYhIX7q1n62Lk4hAS mMUosfjdPTaQBK+AoMSPyfdYuhg5OJgF5CWOXMqGMNUlpkzJhSh/wChxatFNRohyLYkVO1vB WlkEVCUW7V7PDGKzAcX3v7jBBtIrKhAh0X2iEqRXROASo8TShy3sEDNdJKZ85AYxhQVsJY6+ K4MYf5FJ4vq5G2DjOQX0JJYtmck4gRHoRoTjZiEcNwvhuAWMzKsYJVILkguKk9JzjfJSy/WK E3OLS/PS9ZLzczcxglPAM+kdjId3uR9iFOBgVOLhfZDXFi7EmlhWXJl7iFGCg1lJhPfvfaAQ b0piZVVqUX58UWlOavEhRlOg7yYyS4km5wPTU15JvKGxiZmRpZG5oYWRsbmSOO/j/+vChATS E0tSs1NTC1KLYPqYODilGhhnndHPuL0jTL7+v/pE7T6Bw5IFH9P9lm5iXsWgeadwz8/jd2Z7 T1nA07WzeeuM5tz9Ch53QhZJrlXzZzHYwlvekaAeayVz1e/j44OG7g7fFhSduLRHML0t5V/Y RMZ80WXOGWc+z+qZ++/KVqtHD523hXyT/hhZ8aNWfNLtx96Jza8f/FFM5pRQYinOSDTUYi4q TgQACRfQkBcDAAA= DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Chris, On 2016년 07월 13일 11:54, Chris Zhong wrote: > Hi Chanwoo Choi > > On 07/13/2016 10:05 AM, Chanwoo Choi wrote: >> Hi Chris, >> >> On 2016년 07월 13일 10:39, Chris Zhong wrote: >>> Hi Chanwoo Choi >>> >>> >>> On 07/13/2016 09:11 AM, Chanwoo Choi wrote: >>>> Hi Chris, >>>> >>>> I'm now developing the extcon property on extcon-test branch. >>>> But, it has not been completed. >>>> >>>> On next version, I'll remove the notification about extcon property >>>> and only support the following two functions. >>>> - extcon_set_cable_property() >>>> - extcon_get_cable_property() >>>> >>>> Because the number of properties would be risen and the all properties >>>> depend on the specific external connector(e.g., EXTCON_PROP_USB_VBUS >>>> depend on the EXTCON_TYPE_USB type). When the specific external connector >>>> is detached, extcon framework should make the property state as default state. >>> Yes, I think getting the notification from cable state is enough, actually I am using it like you said. >> OK. >> >>>> It may send the too many notification for extcon property. >>>> For example, Assume that EXTCON_TYPE_USB has the over 20 properties, >>>> when EXTCON_USB or EXTCON_USB_HOST is detached, extcon should send >>>> the notification for the over 20 properties and one more notificaiton >>>> for state of external connector. >>>> >>>> So, I'll send the RFC patchset without the notification of proerty. >>>> >>>> Lastly, >>>> I have a comment on below. >>>> >>>> Thanks, >>>> Chanwoo Choi >>>> >>>> On 2016년 07월 13일 00:09, Chris Zhong wrote: >>>>> Add EXTCON_DISP_DP for the Display external connector. For Type-C >>>>> connector the DisplayPort can work as an Alternate Mode(VESA DisplayPort >>>>> Alt Mode on USB Type-C Standard). The Type-C support both normal and >>>>> flipped orientation, so add a property to extcon. >>>>> >>>>> Signe-off-by: Chris Zhong >>>>> >>>>> Signed-off-by: Chris Zhong >>>>> --- >>>>> >>>>> Changes in v5: >>>>> - support get property >>>>> >>>>> Changes in v4: None >>>>> Changes in v3: None >>>>> Changes in v2: None >>>>> Changes in v1: None >>>>> >>>>> drivers/extcon/extcon.c | 28 ++++++++++++++++++++++++++++ >>>>> include/linux/extcon.h | 13 +++++++++++++ >>>>> 2 files changed, 41 insertions(+) >>>>> >>>>> diff --git a/drivers/extcon/extcon.c b/drivers/extcon/extcon.c >>>>> index a1117db..2591b28 100644 >>>>> --- a/drivers/extcon/extcon.c >>>>> +++ b/drivers/extcon/extcon.c >>>>> @@ -157,6 +157,11 @@ struct __extcon_info { >>>>> .id = EXTCON_DISP_VGA, >>>>> .name = "VGA", >>>>> }, >>>>> + [EXTCON_DISP_DP] = { >>>>> + .type = EXTCON_TYPE_DISP, >>>>> + .id = EXTCON_DISP_DP, >>>>> + .name = "DP", >>>>> + }, >>>>> /* Miscellaneous external connector */ >>>>> [EXTCON_DOCK] = { >>>>> @@ -270,6 +275,7 @@ static bool is_extcon_property_supported(unsigned int id, >>>>> switch (prop) { >>>>> case EXTCON_PROP_USB_ID: >>>>> case EXTCON_PROP_USB_VBUS: >>>>> + case EXTCON_PROP_TYPEC_POLARITY: >>>>> return true; >>>>> default: >>>>> break; >>>>> @@ -286,6 +292,8 @@ static bool is_extcon_property_supported(unsigned int id, >>>>> } >>>>> case EXTCON_TYPE_DISP: >>>>> switch (prop) { >>>>> + case EXTCON_PROP_TYPEC_POLARITY: >>>> Should EXTCON_PROP_TYPEC_POLARITY property add to both EXTCON_TYPE_USB and EXTCON_TYP_DISP? >>>> EXTCON_PROP_TYPEC_POLARITY is the property of USB C-type? >>> it is for USB Type-C, But at Display Port alt mode, both EXTCON_USB and EXTCON_USB_HOST may be detached. Does it support set the property to a detached cable, if so, I think move this case to EXTCON_USB is fine. >> One external connector can set the state of one more external connector >> if the one connector support the various functions. >> For example, EXTCON_USB and EXTCON_CHG_USB_SDP >> The existing extcon driver[1](e.g., max14577/max77693 etc.) set the state of both EXTCON_USB and EXTCON_CHG_USB_SDP connector at the same time >> when usb cable is attached. Because in this case, the usb connector uses as both power supply(EXTCON_CHG_USB_SDP) and data transfer(EXTCON_USB). >> [1] https://git.kernel.org/cgit/linux/kernel/git/chanwoo/extcon.git/commit/?h=extcon-next&id=8b45b6a0741678902810d7be95e635c210fbb198 >> >> So, DP Alt mode uses the USB Type-C. So, When USB C-type connector is attached for DP Alt mode, >> Maybe, you can set the following two state of connector and one property: >> - extcon_set_cable_state(edev, [EXTCON_USB or EXTCON_USB_HOST], 1); >> - extcon_set_cable_state(edev, EXTCON_DISP_DP, 1); >> - extcon_set_cable_state(edev, [EXTCON_USB or EXTCON_USB_HOST], EXTCON_PROP_TYPEC_POLARITY, 0 or 1); >> >> Thanks, >> Chanwoo Choi > > There are 4 modes for Type-C DP alt mode: > 1) USB host only : > > extcon_set_cable_state(edev, EXTCON_USB_HOST, 1); > extcon_set_cable_state(edev, EXTCON_USB, 0); > extcon_set_cable_state(edev, EXTCON_DISP_DP, 0); > > 2) USB device only > > extcon_set_cable_state(edev, EXTCON_USB_HOST, 0); > extcon_set_cable_state(edev, EXTCON_USB, 1); > extcon_set_cable_state(edev, EXTCON_DISP_DP, 0); > > 3) DP only > > extcon_set_cable_state(edev, EXTCON_USB_HOST, 0); > extcon_set_cable_state(edev, EXTCON_USB, 0); > extcon_set_cable_state(edev, EXTCON_DISP_DP, 1); > > 4) USB + DP > > extcon_set_cable_state(edev, EXTCON_USB_HOST, 1); > extcon_set_cable_state(edev, EXTCON_USB, 0); > extcon_set_cable_state(edev, EXTCON_DISP_DP, 1); > > > for 3rd mode: DP only, there is only EXTCON_DISP_DP is attached, the EXTCON_USB_HOST > and EXTCON_USB are detached, Can I set the property into these 2 detached cable? > or just call extcon_set_cable_state(edev, EXTCON_DISP_DP, EXTCON_PROP_TYPEC_POLARITY, 0 or 1); I'm thinking to solve this issue. In result, we can add one more type to specific connector. For EXTCON_DISP_DP, we can add the two type as following. And EXTCON_PROP_TYPEC_POLARITY property should be added to EXTCON_TYPE_USB. + [EXTCON_DISP_DP] = { + .type = EXTCON_TYPE_DISP | EXTCON_TYPE_USB, + .id = EXTCON_DISP_DP, + .name = "DP", + }, So, as you mentioned, EXTCON_DISP_DP can set the EXTCON_PROP_TYPEC_POLARITY property as following: - extcon_set_cable_state(edev, EXTCON_DISP_DP, EXTCON_PROP_TYPEC_POLARITY, 0 or 1); I'll again developing the extcon property. Thanks, Chanwoo Choi