tjh.dev / kernel
Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux
fork atom

ARM: dts: Use - instead of @ for DT OPP entries for TI SoCs

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>
Acked-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>

authored by

Viresh Kumar and committed by
Tony Lindgren b9cb2ba7 49b2c130

+28 -28
+10 -10
Documentation/devicetree/bindings/cpufreq/ti-cpufreq.txt
··· 63 63 * because they can not be enabled simultaneously on a 64 64 * single SoC. 65 65 */ 66 - opp50@300000000 { 66 + opp50-300000000 { 67 67 opp-hz = /bits/ 64 <300000000>; 68 68 opp-microvolt = <950000 931000 969000>; 69 69 opp-supported-hw = <0x06 0x0010>; 70 70 opp-suspend; 71 71 }; 72 72 73 - opp100@275000000 { 73 + opp100-275000000 { 74 74 opp-hz = /bits/ 64 <275000000>; 75 75 opp-microvolt = <1100000 1078000 1122000>; 76 76 opp-supported-hw = <0x01 0x00FF>; 77 77 opp-suspend; 78 78 }; 79 79 80 - opp100@300000000 { 80 + opp100-300000000 { 81 81 opp-hz = /bits/ 64 <300000000>; 82 82 opp-microvolt = <1100000 1078000 1122000>; 83 83 opp-supported-hw = <0x06 0x0020>; 84 84 opp-suspend; 85 85 }; 86 86 87 - opp100@500000000 { 87 + opp100-500000000 { 88 88 opp-hz = /bits/ 64 <500000000>; 89 89 opp-microvolt = <1100000 1078000 1122000>; 90 90 opp-supported-hw = <0x01 0xFFFF>; 91 91 }; 92 92 93 - opp100@600000000 { 93 + opp100-600000000 { 94 94 opp-hz = /bits/ 64 <600000000>; 95 95 opp-microvolt = <1100000 1078000 1122000>; 96 96 opp-supported-hw = <0x06 0x0040>; 97 97 }; 98 98 99 - opp120@600000000 { 99 + opp120-600000000 { 100 100 opp-hz = /bits/ 64 <600000000>; 101 101 opp-microvolt = <1200000 1176000 1224000>; 102 102 opp-supported-hw = <0x01 0xFFFF>; 103 103 }; 104 104 105 - opp120@720000000 { 105 + opp120-720000000 { 106 106 opp-hz = /bits/ 64 <720000000>; 107 107 opp-microvolt = <1200000 1176000 1224000>; 108 108 opp-supported-hw = <0x06 0x0080>; 109 109 }; 110 110 111 - oppturbo@720000000 { 111 + oppturbo-720000000 { 112 112 opp-hz = /bits/ 64 <720000000>; 113 113 opp-microvolt = <1260000 1234800 1285200>; 114 114 opp-supported-hw = <0x01 0xFFFF>; 115 115 }; 116 116 117 - oppturbo@800000000 { 117 + oppturbo-800000000 { 118 118 opp-hz = /bits/ 64 <800000000>; 119 119 opp-microvolt = <1260000 1234800 1285200>; 120 120 opp-supported-hw = <0x06 0x0100>; 121 121 }; 122 122 123 - oppnitro@1000000000 { 123 + oppnitro-1000000000 { 124 124 opp-hz = /bits/ 64 <1000000000>; 125 125 opp-microvolt = <1325000 1298500 1351500>; 126 126 opp-supported-hw = <0x04 0x0200>;
+1 -1
arch/arm/boot/dts/am335x-boneblack.dts
··· 22 22 * BeagleBone Blacks have PG 2.0 silicon which is guaranteed 23 23 * to support 1GHz OPP so enable it for PG 2.0 on this board. 24 24 */ 25 - oppnitro@1000000000 { 25 + oppnitro-1000000000 { 26 26 opp-supported-hw = <0x06 0x0100>; 27 27 }; 28 28 };
+10 -10
arch/arm/boot/dts/am33xx.dtsi
··· 64 64 * because the can not be enabled simultaneously on a 65 65 * single SoC. 66 66 */ 67 - opp50@300000000 { 67 + opp50-300000000 { 68 68 opp-hz = /bits/ 64 <300000000>; 69 69 opp-microvolt = <950000 931000 969000>; 70 70 opp-supported-hw = <0x06 0x0010>; 71 71 opp-suspend; 72 72 }; 73 73 74 - opp100@275000000 { 74 + opp100-275000000 { 75 75 opp-hz = /bits/ 64 <275000000>; 76 76 opp-microvolt = <1100000 1078000 1122000>; 77 77 opp-supported-hw = <0x01 0x00FF>; 78 78 opp-suspend; 79 79 }; 80 80 81 - opp100@300000000 { 81 + opp100-300000000 { 82 82 opp-hz = /bits/ 64 <300000000>; 83 83 opp-microvolt = <1100000 1078000 1122000>; 84 84 opp-supported-hw = <0x06 0x0020>; 85 85 opp-suspend; 86 86 }; 87 87 88 - opp100@500000000 { 88 + opp100-500000000 { 89 89 opp-hz = /bits/ 64 <500000000>; 90 90 opp-microvolt = <1100000 1078000 1122000>; 91 91 opp-supported-hw = <0x01 0xFFFF>; 92 92 }; 93 93 94 - opp100@600000000 { 94 + opp100-600000000 { 95 95 opp-hz = /bits/ 64 <600000000>; 96 96 opp-microvolt = <1100000 1078000 1122000>; 97 97 opp-supported-hw = <0x06 0x0040>; 98 98 }; 99 99 100 - opp120@600000000 { 100 + opp120-600000000 { 101 101 opp-hz = /bits/ 64 <600000000>; 102 102 opp-microvolt = <1200000 1176000 1224000>; 103 103 opp-supported-hw = <0x01 0xFFFF>; 104 104 }; 105 105 106 - opp120@720000000 { 106 + opp120-720000000 { 107 107 opp-hz = /bits/ 64 <720000000>; 108 108 opp-microvolt = <1200000 1176000 1224000>; 109 109 opp-supported-hw = <0x06 0x0080>; 110 110 }; 111 111 112 - oppturbo@720000000 { 112 + oppturbo-720000000 { 113 113 opp-hz = /bits/ 64 <720000000>; 114 114 opp-microvolt = <1260000 1234800 1285200>; 115 115 opp-supported-hw = <0x01 0xFFFF>; 116 116 }; 117 117 118 - oppturbo@800000000 { 118 + oppturbo-800000000 { 119 119 opp-hz = /bits/ 64 <800000000>; 120 120 opp-microvolt = <1260000 1234800 1285200>; 121 121 opp-supported-hw = <0x06 0x0100>; 122 122 }; 123 123 124 - oppnitro@1000000000 { 124 + oppnitro-1000000000 { 125 125 opp-hz = /bits/ 64 <1000000000>; 126 126 opp-microvolt = <1325000 1298500 1351500>; 127 127 opp-supported-hw = <0x04 0x0200>;
+5 -5
arch/arm/boot/dts/am4372.dtsi
··· 59 59 compatible = "operating-points-v2-ti-cpu"; 60 60 syscon = <&scm_conf>; 61 61 62 - opp50@300000000 { 62 + opp50-300000000 { 63 63 opp-hz = /bits/ 64 <300000000>; 64 64 opp-microvolt = <950000 931000 969000>; 65 65 opp-supported-hw = <0xFF 0x01>; 66 66 opp-suspend; 67 67 }; 68 68 69 - opp100@600000000 { 69 + opp100-600000000 { 70 70 opp-hz = /bits/ 64 <600000000>; 71 71 opp-microvolt = <1100000 1078000 1122000>; 72 72 opp-supported-hw = <0xFF 0x04>; 73 73 }; 74 74 75 - opp120@720000000 { 75 + opp120-720000000 { 76 76 opp-hz = /bits/ 64 <720000000>; 77 77 opp-microvolt = <1200000 1176000 1224000>; 78 78 opp-supported-hw = <0xFF 0x08>; 79 79 }; 80 80 81 - oppturbo@800000000 { 81 + oppturbo-800000000 { 82 82 opp-hz = /bits/ 64 <800000000>; 83 83 opp-microvolt = <1260000 1234800 1285200>; 84 84 opp-supported-hw = <0xFF 0x10>; 85 85 }; 86 86 87 - oppnitro@1000000000 { 87 + oppnitro-1000000000 { 88 88 opp-hz = /bits/ 64 <1000000000>; 89 89 opp-microvolt = <1325000 1298500 1351500>; 90 90 opp-supported-hw = <0xFF 0x20>;
+2 -2
arch/arm/boot/dts/dra7.dtsi
··· 99 99 compatible = "operating-points-v2-ti-cpu"; 100 100 syscon = <&scm_wkup>; 101 101 102 - opp_nom@1000000000 { 102 + opp_nom-1000000000 { 103 103 opp-hz = /bits/ 64 <1000000000>; 104 104 opp-microvolt = <1060000 850000 1150000>; 105 105 opp-supported-hw = <0xFF 0x01>; 106 106 opp-suspend; 107 107 }; 108 108 109 - opp_od@1176000000 { 109 + opp_od-1176000000 { 110 110 opp-hz = /bits/ 64 <1176000000>; 111 111 opp-microvolt = <1160000 885000 1160000>; 112 112 opp-supported-hw = <0xFF 0x02>;