Skip to content

Commit

Permalink
ARM: sun8i: Use - instead of @ for DT OPP entries
Browse files Browse the repository at this point in the history
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 <[email protected]>
Reported-by: Masahiro Yamada <[email protected]>
Suggested-by: Mark Rutland <[email protected]>
Signed-off-by: Viresh Kumar <[email protected]>
Acked-by: Maxime Ripard <[email protected]>
Acked-by: Rob Herring <[email protected]>
Signed-off-by: Maxime Ripard <[email protected]>
  • Loading branch information
vireshk authored and mripard committed May 14, 2017
1 parent ef00b70 commit 84f1738
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 17 deletions.
4 changes: 2 additions & 2 deletions arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts
Original file line number Diff line number Diff line change
Expand Up @@ -88,13 +88,13 @@
};

&cpu0_opp_table {
opp@1104000000 {
opp-1104000000 {
opp-hz = /bits/ 64 <1104000000>;
opp-microvolt = <1320000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@1200000000 {
opp-1200000000 {
opp-hz = /bits/ 64 <1200000000>;
opp-microvolt = <1320000>;
clock-latency-ns = <244144>; /* 8 32k periods */
Expand Down
30 changes: 15 additions & 15 deletions arch/arm/boot/dts/sun8i-a33.dtsi
Original file line number Diff line number Diff line change
Expand Up @@ -50,73 +50,73 @@
compatible = "operating-points-v2";
opp-shared;

opp@120000000 {
opp-120000000 {
opp-hz = /bits/ 64 <120000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@240000000 {
opp-240000000 {
opp-hz = /bits/ 64 <240000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@312000000 {
opp-312000000 {
opp-hz = /bits/ 64 <312000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@408000000 {
opp-408000000 {
opp-hz = /bits/ 64 <408000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@480000000 {
opp-480000000 {
opp-hz = /bits/ 64 <480000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@504000000 {
opp-504000000 {
opp-hz = /bits/ 64 <504000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@600000000 {
opp-600000000 {
opp-hz = /bits/ 64 <600000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@648000000 {
opp-648000000 {
opp-hz = /bits/ 64 <648000000>;
opp-microvolt = <1040000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@720000000 {
opp-720000000 {
opp-hz = /bits/ 64 <720000000>;
opp-microvolt = <1100000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@816000000 {
opp-816000000 {
opp-hz = /bits/ 64 <816000000>;
opp-microvolt = <1100000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@912000000 {
opp-912000000 {
opp-hz = /bits/ 64 <912000000>;
opp-microvolt = <1200000>;
clock-latency-ns = <244144>; /* 8 32k periods */
};

opp@1008000000 {
opp-1008000000 {
opp-hz = /bits/ 64 <1008000000>;
opp-microvolt = <1200000>;
clock-latency-ns = <244144>; /* 8 32k periods */
Expand Down Expand Up @@ -164,15 +164,15 @@
mali_opp_table: gpu-opp-table {
compatible = "operating-points-v2";

opp@144000000 {
opp-144000000 {
opp-hz = /bits/ 64 <144000000>;
};

opp@240000000 {
opp-240000000 {
opp-hz = /bits/ 64 <240000000>;
};

opp@384000000 {
opp-384000000 {
opp-hz = /bits/ 64 <384000000>;
};
};
Expand Down

0 comments on commit 84f1738

Please sign in to comment.