diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2017-06-21 23:45:11 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-06-23 19:28:52 -0400 |
commit | b21569cf1de925e0a42c9964bd7f520cb4a4d875 (patch) | |
tree | c2062ef3b9d917e3c43ebdc2c0c2c81a640be4a2 | |
parent | 1fae788ed640e2a961c2edab54bfd56c73e2506a (diff) |
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: Krzysztof Kozlowski <krzk@kernel.org>
Reported-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Suggested-by: Mark Rutland <mark.rutland@arm.com>
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>
-rw-r--r-- | Documentation/devicetree/bindings/opp/opp.txt | 38 |
1 files changed, 19 insertions, 19 deletions
diff --git a/Documentation/devicetree/bindings/opp/opp.txt b/Documentation/devicetree/bindings/opp/opp.txt index 63725498bd20..e36d261b9ba6 100644 --- a/Documentation/devicetree/bindings/opp/opp.txt +++ b/Documentation/devicetree/bindings/opp/opp.txt | |||
@@ -186,20 +186,20 @@ Example 1: Single cluster Dual-core ARM cortex A9, switch DVFS states together. | |||
186 | compatible = "operating-points-v2"; | 186 | compatible = "operating-points-v2"; |
187 | opp-shared; | 187 | opp-shared; |
188 | 188 | ||
189 | opp@1000000000 { | 189 | opp-1000000000 { |
190 | opp-hz = /bits/ 64 <1000000000>; | 190 | opp-hz = /bits/ 64 <1000000000>; |
191 | opp-microvolt = <975000 970000 985000>; | 191 | opp-microvolt = <975000 970000 985000>; |
192 | opp-microamp = <70000>; | 192 | opp-microamp = <70000>; |
193 | clock-latency-ns = <300000>; | 193 | clock-latency-ns = <300000>; |
194 | opp-suspend; | 194 | opp-suspend; |
195 | }; | 195 | }; |
196 | opp@1100000000 { | 196 | opp-1100000000 { |
197 | opp-hz = /bits/ 64 <1100000000>; | 197 | opp-hz = /bits/ 64 <1100000000>; |
198 | opp-microvolt = <1000000 980000 1010000>; | 198 | opp-microvolt = <1000000 980000 1010000>; |
199 | opp-microamp = <80000>; | 199 | opp-microamp = <80000>; |
200 | clock-latency-ns = <310000>; | 200 | clock-latency-ns = <310000>; |
201 | }; | 201 | }; |
202 | opp@1200000000 { | 202 | opp-1200000000 { |
203 | opp-hz = /bits/ 64 <1200000000>; | 203 | opp-hz = /bits/ 64 <1200000000>; |
204 | opp-microvolt = <1025000>; | 204 | opp-microvolt = <1025000>; |
205 | clock-latency-ns = <290000>; | 205 | clock-latency-ns = <290000>; |
@@ -265,20 +265,20 @@ independently. | |||
265 | * independently. | 265 | * independently. |
266 | */ | 266 | */ |
267 | 267 | ||
268 | opp@1000000000 { | 268 | opp-1000000000 { |
269 | opp-hz = /bits/ 64 <1000000000>; | 269 | opp-hz = /bits/ 64 <1000000000>; |
270 | opp-microvolt = <975000 970000 985000>; | 270 | opp-microvolt = <975000 970000 985000>; |
271 | opp-microamp = <70000>; | 271 | opp-microamp = <70000>; |
272 | clock-latency-ns = <300000>; | 272 | clock-latency-ns = <300000>; |
273 | opp-suspend; | 273 | opp-suspend; |
274 | }; | 274 | }; |
275 | opp@1100000000 { | 275 | opp-1100000000 { |
276 | opp-hz = /bits/ 64 <1100000000>; | 276 | opp-hz = /bits/ 64 <1100000000>; |
277 | opp-microvolt = <1000000 980000 1010000>; | 277 | opp-microvolt = <1000000 980000 1010000>; |
278 | opp-microamp = <80000>; | 278 | opp-microamp = <80000>; |
279 | clock-latency-ns = <310000>; | 279 | clock-latency-ns = <310000>; |
280 | }; | 280 | }; |
281 | opp@1200000000 { | 281 | opp-1200000000 { |
282 | opp-hz = /bits/ 64 <1200000000>; | 282 | opp-hz = /bits/ 64 <1200000000>; |
283 | opp-microvolt = <1025000>; | 283 | opp-microvolt = <1025000>; |
284 | opp-microamp = <90000; | 284 | opp-microamp = <90000; |
@@ -341,20 +341,20 @@ DVFS state together. | |||
341 | compatible = "operating-points-v2"; | 341 | compatible = "operating-points-v2"; |
342 | opp-shared; | 342 | opp-shared; |
343 | 343 | ||
344 | opp@1000000000 { | 344 | opp-1000000000 { |
345 | opp-hz = /bits/ 64 <1000000000>; | 345 | opp-hz = /bits/ 64 <1000000000>; |
346 | opp-microvolt = <975000 970000 985000>; | 346 | opp-microvolt = <975000 970000 985000>; |
347 | opp-microamp = <70000>; | 347 | opp-microamp = <70000>; |
348 | clock-latency-ns = <300000>; | 348 | clock-latency-ns = <300000>; |
349 | opp-suspend; | 349 | opp-suspend; |
350 | }; | 350 | }; |
351 | opp@1100000000 { | 351 | opp-1100000000 { |
352 | opp-hz = /bits/ 64 <1100000000>; | 352 | opp-hz = /bits/ 64 <1100000000>; |
353 | opp-microvolt = <1000000 980000 1010000>; | 353 | opp-microvolt = <1000000 980000 1010000>; |
354 | opp-microamp = <80000>; | 354 | opp-microamp = <80000>; |
355 | clock-latency-ns = <310000>; | 355 | clock-latency-ns = <310000>; |
356 | }; | 356 | }; |
357 | opp@1200000000 { | 357 | opp-1200000000 { |
358 | opp-hz = /bits/ 64 <1200000000>; | 358 | opp-hz = /bits/ 64 <1200000000>; |
359 | opp-microvolt = <1025000>; | 359 | opp-microvolt = <1025000>; |
360 | opp-microamp = <90000>; | 360 | opp-microamp = <90000>; |
@@ -367,20 +367,20 @@ DVFS state together. | |||
367 | compatible = "operating-points-v2"; | 367 | compatible = "operating-points-v2"; |
368 | opp-shared; | 368 | opp-shared; |
369 | 369 | ||
370 | opp@1300000000 { | 370 | opp-1300000000 { |
371 | opp-hz = /bits/ 64 <1300000000>; | 371 | opp-hz = /bits/ 64 <1300000000>; |
372 | opp-microvolt = <1050000 1045000 1055000>; | 372 | opp-microvolt = <1050000 1045000 1055000>; |
373 | opp-microamp = <95000>; | 373 | opp-microamp = <95000>; |
374 | clock-latency-ns = <400000>; | 374 | clock-latency-ns = <400000>; |
375 | opp-suspend; | 375 | opp-suspend; |
376 | }; | 376 | }; |
377 | opp@1400000000 { | 377 | opp-1400000000 { |
378 | opp-hz = /bits/ 64 <1400000000>; | 378 | opp-hz = /bits/ 64 <1400000000>; |
379 | opp-microvolt = <1075000>; | 379 | opp-microvolt = <1075000>; |
380 | opp-microamp = <100000>; | 380 | opp-microamp = <100000>; |
381 | clock-latency-ns = <400000>; | 381 | clock-latency-ns = <400000>; |
382 | }; | 382 | }; |
383 | opp@1500000000 { | 383 | opp-1500000000 { |
384 | opp-hz = /bits/ 64 <1500000000>; | 384 | opp-hz = /bits/ 64 <1500000000>; |
385 | opp-microvolt = <1100000 1010000 1110000>; | 385 | opp-microvolt = <1100000 1010000 1110000>; |
386 | opp-microamp = <95000>; | 386 | opp-microamp = <95000>; |
@@ -409,7 +409,7 @@ Example 4: Handling multiple regulators | |||
409 | compatible = "operating-points-v2"; | 409 | compatible = "operating-points-v2"; |
410 | opp-shared; | 410 | opp-shared; |
411 | 411 | ||
412 | opp@1000000000 { | 412 | opp-1000000000 { |
413 | opp-hz = /bits/ 64 <1000000000>; | 413 | opp-hz = /bits/ 64 <1000000000>; |
414 | opp-microvolt = <970000>, /* Supply 0 */ | 414 | opp-microvolt = <970000>, /* Supply 0 */ |
415 | <960000>, /* Supply 1 */ | 415 | <960000>, /* Supply 1 */ |
@@ -422,7 +422,7 @@ Example 4: Handling multiple regulators | |||
422 | 422 | ||
423 | /* OR */ | 423 | /* OR */ |
424 | 424 | ||
425 | opp@1000000000 { | 425 | opp-1000000000 { |
426 | opp-hz = /bits/ 64 <1000000000>; | 426 | opp-hz = /bits/ 64 <1000000000>; |
427 | opp-microvolt = <975000 970000 985000>, /* Supply 0 */ | 427 | opp-microvolt = <975000 970000 985000>, /* Supply 0 */ |
428 | <965000 960000 975000>, /* Supply 1 */ | 428 | <965000 960000 975000>, /* Supply 1 */ |
@@ -435,7 +435,7 @@ Example 4: Handling multiple regulators | |||
435 | 435 | ||
436 | /* OR */ | 436 | /* OR */ |
437 | 437 | ||
438 | opp@1000000000 { | 438 | opp-1000000000 { |
439 | opp-hz = /bits/ 64 <1000000000>; | 439 | opp-hz = /bits/ 64 <1000000000>; |
440 | opp-microvolt = <975000 970000 985000>, /* Supply 0 */ | 440 | opp-microvolt = <975000 970000 985000>, /* Supply 0 */ |
441 | <965000 960000 975000>, /* Supply 1 */ | 441 | <965000 960000 975000>, /* Supply 1 */ |
@@ -467,7 +467,7 @@ Example 5: opp-supported-hw | |||
467 | status = "okay"; | 467 | status = "okay"; |
468 | opp-shared; | 468 | opp-shared; |
469 | 469 | ||
470 | opp@600000000 { | 470 | opp-600000000 { |
471 | /* | 471 | /* |
472 | * Supports all substrate and process versions for 0xF | 472 | * Supports all substrate and process versions for 0xF |
473 | * cuts, i.e. only first four cuts. | 473 | * cuts, i.e. only first four cuts. |
@@ -478,7 +478,7 @@ Example 5: opp-supported-hw | |||
478 | ... | 478 | ... |
479 | }; | 479 | }; |
480 | 480 | ||
481 | opp@800000000 { | 481 | opp-800000000 { |
482 | /* | 482 | /* |
483 | * Supports: | 483 | * Supports: |
484 | * - cuts: only one, 6th cut (represented by 6th bit). | 484 | * - cuts: only one, 6th cut (represented by 6th bit). |
@@ -510,7 +510,7 @@ Example 6: opp-microvolt-<name>, opp-microamp-<name>: | |||
510 | compatible = "operating-points-v2"; | 510 | compatible = "operating-points-v2"; |
511 | opp-shared; | 511 | opp-shared; |
512 | 512 | ||
513 | opp@1000000000 { | 513 | opp-1000000000 { |
514 | opp-hz = /bits/ 64 <1000000000>; | 514 | opp-hz = /bits/ 64 <1000000000>; |
515 | opp-microvolt-slow = <915000 900000 925000>; | 515 | opp-microvolt-slow = <915000 900000 925000>; |
516 | opp-microvolt-fast = <975000 970000 985000>; | 516 | opp-microvolt-fast = <975000 970000 985000>; |
@@ -518,7 +518,7 @@ Example 6: opp-microvolt-<name>, opp-microamp-<name>: | |||
518 | opp-microamp-fast = <71000>; | 518 | opp-microamp-fast = <71000>; |
519 | }; | 519 | }; |
520 | 520 | ||
521 | opp@1200000000 { | 521 | opp-1200000000 { |
522 | opp-hz = /bits/ 64 <1200000000>; | 522 | opp-hz = /bits/ 64 <1200000000>; |
523 | opp-microvolt-slow = <915000 900000 925000>, /* Supply vcc0 */ | 523 | opp-microvolt-slow = <915000 900000 925000>, /* Supply vcc0 */ |
524 | <925000 910000 935000>; /* Supply vcc1 */ | 524 | <925000 910000 935000>; /* Supply vcc1 */ |