dt-bindings: remoteproc: qcom,wcnss: Deprecate regulators for PDs

So far we have been doing all proxy votes by voting for raw voltages/load
through the regulator interface. But actually VDDCX and VDDMX represent
power domains that should be preferably managed using corner votes
through the power domain interface.

Document that those should be specified as power domains for
qcom,pronto-v1/2-pil and deprecate using them through the regulator
interface.

Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
Link: https://lore.kernel.org/r/20200916104135.25085-8-stephan@gerhold.net
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,wcnss-pil.txt b/Documentation/devicetree/bindings/remoteproc/qcom,wcnss-pil.txt
index d420f84ddfb06efd8ae6fd4ac0851ef345f252b1..cc0b7fc1c29b6d4b3ece3e5e26aff431ac5ce23b 100644 (file)
--- a/Documentation/devicetree/bindings/remoteproc/qcom,wcnss-pil.txt
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,wcnss-pil.txt
@@ -34,14 +34,25 @@ on the Qualcomm WCNSS core.
        Definition: should be "wdog", "fatal", optionally followed by "ready",
                    "handover", "stop-ack"
 
-- vddmx-supply:
-- vddcx-supply:
+- vddmx-supply: (deprecated for qcom,pronto-v1/2-pil)
+- vddcx-supply: (deprecated for qcom,pronto-v1/2-pil)
 - vddpx-supply:
        Usage: required
        Value type: <phandle>
        Definition: reference to the regulators to be held on behalf of the
                    booting of the WCNSS core
 
+- power-domains:
+       Usage: required (for qcom,pronto-v1/2-pil)
+       Value type: <phandle>
+       Definition: reference to the power domains to be held on behalf of the
+                   booting of the WCNSS core
+
+- power-domain-names:
+       Usage: required (for qcom,pronto-v1/2-pil)
+       Value type: <stringlist>
+       Definition: must be "cx", "mx"
+
 - qcom,smem-states:
        Usage: optional
        Value type: <prop-encoded-array>
@@ -111,8 +122,9 @@ pronto@fb204000 {
                              <&wcnss_smp2p_slave 3 0>;
        interrupt-names = "wdog", "fatal", "ready", "handover", "stop-ack";
 
-       vddmx-supply = <&pm8841_s1>;
-       vddcx-supply = <&pm8841_s2>;
+       power-domains = <&rpmpd MSM8974_VDDCX>, <&rpmpd MSM8974_VDDMX>;
+       power-domain-names = "cx", "mx";
+
        vddpx-supply = <&pm8941_s3>;
 
        qcom,smem-states = <&wcnss_smp2p_out 0>;