From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Liam Beguin <liambeguin@gmail.com>
Cc: Anurag Kumar Vulisha <anurag.kumar.vulisha@xilinx.com>,
Kishon Vijay Abraham I <kishon@ti.com>,
Vinod Koul <vkoul@kernel.org>, Rob Herring <robh+dt@kernel.org>,
Michal Simek <michal.simek@xilinx.com>,
Liam Beguin <lvb@xiphos.com>,
linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/1] dt-bindings: phy: zynqmp-psgtr: fix USB phy name
Date: Wed, 17 Nov 2021 09:50:18 +0200 [thread overview]
Message-ID: <YZS0Ov0CL9mcrk/l@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20211117003841.2030813-1-lvb@xiphos.com>
Hi Liam,
Thank you for the patch.
On Tue, Nov 16, 2021 at 07:38:41PM -0500, Liam Beguin wrote:
> PHY_TYPE_USB is undefined and was added as PHY_TYPE_USB2 and
> PHY_TYPE_USB3 in 2fbbc96d1600 (phy: Add PHY header file for DT x Driver
> defines, 2014-11-04). Fix documentation to avoid misleading users.
>
> Signed-off-by: Liam Beguin <lvb@xiphos.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> Documentation/devicetree/bindings/phy/xlnx,zynqmp-psgtr.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/phy/xlnx,zynqmp-psgtr.yaml b/Documentation/devicetree/bindings/phy/xlnx,zynqmp-psgtr.yaml
> index 04d5654efb38..79906519c652 100644
> --- a/Documentation/devicetree/bindings/phy/xlnx,zynqmp-psgtr.yaml
> +++ b/Documentation/devicetree/bindings/phy/xlnx,zynqmp-psgtr.yaml
> @@ -29,7 +29,7 @@ properties:
> - PHY_TYPE_PCIE
> - PHY_TYPE_SATA
> - PHY_TYPE_SGMII
> - - PHY_TYPE_USB
> + - PHY_TYPE_USB3
> - description: The PHY instance
> minimum: 0
> maximum: 1 # for DP, SATA or USB
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2021-11-17 7:50 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-17 0:38 [PATCH 1/1] dt-bindings: phy: zynqmp-psgtr: fix USB phy name Liam Beguin
2021-11-17 7:50 ` Laurent Pinchart [this message]
2021-11-23 5:11 ` Vinod Koul
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=YZS0Ov0CL9mcrk/l@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=anurag.kumar.vulisha@xilinx.com \
--cc=devicetree@vger.kernel.org \
--cc=kishon@ti.com \
--cc=liambeguin@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=lvb@xiphos.com \
--cc=michal.simek@xilinx.com \
--cc=robh+dt@kernel.org \
--cc=vkoul@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox