All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3] dt-bindings: phy: qcom: Relax required supplies
@ 2026-11-12 19:23 ` Rob Herring (Arm)
  0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring (Arm) @ 2026-11-12 19:23 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Manivannan Sadhasivam,
	Krzysztof Kozlowski, Conor Dooley, Wesley Cheng
  Cc: Krzysztof Kozlowski, linux-arm-msm, linux-phy, devicetree,
	linux-kernel

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


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH v3] dt-bindings: phy: qcom: Relax required supplies
@ 2026-11-12 19:23 ` Rob Herring (Arm)
  0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring (Arm) @ 2026-11-12 19:23 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Manivannan Sadhasivam,
	Krzysztof Kozlowski, Conor Dooley, Wesley Cheng
  Cc: Krzysztof Kozlowski, linux-arm-msm, linux-phy, devicetree,
	linux-kernel

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


^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-11-12 19:23 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-12 19:23 [PATCH v3] dt-bindings: phy: qcom: Relax required supplies Rob Herring (Arm)
2026-11-12 19:23 ` Rob Herring (Arm)

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.