Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f1219217 authored by Viresh Kumar's avatar Viresh Kumar Committed by Alex Shi
Browse files

PM / OPP: Rename OPP nodes as opp@<opp-hz> 



It would be better to name OPP nodes as opp@<opp-hz> as that will ensure
that multiple DT nodes don't contain the same frequency. Of course we
expect the writer to name the node with its opp-hz frequency and not any
other frequency.

And that will let the compile error out if multiple nodes are using the
same opp-hz frequency.

Suggested-by: default avatarStephen Boyd <sboyd@codeaurora.org>
Reviewed-by: default avatarStephen Boyd <sboyd@codeaurora.org>
Acked-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
(cherry picked from commit 754dcf35f34698661801ae1d391efa02affe83a7)
Signed-off-by: default avatarAlex Shi <alex.shi@linaro.org>
parent c68b222a

Documentation/devicetree/bindings/opp/opp.txt

+19 −19
Original line number Diff line number Diff line
@@ -165,20 +165,20 @@ Example 1: Single cluster Dual-core ARM cortex A9, switch DVFS states together. 
		compatible = "operating-points-v2";

		opp-shared;



		opp00 {

		opp@1000000000 {

			opp-hz = /bits/ 64 <1000000000>;

			opp-microvolt = <970000 975000 985000>;

			opp-microamp = <70000>;

			clock-latency-ns = <300000>;

			opp-suspend;

		};

		opp01 {

		opp@1100000000 {

			opp-hz = /bits/ 64 <1100000000>;

			opp-microvolt = <980000 1000000 1010000>;

			opp-microamp = <80000>;

			clock-latency-ns = <310000>;

		};

		opp02 {

		opp@1200000000 {

			opp-hz = /bits/ 64 <1200000000>;

			opp-microvolt = <1025000>;

			clock-latency-ns = <290000>;
@@ -244,20 +244,20 @@ independently. 
		 * independently.

		 */



		opp00 {

		opp@1000000000 {

			opp-hz = /bits/ 64 <1000000000>;

			opp-microvolt = <970000 975000 985000>;

			opp-microamp = <70000>;

			clock-latency-ns = <300000>;

			opp-suspend;

		};

		opp01 {

		opp@1100000000 {

			opp-hz = /bits/ 64 <1100000000>;

			opp-microvolt = <980000 1000000 1010000>;

			opp-microamp = <80000>;

			clock-latency-ns = <310000>;

		};

		opp02 {

		opp@1200000000 {

			opp-hz = /bits/ 64 <1200000000>;

			opp-microvolt = <1025000>;

			opp-microamp = <90000;
@@ -320,20 +320,20 @@ DVFS state together. 
		compatible = "operating-points-v2";

		opp-shared;



		opp00 {

		opp@1000000000 {

			opp-hz = /bits/ 64 <1000000000>;

			opp-microvolt = <970000 975000 985000>;

			opp-microamp = <70000>;

			clock-latency-ns = <300000>;

			opp-suspend;

		};

		opp01 {

		opp@1100000000 {

			opp-hz = /bits/ 64 <1100000000>;

			opp-microvolt = <980000 1000000 1010000>;

			opp-microamp = <80000>;

			clock-latency-ns = <310000>;

		};

		opp02 {

		opp@1200000000 {

			opp-hz = /bits/ 64 <1200000000>;

			opp-microvolt = <1025000>;

			opp-microamp = <90000>;
@@ -346,20 +346,20 @@ DVFS state together. 
		compatible = "operating-points-v2";

		opp-shared;



		opp10 {

		opp@1300000000 {

			opp-hz = /bits/ 64 <1300000000>;

			opp-microvolt = <1045000 1050000 1055000>;

			opp-microamp = <95000>;

			clock-latency-ns = <400000>;

			opp-suspend;

		};

		opp11 {

		opp@1400000000 {

			opp-hz = /bits/ 64 <1400000000>;

			opp-microvolt = <1075000>;

			opp-microamp = <100000>;

			clock-latency-ns = <400000>;

		};

		opp12 {

		opp@1500000000 {

			opp-hz = /bits/ 64 <1500000000>;

			opp-microvolt = <1010000 1100000 1110000>;

			opp-microamp = <95000>;
@@ -386,7 +386,7 @@ Example 4: Handling multiple regulators 
		compatible = "operating-points-v2";

		opp-shared;



		opp00 {

		opp@1000000000 {

			opp-hz = /bits/ 64 <1000000000>;

			opp-microvolt = <970000>, /* Supply 0 */

					<960000>, /* Supply 1 */
@@ -399,7 +399,7 @@ Example 4: Handling multiple regulators 


		/* OR */



		opp00 {

		opp@1000000000 {

			opp-hz = /bits/ 64 <1000000000>;

			opp-microvolt = <970000 975000 985000>, /* Supply 0 */

					<960000 965000 975000>, /* Supply 1 */
@@ -412,7 +412,7 @@ Example 4: Handling multiple regulators 


		/* OR */



		opp00 {

		opp@1000000000 {

			opp-hz = /bits/ 64 <1000000000>;

			opp-microvolt = <970000 975000 985000>, /* Supply 0 */

					<960000 965000 975000>, /* Supply 1 */
@@ -444,7 +444,7 @@ Example 5: opp-supported-hw 
		status = "okay";

		opp-shared;



		opp00 {

		opp@600000000 {

			/*

			 * Supports all substrate and process versions for 0xF

			 * cuts, i.e. only first four cuts.
@@ -455,7 +455,7 @@ Example 5: opp-supported-hw 
			...

		};



		opp01 {

		opp@800000000 {

			/*

			 * Supports:

			 * - cuts: only one, 6th cut (represented by 6th bit).
@@ -487,7 +487,7 @@ Example 6: opp-microvolt-<name>, opp-microamp-<name>: 
		compatible = "operating-points-v2";

		opp-shared;



		opp00 {

		opp@1000000000 {

			opp-hz = /bits/ 64 <1000000000>;

			opp-microvolt-slow = <900000 915000 925000>;

			opp-microvolt-fast = <970000 975000 985000>;
@@ -495,7 +495,7 @@ Example 6: opp-microvolt-<name>, opp-microamp-<name>: 
			opp-microamp-fast =  <71000>;

		};



		opp01 {

		opp@1200000000 {

			opp-hz = /bits/ 64 <1200000000>;

			opp-microvolt-slow = <900000 915000 925000>, /* Supply vcc0 */

					      <910000 925000 935000>; /* Supply vcc1 */