[PATCH 2/9] dt-bindings: phy: qcom,qmp-usb: Add Glymur USB UNI PHY compatible
Dmitry Baryshkov
dmitry.baryshkov at oss.qualcomm.com
Sat Sep 20 08:22:36 PDT 2025
On Fri, Sep 19, 2025 at 08:21:01PM -0700, Wesley Cheng wrote:
> The Glymur USB subsystem contains a multiport controller, which utilizes
> two QMP UNI PHYs. Add the proper compatible string for the Glymur SoC.
>
> Signed-off-by: Wesley Cheng <wesley.cheng at oss.qualcomm.com>
> ---
> .../bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml
> index a1b55168e050..772a727a5462 100644
> --- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml
> @@ -16,6 +16,7 @@ description:
> properties:
> compatible:
> enum:
> + - qcom,glymur-qmp-usb3-uni-phy
> - qcom,ipq5424-qmp-usb3-phy
> - qcom,ipq6018-qmp-usb3-phy
> - qcom,ipq8074-qmp-usb3-phy
> @@ -62,6 +63,8 @@ properties:
>
> vdda-pll-supply: true
>
> + refgen-supply: true
You've added it, but it's not referenced as required. Why is it so?
> +
> "#clock-cells":
> const: 0
>
> @@ -139,6 +142,7 @@ allOf:
> compatible:
> contains:
> enum:
> + - qcom,glymur-qmp-usb3-uni-phy
> - qcom,sdm845-qmp-usb3-uni-phy
> then:
> properties:
> @@ -147,7 +151,7 @@ allOf:
> clock-names:
> items:
> - const: aux
> - - const: cfg_ahb
> + - enum: [cfg_ahb, clkref]
Why is it being placed here? Please comment in the commit message.
> - const: ref
> - const: com_aux
> - const: pipe
> @@ -157,6 +161,7 @@ allOf:
> compatible:
> contains:
> enum:
> + - qcom,glymur-qmp-usb3-uni-phy
> - qcom,sa8775p-qmp-usb3-uni-phy
> - qcom,sc8180x-qmp-usb3-uni-phy
> - qcom,sc8280xp-qmp-usb3-uni-phy
>
> --
> linux-phy mailing list
> linux-phy at lists.infradead.org
> https://lists.infradead.org/mailman/listinfo/linux-phy
--
With best wishes
Dmitry
More information about the linux-phy
mailing list