arm64: dts: rockchip: rk3399: Rename OPP nodes as opp@<opp-hz>
authorFinley Xiao <finley.xiao@rock-chips.com>
Sun, 25 Dec 2016 08:09:44 +0000 (16:09 +0800)
committerHuang, Tao <huangtao@rock-chips.com>
Mon, 26 Dec 2016 04:02:00 +0000 (12:02 +0800)
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.

Change-Id: I8c77646329e39390fb135d4d75d34893a8168876
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
arch/arm64/boot/dts/rockchip/rk3399-early-opp.dtsi
arch/arm64/boot/dts/rockchip/rk3399-opp.dtsi

index c6ba6e1a477bf73c7e731661fa29241bbb6c853e..ac848c3a00a9c109c74183da7dc61c9a3f5817ed 100644 (file)
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@408000000 {
                        opp-hz = /bits/ 64 <408000000>;
                        opp-microvolt = <900000>;
                        clock-latency-ns = <40000>;
                };
-               opp01 {
+               opp@600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <900000>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <900000>;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <900000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <950000>;
                };
-               opp05 {
+               opp@1416000000 {
                        opp-hz = /bits/ 64 <1416000000>;
                        opp-microvolt = <1050000>;
                };
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@408000000 {
                        opp-hz = /bits/ 64 <408000000>;
                        opp-microvolt = <900000>;
                        clock-latency-ns = <40000>;
                };
-               opp01 {
+               opp@600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <900000>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <900000>;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <950000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <100000>;
                };
        gpu_opp_table: opp-table2 {
                compatible = "operating-points-v2";
 
-               opp00 {
+               opp@200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                        opp-microvolt = <900000>;
                };
-               opp01 {
+               opp@297000000 {
                        opp-hz = /bits/ 64 <297000000>;
                        opp-microvolt = <900000>;
                };
-               opp02 {
+               opp@400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                        opp-microvolt = <900000>;
                };
index 5f7bea6600bd68b39dfe0732cbaf5d9aaefc89a2..d3a5171b121d1f4ca5e4e753c2d528151bc1caa4 100644 (file)
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@408000000 {
                        opp-hz = /bits/ 64 <408000000>;
                        opp-microvolt = <800000>;
                        clock-latency-ns = <40000>;
                };
-               opp01 {
+               opp@600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <800000>;
                        clock-latency-ns = <40000>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <850000>;
                        clock-latency-ns = <40000>;
                        opp-suspend;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <925000>;
                        clock-latency-ns = <40000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <1000000>;
                        clock-latency-ns = <40000>;
                };
-               opp05 {
+               opp@1416000000 {
                        opp-hz = /bits/ 64 <1416000000>;
                        opp-microvolt = <1125000>;
                        clock-latency-ns = <40000>;
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@408000000 {
                        opp-hz = /bits/ 64 <408000000>;
                        opp-microvolt = <800000>;
                        clock-latency-ns = <40000>;
                };
-               opp01 {
+               opp@600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <800000>;
                        clock-latency-ns = <40000>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <825000>;
                        clock-latency-ns = <40000>;
                        opp-suspend;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <875000>;
                        clock-latency-ns = <40000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <950000>;
                        clock-latency-ns = <40000>;
                };
-               opp05 {
+               opp@1416000000 {
                        opp-hz = /bits/ 64 <1416000000>;
                        opp-microvolt = <1025000>;
                        clock-latency-ns = <40000>;
                };
-               opp06 {
+               opp@1608000000 {
                        opp-hz = /bits/ 64 <1608000000>;
                        opp-microvolt = <1100000>;
                        clock-latency-ns = <40000>;
                };
-               opp07 {
+               opp@1800000000 {
                        opp-hz = /bits/ 64 <1800000000>;
                        opp-microvolt = <1200000>;
                        clock-latency-ns = <40000>;
        gpu_opp_table: opp-table2 {
                compatible = "operating-points-v2";
 
-               opp00 {
+               opp@200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                        opp-microvolt = <800000>;
                };
-               opp01 {
+               opp@297000000 {
                        opp-hz = /bits/ 64 <297000000>;
                        opp-microvolt = <800000>;
                };
-               opp02 {
+               opp@400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                        opp-microvolt = <825000>;
                };
-               opp03 {
+               opp@500000000 {
                        opp-hz = /bits/ 64 <500000000>;
                        opp-microvolt = <875000>;
                };
-               opp04 {
+               opp@594000000 {
                        opp-hz = /bits/ 64 <594000000>;
                        opp-microvolt = <925000>;
                };
-               opp05 {
+               opp@800000000 {
                        opp-hz = /bits/ 64 <800000000>;
                        opp-microvolt = <1100000>;
                };
        dmc_opp_table: opp-table3 {
                compatible = "operating-points-v2";
 
-               opp00 {
+               opp@200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                        opp-microvolt = <825000>;
                };
-               opp01 {
+               opp@297000000 {
                        opp-hz = /bits/ 64 <297000000>;
                        opp-microvolt = <850000>;
                };
-               opp02 {
+               opp@400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                        opp-microvolt = <850000>;
                };
-               opp03 {
+               opp@594000000 {
                        opp-hz = /bits/ 64 <594000000>;
                        opp-microvolt = <900000>;
                };
-               opp04 {
+               opp@800000000 {
                        opp-hz = /bits/ 64 <800000000>;
                        opp-microvolt = <900000>;
                };