From: Rob Herring <robh@kernel.org>
To: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
Cc: Kishon Vijay Abraham I <kishon@ti.com>,
Mark Rutland <mark.rutland@arm.com>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
Masami Hiramatsu <masami.hiramatsu@linaro.org>,
Jassi Brar <jaswinder.singh@linaro.org>
Subject: Re: [PATCH 1/4] dt-bindings: phy: add DT bindings for UniPhier USB3 PHY driver
Date: Mon, 16 Jul 2018 14:50:49 -0600 [thread overview]
Message-ID: <20180716205049.GA3663@rob-hp-laptop> (raw)
In-Reply-To: <1530261541-23104-2-git-send-email-hayashi.kunihiko@socionext.com>
On Fri, Jun 29, 2018 at 05:38:58PM +0900, Kunihiko Hayashi wrote:
> Add DT bindings for PHY interface built into USB3 controller
> implemented in UniPhier SoCs.
>
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> ---
> .../devicetree/bindings/phy/uniphier-usb3-phy.txt | 118 +++++++++++++++++++++
> 1 file changed, 118 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/phy/uniphier-usb3-phy.txt
>
> diff --git a/Documentation/devicetree/bindings/phy/uniphier-usb3-phy.txt b/Documentation/devicetree/bindings/phy/uniphier-usb3-phy.txt
> new file mode 100644
> index 0000000..3df4a486
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/phy/uniphier-usb3-phy.txt
> @@ -0,0 +1,118 @@
> +Socionext UniPhier USB3 PHY
> +
> +This describes the devicetree bindings for PHY interfaces built into
> +USB3 controller implemented on Socionext UniPhier SoCs.
> +The controller includes High-Speed PHY and Super-Speed PHY.
> +
> +USB3 High-Speed (HS) PHY
> +------------------------
> +
> +Required properties:
> +- compatible: Should contain one of the following:
> + "socionext,uniphier-pro4-usb3-hsphy" - for Pro4 SoC
> + "socionext,uniphier-pxs2-usb3-hsphy" - for PXs2 SoC
> + "socionext,uniphier-ld20-usb3-hsphy" - for LD20 SoC
> + "socionext,uniphier-pxs3-usb3-hsphy" - for PXs3 SoC
> +- reg: Specifies offset and length of the register set for the device.
> +- #phy-cells: Should be 0.
> +- clocks: A list of phandles to the clock gate for USB3 glue layer.
> + According to the clock-names, appropriate clocks are required.
> +- clock-names: Should contain the following:
> + "gio", "link" - for Pro4 SoC
> + "link", "phy", "phy-ext" - for PXs3 SoC, "phy-ext" is optional.
> + "link", "phy" - for others
Can't you make 'link' always first.
> +- resets: A list of phandles to the reset control for USB3 glue layer.
> + According to the reset-names, appropriate resets are required.
> +- reset-names: Should contain the following:
> + "gio", "link" - for Pro4 SoC
> + "link", "phy" - for others
> +
> +Optional properties:
> +- phy-supply: A phandle to the regulator for USB VBUS.
The phy actually needs Vbus or you just want to control Vbus from the
phy driver?
> +- nvmem-cells: Phandles to nvmem cell that contains the trimming data.
> + Available only for HS-PHY implemented on LD20 and PXs3, and
> + if unspecified, default value is used.
> +- nvmem-cell-names: Should be the following names, which correspond to
> + each nvmem-cells.
> + All of the 3 parameters associated with the following names are
> + required for each port, if any one is omitted, the trimming data
> + of the port will not be set at all.
> + "rterm", "sel_t", "hs_i" - Each cell name for phy parameters
> +
> +Refer to phy/phy-bindings.txt for the generic PHY binding properties.
> +
> +Example:
> +
> + usb-glue@65b00000 {
> + compatible = "socionext,uniphier-ld20-dwc3-glue",
> + "simple-mfd";
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges = <0 0x65b00000 0x400>;
> +
> + usb_hsphy0: hs-phy@200 {
> + compatible = "socionext,uniphier-ld20-usb3-hsphy";
> + reg = <0x200 0x10>;
> + #phy-cells = <0>;
> + clock-names = "link", "phy";
> + clocks = <&sys_clk 14>, <&sys_clk 16>;
> + reset-names = "link", "phy";
> + resets = <&sys_rst 14>, <&sys_rst 16>;
> + phy-supply = <&usb_vbus0>;
> + nvmem-cell-names = "rterm", "sel_t", "hs_i";
> + nvmem-cells = <&usb_rterm0>, <&usb_sel_t0>,
> + <&usb_hs_i0>;
> + };
> + };
> +
> +
> +USB3 Super-Speed (SS) PHY
> +-------------------------
Nothing seems to be shared here. Make this 2 docs.
> +
> +Required properties:
> +- compatible: Should contain one of the following:
> + "socionext,uniphier-pro4-usb3-ssphy" - for Pro4 SoC
> + "socionext,uniphier-pxs2-usb3-ssphy" - for PXs2 SoC
> + "socionext,uniphier-ld20-usb3-ssphy" - for LD20 SoC
> + "socionext,uniphier-pxs3-usb3-ssphy" - for PXs3 SoC
> +- reg: Specifies offset and length of the register set for the device.
> +- #phy-cells: Should be 0.
> +- clocks: A list of phandles to the clock gate for USB3 glue layer.
> + According to the clock-names, appropriate clocks are required.
> +- clock-names:
> + "gio", "link" - for Pro4 SoC
> + "link", "phy", "phy-ext" - for PXs3 SoC, "phy-ext" is optional.
> + "link", "phy" - for others
Can't you make 'link' always first.
> +- resets: A list of phandles to the reset control for USB3 glue layer.
> + According to the reset-names, appropriate resets are required.
> +- reset-names:
> + "gio", "link" - for Pro4 SoC
> + "link", "phy" - for others
And here.
> +
> +Optional properties:
> +- phy-supply: A phandle to the regulator for USB VBUS.
> +
> +Refer to phy/phy-bindings.txt for the generic PHY binding properties.
> +
> +Example:
> +
> + usb-glue@65b00000 {
> + compatible = "socionext,uniphier-ld20-dwc3-glue",
> + "simple-mfd";
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges = <0 0x65b00000 0x400>;
> +
> + usb_ssphy0: ss-phy@300 {
> + compatible = "socionext,uniphier-ld20-usb3-ssphy";
> + reg = <0x300 0x10>;
> + #phy-cells = <0>;
> + clock-names = "link", "phy";
> + clocks = <&sys_clk 14>, <&sys_clk 16>;
> + reset-names = "link", "phy";
> + resets = <&sys_rst 14>, <&sys_rst 16>;
> + phy-supply = <&usb_vbus0>;
> + };
> +
> + other nodes ...
> + };
> --
> 2.7.4
>
next prev parent reply other threads:[~2018-07-16 20:50 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-29 8:38 [PATCH 0/4] phy: socionext: add new UniPhier USB PHY driver support Kunihiko Hayashi
2018-06-29 8:38 ` [PATCH 1/4] dt-bindings: phy: add DT bindings for UniPhier USB3 PHY driver Kunihiko Hayashi
2018-07-16 20:50 ` Rob Herring [this message]
2018-07-17 10:55 ` Kunihiko Hayashi
2018-07-17 14:16 ` Rob Herring
2018-07-24 11:10 ` Kunihiko Hayashi
2018-06-29 8:38 ` [PATCH 2/4] phy: socionext: add USB3 PHY driver for UniPhier SoC Kunihiko Hayashi
2018-07-09 5:19 ` Kishon Vijay Abraham I
2018-07-09 11:23 ` Kunihiko Hayashi
2018-07-11 12:05 ` Kunihiko Hayashi
2018-07-13 7:15 ` Kishon Vijay Abraham I
2018-07-17 11:27 ` Kunihiko Hayashi
2018-07-24 4:01 ` Kishon Vijay Abraham I
2018-07-24 11:02 ` Kunihiko Hayashi
2018-06-29 8:39 ` [PATCH 3/4] dt-bindings: phy: add DT bindings for UniPhier USB2 PHY driver Kunihiko Hayashi
2018-06-29 8:39 ` [PATCH 4/4] phy: socionext: add USB2 PHY driver for UniPhier SoC Kunihiko Hayashi
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=20180716205049.GA3663@rob-hp-laptop \
--to=robh@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=hayashi.kunihiko@socionext.com \
--cc=jaswinder.singh@linaro.org \
--cc=kishon@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=masami.hiramatsu@linaro.org \
--cc=yamada.masahiro@socionext.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).