arm64: dts: exynos: Use - instead of @ for DT OPP 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>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Acked-by: Rob Herring <robh@kernel.org>
[k.kozlowski: Split patch per ARM and ARM64]
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

diff --git a/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi
index c42dc39..ec11343 100644 (file)
--- a/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi
@@ -94,27 +94,27 @@
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp@400000000 {
+               opp-400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                        opp-microvolt = <1075000>;
                };
-               opp@267000000 {
+               opp-267000000 {
                        opp-hz = /bits/ 64 <267000000>;
                        opp-microvolt = <1000000>;
                };
-               opp@200000000 {
+               opp-200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                        opp-microvolt = <975000>;
                };
-               opp@160000000 {
+               opp-160000000 {
                        opp-hz = /bits/ 64 <160000000>;
                        opp-microvolt = <962500>;
                };
-               opp@134000000 {
+               opp-134000000 {
                        opp-hz = /bits/ 64 <134000000>;
                        opp-microvolt = <950000>;
                };
-               opp@100000000 {
+               opp-100000000 {
                        opp-hz = /bits/ 64 <100000000>;
                        opp-microvolt = <937500>;
                };
@@ -123,19 +123,19 @@
        bus_g2d_266_opp_table: opp_table3 {
                compatible = "operating-points-v2";
 
-               opp@267000000 {
+               opp-267000000 {
                        opp-hz = /bits/ 64 <267000000>;
                };
-               opp@200000000 {
+               opp-200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                };
-               opp@160000000 {
+               opp-160000000 {
                        opp-hz = /bits/ 64 <160000000>;
                };
-               opp@134000000 {
+               opp-134000000 {
                        opp-hz = /bits/ 64 <134000000>;
                };
-               opp@100000000 {
+               opp-100000000 {
                        opp-hz = /bits/ 64 <100000000>;
                };
        };
@@ -143,13 +143,13 @@
        bus_gscl_opp_table: opp_table4 {
                compatible = "operating-points-v2";
 
-               opp@333000000 {
+               opp-333000000 {
                        opp-hz = /bits/ 64 <333000000>;
                };
-               opp@222000000 {
+               opp-222000000 {
                        opp-hz = /bits/ 64 <222000000>;
                };
-               opp@166500000 {
+               opp-166500000 {
                        opp-hz = /bits/ 64 <166500000>;
                };
        };
@@ -158,22 +158,22 @@
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp@400000000 {
+               opp-400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                };
-               opp@267000000 {
+               opp-267000000 {
                        opp-hz = /bits/ 64 <267000000>;
                };
-               opp@200000000 {
+               opp-200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                };
-               opp@160000000 {
+               opp-160000000 {
                        opp-hz = /bits/ 64 <160000000>;
                };
-               opp@134000000 {
+               opp-134000000 {
                        opp-hz = /bits/ 64 <134000000>;
                };
-               opp@100000000 {
+               opp-100000000 {
                        opp-hz = /bits/ 64 <100000000>;
                };
        };
@@ -181,16 +181,16 @@
        bus_noc2_opp_table: opp_table6 {
                compatible = "operating-points-v2";
 
-               opp@400000000 {
+               opp-400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                };
-               opp@200000000 {
+               opp-200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                };
-               opp@134000000 {
+               opp-134000000 {
                        opp-hz = /bits/ 64 <134000000>;
                };
-               opp@100000000 {
+               opp-100000000 {
                        opp-hz = /bits/ 64 <100000000>;
                };
        };

diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
index 16072c1..727f36a 100644 (file)
--- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
@@ -119,43 +119,43 @@
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp@400000000 {
+               opp-400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                        opp-microvolt = <900000>;
                };
-               opp@500000000 {
+               opp-500000000 {
                        opp-hz = /bits/ 64 <500000000>;
                        opp-microvolt = <925000>;
                };
-               opp@600000000 {
+               opp-600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <950000>;
                };
-               opp@700000000 {
+               opp-700000000 {
                        opp-hz = /bits/ 64 <700000000>;
                        opp-microvolt = <975000>;
                };
-               opp@800000000 {
+               opp-800000000 {
                        opp-hz = /bits/ 64 <800000000>;
                        opp-microvolt = <1000000>;
                };
-               opp@900000000 {
+               opp-900000000 {
                        opp-hz = /bits/ 64 <900000000>;
                        opp-microvolt = <1050000>;
                };
-               opp@1000000000 {
+               opp-1000000000 {
                        opp-hz = /bits/ 64 <1000000000>;
                        opp-microvolt = <1075000>;
                };
-               opp@1100000000 {
+               opp-1100000000 {
                        opp-hz = /bits/ 64 <1100000000>;
                        opp-microvolt = <1112500>;
                };
-               opp@1200000000 {
+               opp-1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <1112500>;
                };
-               opp@1300000000 {
+               opp-1300000000 {
                        opp-hz = /bits/ 64 <1300000000>;
                        opp-microvolt = <1150000>;
                };
@@ -165,63 +165,63 @@
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp@500000000 {
+               opp-500000000 {
                        opp-hz = /bits/ 64 <500000000>;
                        opp-microvolt = <900000>;
                };
-               opp@600000000 {
+               opp-600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <900000>;
                };
-               opp@700000000 {
+               opp-700000000 {
                        opp-hz = /bits/ 64 <700000000>;
                        opp-microvolt = <912500>;
                };
-               opp@800000000 {
+               opp-800000000 {
                        opp-hz = /bits/ 64 <800000000>;
                        opp-microvolt = <912500>;
                };
-               opp@900000000 {
+               opp-900000000 {
                        opp-hz = /bits/ 64 <900000000>;
                        opp-microvolt = <937500>;
                };
-               opp@1000000000 {
+               opp-1000000000 {
                        opp-hz = /bits/ 64 <1000000000>;
                        opp-microvolt = <975000>;
                };
-               opp@1100000000 {
+               opp-1100000000 {
                        opp-hz = /bits/ 64 <1100000000>;
                        opp-microvolt = <1012500>;
                };
-               opp@1200000000 {
+               opp-1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <1037500>;
                };
-               opp@1300000000 {
+               opp-1300000000 {
                        opp-hz = /bits/ 64 <1300000000>;
                        opp-microvolt = <1062500>;
                };
-               opp@1400000000 {
+               opp-1400000000 {
                        opp-hz = /bits/ 64 <1400000000>;
                        opp-microvolt = <1087500>;
                };
-               opp@1500000000 {
+               opp-1500000000 {
                        opp-hz = /bits/ 64 <1500000000>;
                        opp-microvolt = <1125000>;
                };
-               opp@1600000000 {
+               opp-1600000000 {
                        opp-hz = /bits/ 64 <1600000000>;
                        opp-microvolt = <1137500>;
                };
-               opp@1700000000 {
+               opp-1700000000 {
                        opp-hz = /bits/ 64 <1700000000>;
                        opp-microvolt = <1175000>;
                };
-               opp@1800000000 {
+               opp-1800000000 {
                        opp-hz = /bits/ 64 <1800000000>;
                        opp-microvolt = <1212500>;
                };
-               opp@1900000000 {
+               opp-1900000000 {
                        opp-hz = /bits/ 64 <1900000000>;
                        opp-microvolt = <1262500>;
                };