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

Commit b21569cf authored by Viresh Kumar's avatar Viresh Kumar Committed by Rafael J. Wysocki
Browse files

PM / OPP: Use - instead of @ for DT entries 



Compiling the DT file with W=1, DTC warns like follows:

Warning (unit_address_vs_reg): Node /opp_table0/opp@1000000000 has a
unit name, but no reg property

Fix this by replacing '@' with '-' as the OPP nodes will never have a
"reg" property.

Reported-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
Reported-by: default avatarMasahiro Yamada <yamada.masahiro@socionext.com>
Suggested-by: default avatarMark Rutland <mark.rutland@arm.com>
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Acked-by: default avatarRob Herring <robh@kernel.org>
Reviewed-by: default avatarStephen Boyd <sboyd@codeaurora.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 1fae788e

Documentation/devicetree/bindings/opp/opp.txt

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

		opp-shared;



		opp@1000000000 {

		opp-1000000000 {

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

			opp-microvolt = <975000 970000 985000>;

			opp-microamp = <70000>;

			clock-latency-ns = <300000>;

			opp-suspend;

		};

		opp@1100000000 {

		opp-1100000000 {

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

			opp-microvolt = <1000000 980000 1010000>;

			opp-microamp = <80000>;

			clock-latency-ns = <310000>;

		};

		opp@1200000000 {

		opp-1200000000 {

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

			opp-microvolt = <1025000>;

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

		 */



		opp@1000000000 {

		opp-1000000000 {

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

			opp-microvolt = <975000 970000 985000>;

			opp-microamp = <70000>;

			clock-latency-ns = <300000>;

			opp-suspend;

		};

		opp@1100000000 {

		opp-1100000000 {

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

			opp-microvolt = <1000000 980000 1010000>;

			opp-microamp = <80000>;

			clock-latency-ns = <310000>;

		};

		opp@1200000000 {

		opp-1200000000 {

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

			opp-microvolt = <1025000>;

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

		opp-shared;



		opp@1000000000 {

		opp-1000000000 {

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

			opp-microvolt = <975000 970000 985000>;

			opp-microamp = <70000>;

			clock-latency-ns = <300000>;

			opp-suspend;

		};

		opp@1100000000 {

		opp-1100000000 {

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

			opp-microvolt = <1000000 980000 1010000>;

			opp-microamp = <80000>;

			clock-latency-ns = <310000>;

		};

		opp@1200000000 {

		opp-1200000000 {

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

			opp-microvolt = <1025000>;

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

		opp-shared;



		opp@1300000000 {

		opp-1300000000 {

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

			opp-microvolt = <1050000 1045000 1055000>;

			opp-microamp = <95000>;

			clock-latency-ns = <400000>;

			opp-suspend;

		};

		opp@1400000000 {

		opp-1400000000 {

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

			opp-microvolt = <1075000>;

			opp-microamp = <100000>;

			clock-latency-ns = <400000>;

		};

		opp@1500000000 {

		opp-1500000000 {

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

			opp-microvolt = <1100000 1010000 1110000>;

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

		opp-shared;



		opp@1000000000 {

		opp-1000000000 {

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

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

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


		/* OR */



		opp@1000000000 {

		opp-1000000000 {

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

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

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


		/* OR */



		opp@1000000000 {

		opp-1000000000 {

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

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

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

		opp-shared;



		opp@600000000 {

		opp-600000000 {

			/*

			 * Supports all substrate and process versions for 0xF

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

		};



		opp@800000000 {

		opp-800000000 {

			/*

			 * Supports:

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

		opp-shared;



		opp@1000000000 {

		opp-1000000000 {

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

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

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

		};



		opp@1200000000 {

		opp-1200000000 {

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

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

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