From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 14DD5C0044C for ; Mon, 29 Oct 2018 12:25:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id CBAEB20870 for ; Mon, 29 Oct 2018 12:25:25 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CBAEB20870 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=huawei.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729290AbeJ2VNv (ORCPT ); Mon, 29 Oct 2018 17:13:51 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:14580 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728772AbeJ2VNu (ORCPT ); Mon, 29 Oct 2018 17:13:50 -0400 Received: from DGGEMS409-HUB.china.huawei.com (unknown [172.30.72.60]) by Forcepoint Email with ESMTP id 9A64C5E47DD94; Mon, 29 Oct 2018 20:25:18 +0800 (CST) Received: from [127.0.0.1] (10.142.63.192) by DGGEMS409-HUB.china.huawei.com (10.3.19.209) with Microsoft SMTP Server id 14.3.408.0; Mon, 29 Oct 2018 20:25:11 +0800 CC: , , , Rob Herring , Mark Rutland , John Stultz Subject: Re: [PATCH 03/10] dt-bindings: misc: Add bindings for HiSilicon usb hub and data role switch functionality on HiKey960 To: Sergei Shtylyov , , , References: <20181027095820.40056-1-chenyu56@huawei.com> <20181027095820.40056-4-chenyu56@huawei.com> <4872a689-bfdd-a8c1-f2cf-a2bba9e91fa9@cogentembedded.com> From: Chen Yu Message-ID: <644437e0-fa67-e3c5-ed57-48ae9f10b046@huawei.com> Date: Mon, 29 Oct 2018 20:25:09 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 MIME-Version: 1.0 In-Reply-To: <4872a689-bfdd-a8c1-f2cf-a2bba9e91fa9@cogentembedded.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Content-Language: en-US X-Originating-IP: [10.142.63.192] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2018/10/28 2:06, Sergei Shtylyov wrote: > On 27.10.2018 12:58, Yu Chen wrote: > >> This patch adds binding documentation to support usb hub and usb >> data role switch of Hisilicon HiKey&HiKey960 Board. >> >> Cc: Rob Herring >> Cc: Mark Rutland >> Cc: John Stultz >> Cc: Binghui Wang >> Signed-off-by: Yu Chen >> --- >>  .../bindings/misc/hisilicon-hikey-usb.txt          | 24 ++++++++++++++++++++++ >>  1 file changed, 24 insertions(+) >>  create mode 100644 Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt >> >> diff --git a/Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt b/Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt >> new file mode 100644 >> index 000000000000..f94e990362c9 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt >> @@ -0,0 +1,24 @@ >> +Support usb hub and usb data role switch of Hisilicon HiKey&HiKey960 Board. >> + >> +----------------------------- >> + >> +Required properties: >> +- compatible: "hisilicon,gpio_hubv1","hisilicon,hikey960_usb" >> +- typc_vbus_int_gpio,typec-gpios: gpio to control the vbus of typeC port >> +- typc_vbus_enable_val: gpio value that enable the vbus of typeC port > >    The part of the prop name preceding the comma is supposed to be vendor prefix. > Also hyphens are preferred to underscores. I will fix the name of properties. Thanks! > >> +- otg_gpio: gpio to switch DP&DM between the hub and typeC port >> +- hub_vdd33_en_gpio: gpio to enable the power of hub >> +- pinctrl-0: pinctrl config > [...] > > MBR, Sergei > > > > . > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Yu Subject: Re: [PATCH 03/10] dt-bindings: misc: Add bindings for HiSilicon usb hub and data role switch functionality on HiKey960 Date: Mon, 29 Oct 2018 20:25:09 +0800 Message-ID: <644437e0-fa67-e3c5-ed57-48ae9f10b046@huawei.com> References: <20181027095820.40056-1-chenyu56@huawei.com> <20181027095820.40056-4-chenyu56@huawei.com> <4872a689-bfdd-a8c1-f2cf-a2bba9e91fa9@cogentembedded.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <4872a689-bfdd-a8c1-f2cf-a2bba9e91fa9@cogentembedded.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Sergei Shtylyov , linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Cc: wangbinghui@hisilicon.com, suzhuangluan@hisilicon.com, kongfei@hisilicon.com, Rob Herring , Mark Rutland , John Stultz List-Id: devicetree@vger.kernel.org On 2018/10/28 2:06, Sergei Shtylyov wrote: > On 27.10.2018 12:58, Yu Chen wrote: > >> This patch adds binding documentation to support usb hub and usb >> data role switch of Hisilicon HiKey&HiKey960 Board. >> >> Cc: Rob Herring >> Cc: Mark Rutland >> Cc: John Stultz >> Cc: Binghui Wang >> Signed-off-by: Yu Chen >> --- >>  .../bindings/misc/hisilicon-hikey-usb.txt          | 24 ++++++++++++++++++++++ >>  1 file changed, 24 insertions(+) >>  create mode 100644 Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt >> >> diff --git a/Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt b/Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt >> new file mode 100644 >> index 000000000000..f94e990362c9 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/misc/hisilicon-hikey-usb.txt >> @@ -0,0 +1,24 @@ >> +Support usb hub and usb data role switch of Hisilicon HiKey&HiKey960 Board. >> + >> +----------------------------- >> + >> +Required properties: >> +- compatible: "hisilicon,gpio_hubv1","hisilicon,hikey960_usb" >> +- typc_vbus_int_gpio,typec-gpios: gpio to control the vbus of typeC port >> +- typc_vbus_enable_val: gpio value that enable the vbus of typeC port > >    The part of the prop name preceding the comma is supposed to be vendor prefix. > Also hyphens are preferred to underscores. I will fix the name of properties. Thanks! > >> +- otg_gpio: gpio to switch DP&DM between the hub and typeC port >> +- hub_vdd33_en_gpio: gpio to enable the power of hub >> +- pinctrl-0: pinctrl config > [...] > > MBR, Sergei > > > > . > 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: [03/10] dt-bindings: misc: Add bindings for HiSilicon usb hub and data role switch functionality on HiKey960 From: Yu Chen Message-Id: <644437e0-fa67-e3c5-ed57-48ae9f10b046@huawei.com> Date: Mon, 29 Oct 2018 20:25:09 +0800 To: Sergei Shtylyov , linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Cc: wangbinghui@hisilicon.com, suzhuangluan@hisilicon.com, kongfei@hisilicon.com, Rob Herring , Mark Rutland , John Stultz List-ID: T24gMjAxOC8xMC8yOCAyOjA2LCBTZXJnZWkgU2h0eWx5b3Ygd3JvdGU6Cj4gT24gMjcuMTAuMjAx OCAxMjo1OCwgWXUgQ2hlbiB3cm90ZToKPgo+PiBUaGlzIHBhdGNoIGFkZHMgYmluZGluZyBkb2N1 bWVudGF0aW9uIHRvIHN1cHBvcnQgdXNiIGh1YiBhbmQgdXNiCj4+IGRhdGEgcm9sZSBzd2l0Y2gg b2YgSGlzaWxpY29uIEhpS2V5JkhpS2V5OTYwIEJvYXJkLgo+Pgo+PiBDYzogUm9iIEhlcnJpbmcg PHJvYmgrZHRAa2VybmVsLm9yZz4KPj4gQ2M6IE1hcmsgUnV0bGFuZCA8bWFyay5ydXRsYW5kQGFy bS5jb20+Cj4+IENjOiBKb2huIFN0dWx0eiA8am9obi5zdHVsdHpAbGluYXJvLm9yZz4KPj4gQ2M6 IEJpbmdodWkgV2FuZyA8d2FuZ2JpbmdodWlAaGlzaWxpY29uLmNvbT4KPj4gU2lnbmVkLW9mZi1i eTogWXUgQ2hlbiA8Y2hlbnl1NTZAaHVhd2VpLmNvbT4KPj4gLS0tCj4+IMKgLi4uL2JpbmRpbmdz L21pc2MvaGlzaWxpY29uLWhpa2V5LXVzYi50eHTCoMKgwqDCoMKgwqDCoMKgwqAgfCAyNCArKysr KysrKysrKysrKysrKysrKysrCj4+IMKgMSBmaWxlIGNoYW5nZWQsIDI0IGluc2VydGlvbnMoKykK Pj4gwqBjcmVhdGUgbW9kZSAxMDA2NDQgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdz L21pc2MvaGlzaWxpY29uLWhpa2V5LXVzYi50eHQKPj4KPj4gZGlmZiAtLWdpdCBhL0RvY3VtZW50 YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9taXNjL2hpc2lsaWNvbi1oaWtleS11c2IudHh0IGIv RG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL21pc2MvaGlzaWxpY29uLWhpa2V5LXVz Yi50eHQKPj4gbmV3IGZpbGUgbW9kZSAxMDA2NDQKPj4gaW5kZXggMDAwMDAwMDAwMDAwLi5mOTRl OTkwMzYyYzkKPj4gLS0tIC9kZXYvbnVsbAo+PiArKysgYi9Eb2N1bWVudGF0aW9uL2RldmljZXRy ZWUvYmluZGluZ3MvbWlzYy9oaXNpbGljb24taGlrZXktdXNiLnR4dAo+PiBAQCAtMCwwICsxLDI0 IEBACj4+ICtTdXBwb3J0IHVzYiBodWIgYW5kIHVzYiBkYXRhIHJvbGUgc3dpdGNoIG9mIEhpc2ls aWNvbiBIaUtleSZIaUtleTk2MCBCb2FyZC4KPj4gKwo+PiArLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0KPj4gKwo+PiArUmVxdWlyZWQgcHJvcGVydGllczoKPj4gKy0gY29tcGF0aWJsZTog Imhpc2lsaWNvbixncGlvX2h1YnYxIiwiaGlzaWxpY29uLGhpa2V5OTYwX3VzYiIKPj4gKy0gdHlw Y192YnVzX2ludF9ncGlvLHR5cGVjLWdwaW9zOiBncGlvIHRvIGNvbnRyb2wgdGhlIHZidXMgb2Yg dHlwZUMgcG9ydAo+PiArLSB0eXBjX3ZidXNfZW5hYmxlX3ZhbDogZ3BpbyB2YWx1ZSB0aGF0IGVu YWJsZSB0aGUgdmJ1cyBvZiB0eXBlQyBwb3J0Cj4KPiDCoMKgIFRoZSBwYXJ0IG9mIHRoZSBwcm9w IG5hbWUgcHJlY2VkaW5nIHRoZSBjb21tYSBpcyBzdXBwb3NlZCB0byBiZSB2ZW5kb3IgcHJlZml4 Lgo+IEFsc28gaHlwaGVucyBhcmUgcHJlZmVycmVkIHRvIHVuZGVyc2NvcmVzLgoKSSB3aWxsIGZp eCB0aGUgbmFtZSBvZiBwcm9wZXJ0aWVzLiBUaGFua3MhCgoKPgo+PiArLSBvdGdfZ3BpbzogZ3Bp byB0byBzd2l0Y2ggRFAmRE0gYmV0d2VlbiB0aGUgaHViIGFuZCB0eXBlQyBwb3J0Cj4+ICstIGh1 Yl92ZGQzM19lbl9ncGlvOiBncGlvIHRvIGVuYWJsZSB0aGUgcG93ZXIgb2YgaHViCj4+ICstIHBp bmN0cmwtMDogcGluY3RybCBjb25maWcKPiBbLi4uXQo+Cj4gTUJSLCBTZXJnZWkKPgo+Cj4KPiAu Cj4K