PM / OPP: Fix incorrect cpu-supply property in binding
authorViresh Kumar <viresh.kumar@linaro.org>
Thu, 1 Dec 2016 10:58:14 +0000 (16:28 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 6 Dec 2016 01:27:58 +0000 (02:27 +0100)
The regulator bindings allow the "<name>-supply" property to define a
single parent supply and not a list of parents.

Fix the wrong example code present in OPP bindings.

While at it also change the compatible string as Rob pointed out earlier
that none of A7 implementation have multiple supplies for the CPU core.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Acked-by: Rob Herring <robh@kernel.org>
Reviewed-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Documentation/devicetree/bindings/opp/opp.txt

index ee91cbd..f0239f6 100644 (file)
@@ -386,10 +386,12 @@ Example 4: Handling multiple regulators
 / {
        cpus {
                cpu@0 {
-                       compatible = "arm,cortex-a7";
+                       compatible = "vendor,cpu-type";
                        ...
 
-                       cpu-supply = <&cpu_supply0>, <&cpu_supply1>, <&cpu_supply2>;
+                       vcc0-supply = <&cpu_supply0>;
+                       vcc1-supply = <&cpu_supply1>;
+                       vcc2-supply = <&cpu_supply2>;
                        operating-points-v2 = <&cpu0_opp_table>;
                };
        };