On Wed, Oct 01, 2025 at 03:05:25PM -0700, Wesley Cheng wrote:
> Define a Glymur compatible string for the QMP PHY combo driver, along with
> resource requirements. This adds a new requirement for a clkref clock-name
> for each QMP PHY.
>
> Signed-off-by: Wesley Cheng <wesley.cheng@oss.qualcomm.com>
> ---
> .../phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml | 76 +++++++++++++++----
> 1 file changed, 63 insertions(+), 13 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml
> index c8bc512df08b..377d830f0855 100644
> --- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml
> @@ -16,6 +16,7 @@ description:
> properties:
> compatible:
> enum:
> + - qcom,glymur-qmp-usb3-dp-phy
> - qcom,sar2130p-qmp-usb3-dp-phy
> - qcom,sc7180-qmp-usb3-dp-phy
> - qcom,sc7280-qmp-usb3-dp-phy
> @@ -41,12 +42,7 @@ properties:
>
> clock-names:
> minItems: 4
> - items:
> - - const: aux
> - - const: ref
> - - const: com_aux
> - - const: usb3_pipe
> - - const: cfg_ahb
> + maxItems: 5
>
> power-domains:
> maxItems: 1
> @@ -63,6 +59,8 @@ properties:
>
> vdda-pll-supply: true
>
> + refgen-supply: true
> +
> "#clock-cells":
> const: 1
> description:
> @@ -105,6 +103,22 @@ required:
>
> allOf:
> - $ref: /schemas/usb/usb-switch.yaml#
> +
> + - if:
> + properties:
> + compatible:
> + enum:
> + - qcom,glymur-qmp-usb3-dp-phy
> + then:
> + properties:
Missing constraint for clocks.
> + clock-names:
> + items:
> + - const: aux
> + - const: ref
> + - const: com_aux
> + - const: usb3_pipe
> + - const: clkref
> +
> - if:
> properties:
> compatible:
> @@ -113,21 +127,45 @@ allOf:
> - qcom,sdm845-qmp-usb3-dp-phy
> then:
> properties:
> - clocks:
> - maxItems: 5
No, you can't remove this. clock-names doens't provide constraints (or
description) of the clocks property.
> clock-names:
> - maxItems: 5
> - else:
> + items:
> + - const: aux
> + - const: ref
> + - const: com_aux
> + - const: usb3_pipe
> + - const: cfg_ahb
> +
> + - if:
> + properties:
> + compatible:
> + enum:
> + - qcom,sar2130p-qmp-usb3-dp-phy
> + - qcom,sc7280-qmp-usb3-dp-phy
> + - qcom,sc8180x-qmp-usb3-dp-phy
> + - qcom,sc8280xp-qmp-usb43dp-phy
> + - qcom,sm6350-qmp-usb3-dp-phy
> + - qcom,sm8150-qmp-usb3-dp-phy
> + - qcom,sm8250-qmp-usb3-dp-phy
> + - qcom,sm8350-qmp-usb3-dp-phy
> + - qcom,sm8450-qmp-usb3-dp-phy
> + - qcom,sm8550-qmp-usb3-dp-phy
> + - qcom,sm8650-qmp-usb3-dp-phy
> + - qcom,sm8750-qmp-usb3-dp-phy
> + - qcom,x1e80100-qmp-usb3-dp-phy
> + then:
> properties:
> - clocks:
> - maxItems: 4
Same here.
> clock-names:
> - maxItems: 4
> + items:
> + - const: aux
> + - const: ref
> + - const: com_aux
> + - const: usb3_pipe
>
> - if:
> properties:
> compatible:
> enum:
> + - qcom,glymur-qmp-usb3-dp-phy
> - qcom,sar2130p-qmp-usb3-dp-phy
> - qcom,sc8280xp-qmp-usb43dp-phy
> - qcom,sm6350-qmp-usb3-dp-phy
> @@ -142,6 +180,18 @@ allOf:
> properties:
> power-domains: false
>
> + - if:
> + properties:
> + compatible:
> + enum:
> + - qcom,glymur-qmp-usb3-dp-phy
> + then:
> + required:
> + - refgen-supply
> + else:
> + properties:
> + refgen-supply: false
> +
> additionalProperties: false
>
> examples:
>
> --
> linux-phy mailing list
> linux-phy@lists.infradead.org
> https://lists.infradead.org/mailman/listinfo/linux-phy
--
With best wishes
Dmitry