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: [v2,1/4] usb: dwc3: Add property snps,refclk-period-ns From: Felipe Balbi Message-Id: <87va3ou21i.fsf@linux.intel.com> Date: Thu, 20 Dec 2018 08:48:09 +0200 To: Thinh Nguyen , Rob Herring Thinh Nguyen Cc: Linux USB List , "devicetree@vger.kernel.org" , Mark Rutland , John Youn List-ID: SGksCgpUaGluaCBOZ3V5ZW4gPHRoaW5oLm5ndXllbkBzeW5vcHN5cy5jb20+IHdyaXRlczoKPj4+ IE9uIDEyLzE4LzIwMTggODozOSBBTSwgUm9iIEhlcnJpbmcgd3JvdGU6Cj4+Pj4gT24gRnJpLCBE ZWMgMDcsIDIwMTggYXQgMDY6Mjc6MzBQTSAtMDgwMCwgVGhpbmggTmd1eWVuIHdyb3RlOgo+Pj4+ PiBUaGlzIHBhdGNoIGludHJvZHVjZXMgcHJvcGVydHkgInNucHMscmVmY2xrLXBlcmlvZC1ucyIg dG8gaW5mb3JtIHRoZQo+Pj4+PiBjb250cm9sbGVyIG9mIHRoZSByZWZlcmVuY2UgY2xvY2sgcGVy aW9kLiBJZiB0aGUgcmVmZXJlbmNlIGNsb2NrIHBlcmlvZAo+Pj4+PiBpcyBkaWZmZXJlbnQgZnJv bSB0aGUgZGVmYXVsdCBDb3JlIENvbnN1bHRhbnQgc2V0dGluZywgdGhlbiB0aGlzCj4+Pj4+IHBy b3BlcnR5IGNhbiBiZSBzZXQgdG8gdGhlIHJlZmVyZW5jZSBjbG9jayBwZXJpb2QuCj4+Pj4+Cj4+ Pj4+IFRoaXMgcHJvcGVydHkgZG9lcyBub3QgY29udHJvbCB0aGUgcmVmZXJlbmNlIGNsb2NrIHJh dGUuIFRoZSBjb250cm9sbGVyCj4+Pj4+IHVzZXMgdGhpcyB2YWx1ZSB0byBwZXJmb3JtIGludGVy bmFsIHRpbWluZyBjYWxjdWxhdGlvbnMgdGhhdCBhcmUgYmFzZWQKPj4+Pj4gb24gdGhlIHJlZmVy ZW5jZSBjbG9jay4KPj4+Pj4KPj4+Pj4gU2lnbmVkLW9mZi1ieTogVGhpbmggTmd1eWVuIDx0aGlu aG5Ac3lub3BzeXMuY29tPgo+Pj4+PiAtLS0KPj4+Pj4gQ2hhbmdlcyBpbiB2MjoKPj4+Pj4gLSBT cGxpdCBmcm9tICJ1c2I6IGR3YzM6IEFkZCByZWZlcmVuY2UgY2xvY2sgcHJvcGVydGllcyIKPj4+ Pj4gLSBSZXZpc2UgY29tbWl0IG1lc3NhZ2UgYW5kIHByb3BlcnR5IGRlc2NyaXB0aW9uCj4+Pj4+ Cj4+Pj4+ICBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvdXNiL2R3YzMudHh0IHwg MiArKwo+Pj4+PiAgMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKQo+Pj4+Pgo+Pj4+PiBk aWZmIC0tZ2l0IGEvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3VzYi9kd2MzLnR4 dCBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy91c2IvZHdjMy50eHQKPj4+Pj4g aW5kZXggOGU1MjY1ZTlmNjU4Li5iN2U2N2VkZmY5YjIgMTAwNjQ0Cj4+Pj4+IC0tLSBhL0RvY3Vt ZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy91c2IvZHdjMy50eHQKPj4+Pj4gKysrIGIvRG9j dW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3VzYi9kd2MzLnR4dAo+Pj4+PiBAQCAtOTks NiArOTksOCBAQCBPcHRpb25hbCBwcm9wZXJ0aWVzOgo+Pj4+PiAgICAgICAgICAgICAgICAgICAg ICB0aGlzIGFuZCB0eC10aHItbnVtLXBrdC1wcmQgdG8gYSB2YWxpZCwgbm9uLXplcm8gdmFsdWUK Pj4+Pj4gICAgICAgICAgICAgICAgICAgICAgMS0xNiAoRFdDX3VzYjMxIHByb2dyYW1taW5nIGd1 aWRlIHNlY3Rpb24gMS4yLjMpIHRvCj4+Pj4+ICAgICAgICAgICAgICAgICAgICAgIGVuYWJsZSBw ZXJpb2RpYyBFU1MgVFggdGhyZXNob2xkLgo+Pj4+PiArIC0gc25wcyxyZWZjbGstcGVyaW9kLW5z OiBpZiBzZXQsIHRoaXMgdmFsdWUgaW5mb3JtcyB0aGUgY29udHJvbGxlciBvZiB0aGUKPj4+Pj4g KyAgICAgICAgICAgICAgICAgICAgcmVmZXJlbmNlIGNsb2NrIHBlcmlvZCBpbiBuYW5vc2Vjb25k cy4KPj4+PiBTaG91bGRuJ3QgeW91IGJlIGFibGUgdG8gcmV0cmlldmUgdGhlIHJlZmNsayBmcmVx dWVuY3kgYW5kIHRoZW4KPj4+PiBjYWxjdWxhdGUgdGhlIHBlcmlvZD8KPj4+IFRoZSB0aGluZyBp cyB3ZSBjYW5ub3QgZGV0ZXJtaW5lIHRoZSByZWZfY2xrIGZyZXF1ZW5jeSBmb3Igc29tZSBkZXZp Y2VzCj4+PiB0aGF0IGRvbid0IHNwZWNpZnkgdGhlaXIgY2xvY2tzLiBTbyBJIHRoaW5rIHdlIHNo b3VsZCBoYXZlIGFuIG9wdGlvbiB0bwo+Pj4gaW5mb3JtIHRoZSBjb250cm9sbGVyIG9mIHRoZSBy ZWZfY2xrIHBlcmlvZCBmb3IgdGhvc2UgZGV2aWNlcy4KPj4gU3BlY2lmeWluZyB0aGUgY2xvY2sg c2hvdWxkIGJlIG1hbmRhdG9yeSAoaWYgeW91IHdhbnQvbmVlZCB0aGlzCj4+IGZlYXR1cmUpLiBJ dCBqdXN0IHJlcXVpcmVzIGEgZml4ZWQtY2xvY2sgbm9kZSBhdCBhIG1pbmltdW0uCj4KPiBEZXBl bmRpbmcgb24gdGhlIGRlc2lnbiBvZiB0aGUgY29udHJvbGxlciwgdGhlIHJlZl9jbGsgZnJlcXVl bmN5IGlzIG5vdAo+IHNvbWV0aGluZyB0aGF0IHRoZSBPUyBjYW4gcmVhZC9jb250cm9sLiBTbyB3 ZSBjYW5ub3QgbWFrZSBpdCBtYW5kYXRvcnkKPiBmb3IgZXZlcnkgZGV2aWNlIHRvIGhhdmUgYSBj bG9jayBub2RlLgoKV2UgY2FuIG1ha2UgaXQgbWFuZGF0b3J5IHRvIGV2ZXJ5b25lIHdobyB3YW50 cyB0byB1c2UgdGhlIGZlYXR1cmUuIEl0J3MKbm8gZGlmZmVyZW50IHRoYW4gbWFraW5nIHNucHMs cmVmY2xrLXBlcmlvZC1ucyBtYW5kYXRvcnkgdG8gZXZlcnlvbmUgd2hvCndhbnRzIHRvIHVzZSB0 aGUgZmVhdHVyZSB5b3UncmUgaW50cm9kdWNpbmcuCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Felipe Balbi Subject: Re: [PATCH v2 1/4] usb: dwc3: Add property snps,refclk-period-ns In-Reply-To: <30102591E157244384E984126FC3CB4F639AB64F@us01wembx1.internal.synopsys.com> References: <83adc98adc1760a0fad87d81d171e1dac783e7e5.1544235317.git.thinhn@synopsys.com> <20181218163858.GA12336@bogus> <30102591E157244384E984126FC3CB4F639AB1BC@us01wembx1.internal.synopsys.com> <30102591E157244384E984126FC3CB4F639AB64F@us01wembx1.internal.synopsys.com> Date: Thu, 20 Dec 2018 08:48:09 +0200 Message-ID: <87va3ou21i.fsf@linux.intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" To: Thinh Nguyen , Rob Herring Thinh Nguyen Cc: Linux USB List , "devicetree@vger.kernel.org" , Mark Rutland , John Youn List-ID: --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Hi, Thinh Nguyen writes: >>> On 12/18/2018 8:39 AM, Rob Herring wrote: >>>> On Fri, Dec 07, 2018 at 06:27:30PM -0800, Thinh Nguyen wrote: >>>>> This patch introduces property "snps,refclk-period-ns" to inform the >>>>> controller of the reference clock period. If the reference clock peri= od >>>>> is different from the default Core Consultant setting, then this >>>>> property can be set to the reference clock period. >>>>> >>>>> This property does not control the reference clock rate. The controll= er >>>>> uses this value to perform internal timing calculations that are based >>>>> on the reference clock. >>>>> >>>>> Signed-off-by: Thinh Nguyen >>>>> --- >>>>> Changes in v2: >>>>> - Split from "usb: dwc3: Add reference clock properties" >>>>> - Revise commit message and property description >>>>> >>>>> Documentation/devicetree/bindings/usb/dwc3.txt | 2 ++ >>>>> 1 file changed, 2 insertions(+) >>>>> >>>>> diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt b/Documen= tation/devicetree/bindings/usb/dwc3.txt >>>>> index 8e5265e9f658..b7e67edff9b2 100644 >>>>> --- a/Documentation/devicetree/bindings/usb/dwc3.txt >>>>> +++ b/Documentation/devicetree/bindings/usb/dwc3.txt >>>>> @@ -99,6 +99,8 @@ Optional properties: >>>>> this and tx-thr-num-pkt-prd to a valid, non-zero= value >>>>> 1-16 (DWC_usb31 programming guide section 1.2.3)= to >>>>> enable periodic ESS TX threshold. >>>>> + - snps,refclk-period-ns: if set, this value informs the controller = of the >>>>> + reference clock period in nanoseconds. >>>> Shouldn't you be able to retrieve the refclk frequency and then >>>> calculate the period? >>> The thing is we cannot determine the ref_clk frequency for some devices >>> that don't specify their clocks. So I think we should have an option to >>> inform the controller of the ref_clk period for those devices. >> Specifying the clock should be mandatory (if you want/need this >> feature). It just requires a fixed-clock node at a minimum. > > Depending on the design of the controller, the ref_clk frequency is not > something that the OS can read/control. So we cannot make it mandatory > for every device to have a clock node. We can make it mandatory to everyone who wants to use the feature. It's no different than making snps,refclk-period-ns mandatory to everyone who wants to use the feature you're introducing. =2D-=20 balbi --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEElLzh7wn96CXwjh2IzL64meEamQYFAlwbOykACgkQzL64meEa mQZ5WBAA3Sr5UbFbyuR9SK4zso2Tae4ixAEaMxx2ne9N79rEF33J79zB4fIgku9U QZ0daLGfWcLGMvVIuj1lPZGSJsyGtmymQyp6bgRT9laT1VfNoswrDmPdxexgAg4V VlcLBB2mdvYC/lzMfn2ntwV3qsF7PrSkRuBJyEQnKL58pIug3afzMjSUnuhoFbrL SHjfmNddlZZa7bCZx8VhZi7gcE/wLVpcCzYTSH/FICusuzKLp2ZfrY0FtEgX6Uvf 5ljxUol23JH89BFgnN6FlcI2ynvjU63ZrVkXGDUkDGXxf2wj6caC+nXT1tsHpybY agq/c/f8FX/NYJdIYU7FWhsrJr7ATGP1l96l2tiLGzzzVN0NeBAM6jPFxGhU7NL/ KwqomKdYN6dS3k/L9hz0UkXzgyhnAL9iXyDx4BCuOkM0LpOFXL9kNNMk2Sm9x1sz LPiXggdsLDLAaLFr+7UbcFjnd1yDenzwDOzfVCi2CfYRYv1cIHgHJVDhDVnE8nhU 8uqgd85yN5V05PISyV+6urVlI+uHPqLp6lBM6dgaKmGMjTE5Iaa/zVcuNQiYTwdj K/X+XTr9fZuiCrMnhEd/RaqfVhbC/1L6ETjH+CkRgTtmyrT7Lre2DA0eD4Nnx2j8 WZPpSFWT1rR40Qm2/v6Tv3OUs5RlLbRdI7miiFCMwBdPXNz8uNw= =wvqT -----END PGP SIGNATURE----- --=-=-=--