All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sean Anderson <sean.anderson@seco.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-usb@vger.kernel.org
Cc: Felipe Balbi <balbi@kernel.org>,
	Thinh Nguyen <Thinh.Nguyen@synopsys.com>,
	Balaji Prakash J <bjagadee@codeaurora.org>,
	linux-kernel@vger.kernel.org,
	Robert Hancock <robert.hancock@calian.com>,
	Baruch Siach <baruch@tkos.co.il>,
	Sean Anderson <sean.anderson@seco.com>,
	Rob Herring <robh+dt@kernel.org>,
	devicetree@vger.kernel.org
Subject: [PATCH v2 1/7] dt-bindings: usb: dwc3: Deprecate snps,ref-clock-period-ns
Date: Tue, 18 Jan 2022 19:24:31 -0500	[thread overview]
Message-ID: <20220119002438.106079-2-sean.anderson@seco.com> (raw)
In-Reply-To: <20220119002438.106079-1-sean.anderson@seco.com>

This property is redundant because we can determine the correct value for
REFCLKPER based on the "ref" clock. Deprecate it, and encourage users to
provide a clock instead. This also restricts the minimum and maximum to the
values documented in the register reference [1].

[1] https://www.xilinx.com/html_docs/registers/ug1087/usb3_xhci___guctl.html

Signed-off-by: Sean Anderson <sean.anderson@seco.com>
---

(no changes since v1)

 Documentation/devicetree/bindings/usb/snps,dwc3.yaml | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml
index d29ffcd27472..4f2b0913ad9f 100644
--- a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml
+++ b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml
@@ -263,8 +263,11 @@ properties:
       Value for REFCLKPER field of GUCTL register for reference clock period in
       nanoseconds, when the hardware set default does not match the actual
       clock.
-    minimum: 1
-    maximum: 0x3ff
+
+      This binding is deprecated. Instead, provide an appropriate reference clock.
+    minimum: 8
+    maximum: 62
+    deprecated: true
 
   snps,rx-thr-num-pkt-prd:
     description:
-- 
2.25.1


  reply	other threads:[~2022-01-19  0:25 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-19  0:24 [PATCH v2 0/7] usb: dwc3: Calculate REFCLKPER et. al. from reference clock Sean Anderson
2022-01-19  0:24 ` Sean Anderson
2022-01-19  0:24 ` Sean Anderson [this message]
2022-01-19  0:24 ` [PATCH v2 2/7] usb: dwc3: Get clocks individually Sean Anderson
2022-01-20 16:52   ` Robert Hancock
2022-01-19  0:24 ` [PATCH v2 3/7] usb: dwc3: Calculate REFCLKPER based on reference clock Sean Anderson
2022-01-20 16:53   ` Robert Hancock
2022-01-19  0:24 ` [PATCH v2 4/7] usb: dwc3: Program GFLADJ Sean Anderson
2022-01-20 16:55   ` Robert Hancock
2022-01-24 22:46   ` Thinh Nguyen
2022-01-24 23:06     ` Sean Anderson
2022-01-25  2:11       ` Thinh Nguyen
2022-01-25  6:17         ` Felipe Balbi
2022-01-25 20:02           ` Thinh Nguyen
2022-01-25 16:22         ` Sean Anderson
2022-01-25 19:36           ` Thinh Nguyen
2022-01-26 10:56             ` Felipe Balbi
2022-01-26 10:53           ` Felipe Balbi
2022-01-27 16:45             ` Sean Anderson
2022-01-19  0:24 ` [PATCH v2 5/7] usb: dwc3: Add snps,ref-clock-frequency-hz property for ACPI Sean Anderson
2022-01-24 22:44   ` Thinh Nguyen
2022-01-24 23:07     ` Sean Anderson
2022-01-24 23:14       ` Sean Anderson
2022-01-25  2:21         ` Thinh Nguyen
2022-01-19  0:24 ` [PATCH v2 6/7] arm64: dts: zynqmp: Move USB clocks to dwc3 node Sean Anderson
2022-01-19  0:24   ` Sean Anderson
2022-01-20 16:56   ` Robert Hancock
2022-01-20 16:56     ` Robert Hancock
2022-01-19  0:24 ` [PATCH v2 7/7] arm64: dts: ipq6018: Use reference clock to set dwc3 period Sean Anderson
2022-01-19 18:14 ` [PATCH v2 0/7] usb: dwc3: Calculate REFCLKPER et. al. from reference clock Baruch Siach
2022-01-19 18:14   ` Baruch Siach
2022-01-19 18:23   ` Sean Anderson
2022-01-19 18:23     ` Sean Anderson
2022-01-20  5:23   ` Kathiravan T
2022-01-20 10:29     ` Baruch Siach
2022-01-20 10:29       ` Baruch Siach
2022-01-24 15:11       ` Kathiravan Thirumoorthy
2022-01-24 15:11         ` Kathiravan Thirumoorthy
2022-01-24 23:01         ` Thinh Nguyen
2022-01-24 23:01           ` Thinh Nguyen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220119002438.106079-2-sean.anderson@seco.com \
    --to=sean.anderson@seco.com \
    --cc=Thinh.Nguyen@synopsys.com \
    --cc=balbi@kernel.org \
    --cc=baruch@tkos.co.il \
    --cc=bjagadee@codeaurora.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=robert.hancock@calian.com \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.