[PATCH v3] dt-bindings: phy: qcom: Relax required supplies

[PATCH v3] dt-bindings: phy: qcom: Relax required supplies

[Rob Herring (Arm)]

For the QUSB2 PHY binding, keep the three regulator supplies required
except for legacy IPQ6018 nodes that do not describe the board rails.

This relaxes the requirements for the IPQ5210 as well as if it is
compatible with IPQ6018, then it shouldn't have more required
properties either.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
v3:
 - Rebase on phy-next

v2:
 - Fix trailing whitespace
 - Add blurb to commit msg (for sashiko)
---
 .../bindings/phy/qcom,qusb2-phy.yaml          | 30 ++++++++++++++++---
 1 file changed, 26 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml
index cfc5a1d16b7f..de0041dd2846 100644
--- a/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml
@@ -174,11 +174,35 @@ required:
   - "#phy-cells"
   - clocks
   - clock-names
-  - vdda-pll-supply
-  - vdda-phy-dpdm-supply
   - resets
 
 allOf:
+  - if:
+      not:
+        properties:
+          compatible:
+            contains:
+              enum:
+                - qcom,ipq6018-qusb2-phy
+                - qcom,ipq8074-qusb2-phy
+    then:
+      required:
+        - vdda-pll-supply
+        - vdda-phy-dpdm-supply
+
+  - if:
+      not:
+        properties:
+          compatible:
+            contains:
+              enum:
+                - qcom,ipq6018-qusb2-phy
+                - qcom,ipq8074-qusb2-phy
+                - qcom,msm8996-qusb2-phy
+    then:
+      required:
+        - vdd-supply
+
   - if:
       not:
         properties:
@@ -208,8 +232,6 @@ allOf:
       properties:
         power-domains: false
         required-opps: false
-      required:
-        - vdd-supply
 
 additionalProperties: false
 
-- 
2.53.0

Re: [PATCH v3] dt-bindings: phy: qcom: Relax required supplies

[Vinod Koul]

On Thu, 12 Nov 2026 13:23:44 -0600, Rob Herring (Arm) wrote:
> For the QUSB2 PHY binding, keep the three regulator supplies required
> except for legacy IPQ6018 nodes that do not describe the board rails.
> 
> This relaxes the requirements for the IPQ5210 as well as if it is
> compatible with IPQ6018, then it shouldn't have more required
> properties either.
> 
> [...]

Applied, thanks!

[1/1] dt-bindings: phy: qcom: Relax required supplies
      commit: e77b61b463909da3e21a337295954f48f4ca80ec

Best regards,
-- 
~Vinod