Date:   Wed, 17 Nov 2021 09:50:18 +0200
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
Message-ID: <YZS0Ov0CL9mcrk/l@pendragon.ideasonboard.com>
References: <20211117003841.2030813-1-lvb@xiphos.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20211117003841.2030813-1-lvb@xiphos.com>
Precedence: bulk

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