dt-bindings: soc: qcom: qcom,saw2: define optional regulator node
authorDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Tue, 2 Jan 2024 05:17:23 +0000 (07:17 +0200)
committerBjorn Andersson <andersson@kernel.org>
Fri, 16 Feb 2024 17:30:57 +0000 (11:30 -0600)
The SAW2 device can optionally provide a voltage regulator supplying the
CPU core, cluster or L2 cache. Change the boolean 'regulator' property
into a proper regulator description. This breaks schema compatibility
for the sake of properly describing the regulator.

Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Link: https://lore.kernel.org/r/20240102-saw2-spm-regulator-v7-3-0472ec237f49@linaro.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Documentation/devicetree/bindings/soc/qcom/qcom,saw2.yaml

index 2178fea37b8c00f65b7727bb70f702aecf3d3b0a..f9306f2a6e99e8f17ef51c94c9b69127d436e1f4 100644 (file)
@@ -53,7 +53,7 @@ properties:
     minItems: 1
 
   regulator:
-    type: boolean
+    $ref: /schemas/regulator/regulator.yaml#
     description: Indicates that this SPM device acts as a regulator device
       device for the core (CPU or Cache) the SPM is attached to.
 
@@ -102,4 +102,17 @@ examples:
         reg = <0x17912000 0x1000>;
     };
 
+  - |
+    /*
+     * Example 3: SAW2 with the bundled regulator definition.
+     */
+    power-manager@2089000 {
+        compatible = "qcom,apq8064-saw2-v1.1-cpu", "qcom,saw2";
+        reg = <0x02089000 0x1000>, <0x02009000 0x1000>;
+
+        regulator {
+            regulator-min-microvolt = <850000>;
+            regulator-max-microvolt = <1300000>;
+        };
+    };
 ...