On Fri, Oct 24, 2025 at 05:56:05PM +0800, Ziyue Zhang wrote:
> The gcc_aux_clk is not required by the PCIe PHY on qcs8300 and is not
> specified in the device tree node. Hence, move the qcs8300 phy
> compatibility entry into the list of PHYs that require six clocks.
>
> Removed the phy_aux clock from the PCIe PHY binding as it is no longer
> used by any instance.
>
> Fixes: e46e59b77a9e ("dt-bindings: phy: qcom,sc8280xp-qmp-pcie-phy: Document the QCS8300 QMP PCIe PHY Gen4 x2")
> Signed-off-by: Ziyue Zhang <ziyue.zhang@oss.qualcomm.com>
Acked-by: Manivannan Sadhasivam <mani@kernel.org>
- Mani
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
> Reviewed-by: Johan Hovold <johan+linaro@kernel.org>
> ---
> .../phy/qcom,sc8280xp-qmp-pcie-phy.yaml | 17 ++---------------
> 1 file changed, 2 insertions(+), 15 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-pcie-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-pcie-phy.yaml
> index 119b4ff36dbd..d94d08752cec 100644
> --- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-pcie-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-pcie-phy.yaml
> @@ -55,7 +55,7 @@ properties:
>
> clocks:
> minItems: 5
> - maxItems: 7
> + maxItems: 6
>
> clock-names:
> minItems: 5
> @@ -66,7 +66,6 @@ properties:
> - enum: [rchng, refgen]
> - const: pipe
> - const: pipediv2
> - - const: phy_aux
>
> power-domains:
> maxItems: 1
> @@ -178,6 +177,7 @@ allOf:
> compatible:
> contains:
> enum:
> + - qcom,qcs8300-qmp-gen4x2-pcie-phy
> - qcom,sa8775p-qmp-gen4x2-pcie-phy
> - qcom,sa8775p-qmp-gen4x4-pcie-phy
> - qcom,sc8280xp-qmp-gen3x1-pcie-phy
> @@ -195,19 +195,6 @@ allOf:
> clock-names:
> minItems: 6
>
> - - if:
> - properties:
> - compatible:
> - contains:
> - enum:
> - - qcom,qcs8300-qmp-gen4x2-pcie-phy
> - then:
> - properties:
> - clocks:
> - minItems: 7
> - clock-names:
> - minItems: 7
> -
> - if:
> properties:
> compatible:
> --
> 2.34.1
>
--
மணிவண்ணன் சதாசிவம்