ARM64: dts: rockchip: Rename OPP nodes as opp@<opp-hz>
authorFinley Xiao <finley.xiao@rock-chips.com>
Tue, 5 Jul 2016 02:17:25 +0000 (10:17 +0800)
committerFinley Xiao <finley.xiao@rock-chips.com>
Tue, 5 Jul 2016 02:33:20 +0000 (10:33 +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: Icefba93f7a95752e344b5a092a83931bf4d1e682
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
arch/arm64/boot/dts/rockchip/rk3366.dtsi
arch/arm64/boot/dts/rockchip/rk3368.dtsi
arch/arm64/boot/dts/rockchip/rk3399-evb-rev2.dtsi
arch/arm64/boot/dts/rockchip/rk3399.dtsi

index 783caac064dbdaf1f3a9818b0263fc725a118e77..3ae404c56cd4c17d47624c0d809e1eb94e50b45c 100644 (file)
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@408000000 {
                        opp-hz = /bits/ 64 <408000000>;
                        opp-microvolt = <950000>;
                        clock-latency-ns = <40000>;
                        opp-suspend;
                };
-               opp01 {
+               opp@600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <950000>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <1000000>;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <1075000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <1175000>;
                };
-               opp05 {
+               opp@1296000000 {
                        opp-hz = /bits/ 64 <1296000000>;
                        opp-microvolt = <1250000>;
                };
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@96000000 {
                        opp-hz = /bits/ 64 <96000000>;
                        opp-microvolt = <1100000>;
                };
-               opp01 {
+               opp@192000000 {
                        opp-hz = /bits/ 64 <192000000>;
                        opp-microvolt = <1100000>;
                };
-               opp02 {
+               opp@288000000 {
                        opp-hz = /bits/ 64 <288000000>;
                        opp-microvolt = <1100000>;
                };
-               opp03 {
+               opp@375000000 {
                        opp-hz = /bits/ 64 <375000000>;
                        opp-microvolt = <1125000>;
                };
-               opp04 {
+               opp@480000000 {
                        opp-hz = /bits/ 64 <480000000>;
                        opp-microvolt = <1200000>;
                };
index 058fd911288a9c570c000dde11da4dc12468ef2e..132765e7129db3a2ef274e3957c40fafca8af677 100644 (file)
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@408000000 {
                        opp-hz = /bits/ 64 <408000000>;
                        opp-microvolt = <1200000>;
                        clock-latency-ns = <40000>;
                        opp-suspend;
                };
-               opp01 {
+               opp@600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <1200000>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <1200000>;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <1200000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <1200000>;
                };
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@408000000 {
                        opp-hz = /bits/ 64 <408000000>;
                        opp-microvolt = <1200000>;
                        clock-latency-ns = <40000>;
                        opp-suspend;
                };
-               opp01 {
+               opp@600000000 {
                        opp-hz = /bits/ 64 <600000000>;
                        opp-microvolt = <1200000>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <1200000>;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <1200000>;
                };
index 57b16076efa74c0e6551b558034ab252a341a88b..d58b8406cc7f2d107e3d3f553caf9755251ff737 100644 (file)
 };
 
 &cluster0_opp {
-       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>;
        };
-       opp02 {
+       opp@816000000 {
                opp-hz = /bits/ 64 <816000000>;
                opp-microvolt = <800000>;
        };
-       opp03 {
+       opp@1008000000 {
                opp-hz = /bits/ 64 <1008000000>;
                opp-microvolt = <875000>;
        };
-       opp04 {
+       opp@1200000000 {
                opp-hz = /bits/ 64 <1200000000>;
                opp-microvolt = <925000>;
        };
-       opp05 {
+       opp@1416000000 {
                opp-hz = /bits/ 64 <1416000000>;
                opp-microvolt = <1025000>;
        };
-       opp06 {
+       opp@1512000000 {
                opp-hz = /bits/ 64 <1512000000>;
                opp-microvolt = <1075000>;
        };
 };
 
 &cluster1_opp {
-       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>;
        };
-       opp02 {
+       opp@816000000 {
                opp-hz = /bits/ 64 <816000000>;
                opp-microvolt = <800000>;
        };
-       opp03 {
+       opp@1008000000 {
                opp-hz = /bits/ 64 <1008000000>;
                opp-microvolt = <850000>;
        };
-       opp04 {
+       opp@1200000000 {
                opp-hz = /bits/ 64 <1200000000>;
                opp-microvolt = <925000>;
        };
-       opp05 {
+       opp@1416000000 {
                opp-hz = /bits/ 64 <1416000000>;
                opp-microvolt = <1025000>;
        };
-       opp06 {
+       opp@1608000000 {
                opp-hz = /bits/ 64 <1608000000>;
                opp-microvolt = <1125000>;
        };
-//     opp07 {
-//             opp-hz = /bits/ 64 <1800000000>;
-//             opp-microvolt = <1200000>;
-//     };
+       opp@1800000000 {
+               opp-hz = /bits/ 64 <1800000000>;
+               opp-microvolt = <1200000>;
+               status = "disabeld";
+       };
 };
 
 &CPU_COST_A72 {
index ffa7197611b1347a83a4c9d9ea6c8afa5b047e51..019e3c5b84170aeff4ad6da08fb18a0ebd5ae4c5 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>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <800000>;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <875000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <925000>;
                };
-               opp05 {
+               opp@1416000000 {
                        opp-hz = /bits/ 64 <1416000000>;
                        opp-microvolt = <1025000>;
                };
                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>;
                };
-               opp02 {
+               opp@816000000 {
                        opp-hz = /bits/ 64 <816000000>;
                        opp-microvolt = <800000>;
                };
-               opp03 {
+               opp@1008000000 {
                        opp-hz = /bits/ 64 <1008000000>;
                        opp-microvolt = <850000>;
                };
-               opp04 {
+               opp@1200000000 {
                        opp-hz = /bits/ 64 <1200000000>;
                        opp-microvolt = <925000>;
                };
                compatible = "operating-points-v2";
                opp-shared;
 
-               opp00 {
+               opp@200000000 {
                        opp-hz = /bits/ 64 <200000000>;
                        opp-microvolt = <900000>;
                };
-               opp01 {
+               opp@300000000 {
                        opp-hz = /bits/ 64 <300000000>;
                        opp-microvolt = <900000>;
                };
-               opp02 {
+               opp@400000000 {
                        opp-hz = /bits/ 64 <400000000>;
                        opp-microvolt = <900000>;
                };