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: [v3,4/4] doc: usb: ci-hdrc-usb2: Add pinctrl properties for HSIC pin groups From: Rob Herring Message-Id: <20181207233234.GA1419@bogus> Date: Fri, 7 Dec 2018 17:32:34 -0600 To: PETER CHEN Cc: Schrempf Frieder , Fabio Estevam , USB list , dl-linux-imx , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Schrempf Frieder List-ID: T24gV2VkLCBEZWMgMDUsIDIwMTggYXQgMDc6NTc6MzdBTSArMDAwMCwgUEVURVIgQ0hFTiB3cm90 ZToKPiAgCj4gPiBPbiAwNC4xMi4xOCAyMTowMSwgRmFiaW8gRXN0ZXZhbSB3cm90ZToKPiA+ID4g SGkgRnJpZWRlciwKPiA+ID4KPiA+ID4gT24gVHVlLCBEZWMgNCwgMjAxOCBhdCAxMjozMSBQTSBT Y2hyZW1wZiBGcmllZGVyCj4gPiA+IDxmcmllZGVyLnNjaHJlbXBmQGtvbnRyb24uZGU+IHdyb3Rl Ogo+ID4gPgo+ID4gPj4gVGhlcmUgYXJlIG1hbnkgb3RoZXIgb3B0aW9uYWwgcHJvcGVydGllcyBm b3IgdGhpcyBkcml2ZXIgYW5kIGEgbG90IG9mCj4gPiA+PiB0aGVtIGFyZSBub3QgaW4gdGhlIGdp dmVuIGV4YW1wbGUuIE1heWJlIHdlIHNob3VsZCBqdXN0IGtlZXAgdGhlCj4gPiA+PiBwaW5jdHJs cyBmb3IgSFNJQy1tb2RlIG91dCBvZiB0aGUgZXhhbXBsZSwgdG9vPwo+ID4gPgo+ID4gPiBJIGFt IGp1c3QgdHJ5aW5nIHRvIG1ha2UgbGlmZSBlYXNpZXIgZm9yIHRob3NlIHdobyB3YW50IHRvIHVz ZSBIU0lDCj4gPiA+IHN1cHBvcnQgd2l0aCBjaGlwaWRlYS4KPiA+ID4KPiA+ID4gQ2FuIHdlIGp1 c3QgYWRkIGEgcmVhbCBkdHMgc25pcHBldCBleGFtcGxlIG9mIHlvdXIgYm9hcmQgaW50byB0aGUK PiA+ID4gYmluZGluZyBkb2N1bWVudD8KPiA+IAo+ID4gU3VyZSwgaGVyZSBpcyB3aGF0IEkgaGF2 ZSBpbiBteSBkdHM6Cj4gPiAKPiA+ICZ1c2JoMiB7Cj4gPiAJcGluY3RybC1uYW1lcyA9ICJpZGxl IiwgImFjdGl2ZSI7Cj4gPiAJcGluY3RybC0wID0gPCZwaW5jdHJsX3VzYmgyX2lkbGU+Owo+ID4g CXBpbmN0cmwtMSA9IDwmcGluY3RybF91c2JoMl9hY3RpdmU+Owo+ID4gCXN0YXR1cyA9ICJva2F5 IjsKPiA+IAkjYWRkcmVzcy1jZWxscyA9IDwxPjsKPiA+IAkjc2l6ZS1jZWxscyA9IDwwPjsKPiA+ IAo+ID4gCXVzYm5ldDogc21zY0AxIHsKPiA+IAkJY29tcGF0aWJsZSA9ICJ1c2I0MjQsOTczMCI7 Cj4gPiAJCXJlZyA9IDwxPjsKPiA+IAl9Owo+ID4gfTsKPiA+IAo+ID4gQFBldGVyOiBDYW4geW91 IGFkZCB0aGlzIGFzIGEgc2Vjb25kIGV4YW1wbGUgdG8gdGhlIGJpbmRpbmcgZG9jdW1lbnRhdGlv bj8KPiA+IAo+IAo+IFNvLCB0aGVyZSBhcmUgdHdvIGV4YW1wbGVzIGF0IGJpbmRpbmctZG9jLCBv bmUgZm9yIG5vcm1hbCwgb25lIGZvciBIU0lDPyBGYWJpbywgZG8geW91Cj4gbWVhbiB0aGF0PyBJ ZiBEVCBtYWludGFpbmVyIGFncmVlcyBpdCB0b28sIEkgd2lsbCBhZGQgaXQuCgpPa2F5Lgo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 7 Dec 2018 17:32:34 -0600 From: Rob Herring Subject: Re: [PATCH v3 4/4] doc: usb: ci-hdrc-usb2: Add pinctrl properties for HSIC pin groups Message-ID: <20181207233234.GA1419@bogus> References: <20181127092824.22756-1-peter.chen@nxp.com> <20181127092824.22756-5-peter.chen@nxp.com> <0215e99c-c024-42f6-9dad-4bab70bae193@kontron.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: To: PETER CHEN Cc: Schrempf Frieder , Fabio Estevam , USB list , dl-linux-imx , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Schrempf Frieder List-ID: On Wed, Dec 05, 2018 at 07:57:37AM +0000, PETER CHEN wrote: > > > On 04.12.18 21:01, Fabio Estevam wrote: > > > Hi Frieder, > > > > > > On Tue, Dec 4, 2018 at 12:31 PM Schrempf Frieder > > > wrote: > > > > > >> There are many other optional properties for this driver and a lot of > > >> them are not in the given example. Maybe we should just keep the > > >> pinctrls for HSIC-mode out of the example, too? > > > > > > I am just trying to make life easier for those who want to use HSIC > > > support with chipidea. > > > > > > Can we just add a real dts snippet example of your board into the > > > binding document? > > > > Sure, here is what I have in my dts: > > > > &usbh2 { > > pinctrl-names = "idle", "active"; > > pinctrl-0 = <&pinctrl_usbh2_idle>; > > pinctrl-1 = <&pinctrl_usbh2_active>; > > status = "okay"; > > #address-cells = <1>; > > #size-cells = <0>; > > > > usbnet: smsc@1 { > > compatible = "usb424,9730"; > > reg = <1>; > > }; > > }; > > > > @Peter: Can you add this as a second example to the binding documentation? > > > > So, there are two examples at binding-doc, one for normal, one for HSIC? Fabio, do you > mean that? If DT maintainer agrees it too, I will add it. Okay.