| Message ID | 20250403-dt-cpu-schema-v1-0-076be7171a85@kernel.org |
|---|---|
| Headers | show |
| Series | Arm cpu schema clean-ups | expand |
On 04/04/2025 04:59, Rob Herring (Arm) wrote: > The "clock-latency" property is part of the deprecated opp-v1 binding > and is redundant if the opp-v2 table has equal or larger values in any > "clock-latency-ns". Add any missing "clock-latency-ns" properties and > remove "clock-latency". > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org> > --- > arch/arm64/boot/dts/amlogic/meson-g12a-fbx8am.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-g12a.dtsi | 1 + > arch/arm64/boot/dts/amlogic/meson-g12b-a311d-libretech-cc.dts | 6 ------ > arch/arm64/boot/dts/amlogic/meson-g12b-a311d.dtsi | 2 ++ > arch/arm64/boot/dts/amlogic/meson-g12b-bananapi-cm4.dtsi | 6 ------ > arch/arm64/boot/dts/amlogic/meson-g12b-bananapi.dtsi | 6 ------ > arch/arm64/boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi | 6 ------ > arch/arm64/boot/dts/amlogic/meson-g12b-odroid-go-ultra.dts | 6 ------ > arch/arm64/boot/dts/amlogic/meson-g12b-odroid.dtsi | 6 ------ > arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts | 6 ------ > arch/arm64/boot/dts/amlogic/meson-g12b-s922x.dtsi | 2 ++ > arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi | 6 ------ > arch/arm64/boot/dts/amlogic/meson-sm1-ac2xx.dtsi | 4 ---- > arch/arm64/boot/dts/amlogic/meson-sm1-bananapi.dtsi | 4 ---- > arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi | 4 ---- > arch/arm64/boot/dts/amlogic/meson-sm1-s905d3-libretech-cc.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts | 4 ---- > arch/arm64/boot/dts/amlogic/meson-sm1.dtsi | 1 + > 23 files changed, 6 insertions(+), 92 deletions(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-fbx8am.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-fbx8am.dts > index 9aa36f17ffa2..d0a3b4b9229c 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-fbx8am.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-fbx8am.dts > @@ -267,28 +267,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > ðmac { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dts > index 952b8d02e5c2..4353485c6f26 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dts > @@ -220,28 +220,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cvbs_vdac_port { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts > index 52fbc5103e45..f39fcabc763f 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts > @@ -314,28 +314,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cvbs_vdac_port { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts > index 5407049d2647..b5bf8ecc91e6 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts > @@ -407,28 +407,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &clkc_audio { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts > index 01da83658ae3..5ab460a3e637 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts > @@ -263,28 +263,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cvbs_vdac_port { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi > index 543e70669df5..deee61dbe074 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi > @@ -62,6 +62,7 @@ cpu_opp_table: opp-table { > opp-1000000000 { > opp-hz = /bits/ 64 <1000000000>; > opp-microvolt = <731000>; > + clock-latency-ns = <50000>; > }; > > opp-1200000000 { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-a311d-libretech-cc.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-a311d-libretech-cc.dts > index adedc1340c78..415248931ab1 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-a311d-libretech-cc.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-a311d-libretech-cc.dts > @@ -76,42 +76,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &pwm_ab { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-a311d.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b-a311d.dtsi > index 8e9ad1e51d66..8ecb5bd125c1 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-a311d.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-a311d.dtsi > @@ -14,6 +14,7 @@ cpu_opp_table_0: opp-table-0 { > opp-1000000000 { > opp-hz = /bits/ 64 <1000000000>; > opp-microvolt = <761000>; > + clock-latency-ns = <50000>; > }; > > opp-1200000000 { > @@ -54,6 +55,7 @@ cpub_opp_table_1: opp-table-1 { > opp-1000000000 { > opp-hz = /bits/ 64 <1000000000>; > opp-microvolt = <731000>; > + clock-latency-ns = <50000>; > }; > > opp-1200000000 { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi-cm4.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi-cm4.dtsi > index 92e8b26ecccc..39011b645128 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi-cm4.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi-cm4.dtsi > @@ -155,42 +155,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &ext_mdio { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi.dtsi > index 54663c55a20e..1b08303c4282 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-bananapi.dtsi > @@ -263,42 +263,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > ðmac { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi > index 48650bad230d..fc737499f207 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi > @@ -51,42 +51,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &pwm_ab { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-go-ultra.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-go-ultra.dts > index e21831dfceee..d5938a4a6da3 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-go-ultra.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-go-ultra.dts > @@ -281,42 +281,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > /* RK817 only supports 12.5mV steps, round up the values */ > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid.dtsi > index 7e8964bacfce..3298d59833b6 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid.dtsi > @@ -227,42 +227,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu_thermal { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts > index fc05ecf90714..1e5c6f984945 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts > @@ -259,42 +259,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu_thermal { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-s922x.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b-s922x.dtsi > index 44c23c984034..19cad93a6889 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-s922x.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-s922x.dtsi > @@ -14,6 +14,7 @@ cpu_opp_table_0: opp-table-0 { > opp-1000000000 { > opp-hz = /bits/ 64 <1000000000>; > opp-microvolt = <731000>; > + clock-latency-ns = <50000>; > }; > > opp-1200000000 { > @@ -59,6 +60,7 @@ cpub_opp_table_1: opp-table-1 { > opp-1000000000 { > opp-hz = /bits/ 64 <1000000000>; > opp-microvolt = <771000>; > + clock-latency-ns = <50000>; > }; > > opp-1200000000 { > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi > index a7a0fc264cdc..9b6d780eada7 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi > @@ -213,42 +213,36 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table_0>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu100 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu101 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu102 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cpu103 { > cpu-supply = <&vddcpu_a>; > operating-points-v2 = <&cpub_opp_table_1>; > clocks = <&clkc CLKID_CPUB_CLK>; > - clock-latency = <50000>; > }; > > &cvbs_vdac_port { > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-ac2xx.dtsi b/arch/arm64/boot/dts/amlogic/meson-sm1-ac2xx.dtsi > index a3463149db3d..9be3084b090d 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-ac2xx.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-ac2xx.dtsi > @@ -147,28 +147,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU1_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU2_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU3_CLK>; > - clock-latency = <50000>; > }; > > &cvbs_vdac_port { > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi.dtsi b/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi.dtsi > index 40db95f64636..538b35036954 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi.dtsi > @@ -185,28 +185,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU1_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU2_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU3_CLK>; > - clock-latency = <50000>; > }; > > &ext_mdio { > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts > index 5d75ad3f3e46..a3d9b66b6878 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts > @@ -51,28 +51,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU1_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU2_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU3_CLK>; > - clock-latency = <50000>; > }; > > &pwm_AO_cd { > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi b/arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi > index ad8d07883760..c4524eb4f099 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi > @@ -250,28 +250,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU1_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU2_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU3_CLK>; > - clock-latency = <50000>; > }; > > &ext_mdio { > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-s905d3-libretech-cc.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-s905d3-libretech-cc.dts > index 537370db360f..5daadfb170b4 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-s905d3-libretech-cc.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-s905d3-libretech-cc.dts > @@ -64,26 +64,22 @@ &cpu0 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU1_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU2_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu_b>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU3_CLK>; > - clock-latency = <50000>; > }; > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts > index 37d7f64b6d5d..024d2eb8e6ee 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts > @@ -359,28 +359,24 @@ &cpu0 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU_CLK>; > - clock-latency = <50000>; > }; > > &cpu1 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU1_CLK>; > - clock-latency = <50000>; > }; > > &cpu2 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU2_CLK>; > - clock-latency = <50000>; > }; > > &cpu3 { > cpu-supply = <&vddcpu>; > operating-points-v2 = <&cpu_opp_table>; > clocks = <&clkc CLKID_CPU3_CLK>; > - clock-latency = <50000>; > }; > > ðmac { > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi b/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi > index 97e4b52066dc..966ebb19cc55 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi > @@ -100,6 +100,7 @@ cpu_opp_table: opp-table { > opp-1000000000 { > opp-hz = /bits/ 64 <1000000000>; > opp-microvolt = <770000>; > + clock-latency-ns = <50000>; > }; > > opp-1200000000 { > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
On Fri, Apr 4, 2025 at 5:31 AM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > On Fri, 4 Apr 2025 at 05:02, Rob Herring (Arm) <robh@kernel.org> wrote: > > > > "rpmhpd" is not documented nor used anywhere. As the enable-method is > > "psci" use "psci" for the power-domain name. > > > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org> > > --- > > arch/arm/boot/dts/qcom/qcom-sdx55.dtsi | 2 +- > > arch/arm/boot/dts/qcom/qcom-sdx65.dtsi | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm/boot/dts/qcom/qcom-sdx55.dtsi b/arch/arm/boot/dts/qcom/qcom-sdx55.dtsi > > index 39530eb580ea..64d9858b4248 100644 > > --- a/arch/arm/boot/dts/qcom/qcom-sdx55.dtsi > > +++ b/arch/arm/boot/dts/qcom/qcom-sdx55.dtsi > > @@ -57,7 +57,7 @@ cpu0: cpu@0 { > > enable-method = "psci"; > > clocks = <&apcs>; > > power-domains = <&rpmhpd SDX55_CX>; > > - power-domain-names = "rpmhpd"; > > + power-domain-names = "psci"; > > As I understand it, this isn't for cpu-power-mgmt but for > cpu-performance-scaling. > > I have been thinking of adding a common power-domain-name for this, > but never reached to do it. I think the last one we added was the > Airoha SoC [1] which uses "perf", which seems to be the most common > one. Still I don't see that being documented. It's all a mess and abuse of the power-domains binding... "perf" is documented. I can add "rpmhpd" I suppose, but I don't see anything using the name. Or I can remove it. Rob
On 4/4/25 4:59 AM, Rob Herring (Arm) wrote: > The correct property name is 'qcom,freq-domain', not > 'qcom,freq-domains'. > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org> > --- Thanks for spotting this! Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Konrad
On 4/4/25 10:30 PM, Konrad Dybcio wrote: > On 4/4/25 4:59 AM, Rob Herring (Arm) wrote: >> The "spin-table" enable-method requires "cpu-release-addr" property, >> so add a dummy entry. It is assumed the bootloader will fill in the >> correct values. >> >> Signed-off-by: Rob Herring (Arm) <robh@kernel.org> >> --- > > This looks good to me without knowing any better about the specifics > of this device.. > > +Alexander - does the bootloader you use take care of this? Otherwise > we can just do what Sony devices do and stop on removing the psci node I failed to add Alexander to Cc, second time's the charm.. Konrad
On 4/4/25 4:59 AM, Rob Herring (Arm) wrote: > The "clock-latency" property is part of the deprecated opp-v1 binding > and is redundant if the opp-v2 table has equal or larger values in any > "clock-latency-ns". The OPP table has values of 256000, so it can be > removed. > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Konrad
> On 4/4/25 10:30 PM, Konrad Dybcio wrote: > > On 4/4/25 4:59 AM, Rob Herring (Arm) wrote: > >> The "spin-table" enable-method requires "cpu-release-addr" property, > >> so add a dummy entry. It is assumed the bootloader will fill in the > >> correct values. > >> > >> Signed-off-by: Rob Herring (Arm) <robh@kernel.org> > >> --- > > > > This looks good to me without knowing any better about the specifics > > of this device.. > > > > +Alexander - does the bootloader you use take care of this? Otherwise > > we can just do what Sony devices do and stop on removing the psci node I currently can't test this, but the bootloader (lk2nd) will set it. Alexander
From: Conor Dooley <conor.dooley@microchip.com> On Thu, 03 Apr 2025 21:59:21 -0500, Rob Herring (Arm) wrote: > The Arm cpu.yaml schema fails to restrict allowed properties in 'cpu' > nodes. The result, not surprisely, is a number of additional properties > and errors in .dts files. This series resolves those issues. > > There's still more properties in arm32 DTS files which I have not > documented. Mostly yet more supply names and "fsl,soc-operating-points". > What's a few more warnings on the 10000s of warnings... > > [...] Applied to microchip-dt64, thanks! [04/19] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies https://git.kernel.org/at91/c/ec32344d2a3b Thanks, Conor.
On 03-04-25, 21:59, Rob Herring (Arm) wrote: > .../bindings/cpufreq/cpufreq-mediatek.txt | 250 --------------------- > Documentation/devicetree/bindings/opp/opp-v1.yaml | 18 +- Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
The Arm cpu.yaml schema fails to restrict allowed properties in 'cpu' nodes. The result, not surprisely, is a number of additional properties and errors in .dts files. This series resolves those issues. There's still more properties in arm32 DTS files which I have not documented. Mostly yet more supply names and "fsl,soc-operating-points". What's a few more warnings on the 10000s of warnings... The .dts files can be taken by the respective SoC maintainers. I will take the binding changes. Signed-off-by: "Rob Herring (Arm)" <robh@kernel.org> --- Rob Herring (Arm) (19): arm64: dts: allwinner: h5/h6: Drop spurious 'clock-latency-ns' properties arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names arm64: dts: morello: Fix-up cache nodes arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies arm64: dts: qcom: qdu1000: Fix qcom,freq-domain arm64: dts: qcom: msm8939: Fix CPU node "enable-method" property dependencies arm64: dts: qcom: msm8992-lg-h815: Fix CPU node "enable-method" property dependencies arm: dts: qcom: msm8916: Move "qcom,acc" and "qcom,saw" to 32-bit .dtsi arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names arm/arm64: dts: imx: Drop redundant CPU "clock-latency" arm: dts: qcom: ipq4019: Drop redundant CPU "clock-latency" arm: dts: rockchip: Drop redundant CPU "clock-latency" arm64: dts: amlogic: Drop redundant CPU "clock-latency" dt-bindings: arm/cpus: Add schemas for "enable-method" dependencies dt-bindings: arm/cpus: Re-wrap 'description' entries dt-bindings: Reference opp-v1 schema in CPU schemas dt-bindings: arm/cpus: Add missing properties dt-bindings: arm/cpus: Add power-domains constraints dt-bindings: cpufreq: Drop redundant Mediatek binding Documentation/devicetree/bindings/arm/cpus.yaml | 220 ++++++++++-------- .../bindings/cpufreq/cpufreq-mediatek.txt | 250 --------------------- Documentation/devicetree/bindings/mips/cpus.yaml | 3 +- Documentation/devicetree/bindings/opp/opp-v1.yaml | 18 +- arch/arm/boot/dts/nxp/imx/imx7s.dtsi | 1 - arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi | 4 - arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi | 8 + arch/arm/boot/dts/qcom/qcom-sdx55.dtsi | 2 +- arch/arm/boot/dts/qcom/qcom-sdx65.dtsi | 2 +- arch/arm/boot/dts/rockchip/rk3128.dtsi | 8 +- arch/arm/boot/dts/rockchip/rk3188.dtsi | 1 - arch/arm/boot/dts/rockchip/rk322x.dtsi | 1 - arch/arm/boot/dts/rockchip/rk3288.dtsi | 5 +- arch/arm/boot/dts/rockchip/rv1108.dtsi | 1 - arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi | 4 - arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 4 - arch/arm64/boot/dts/amlogic/meson-g12a-fbx8am.dts | 4 - .../boot/dts/amlogic/meson-g12a-radxa-zero.dts | 4 - arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts | 4 - arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts | 4 - arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts | 4 - arch/arm64/boot/dts/amlogic/meson-g12a.dtsi | 1 + .../dts/amlogic/meson-g12b-a311d-libretech-cc.dts | 6 - arch/arm64/boot/dts/amlogic/meson-g12b-a311d.dtsi | 2 + .../boot/dts/amlogic/meson-g12b-bananapi-cm4.dtsi | 6 - .../boot/dts/amlogic/meson-g12b-bananapi.dtsi | 6 - .../boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi | 6 - .../dts/amlogic/meson-g12b-odroid-go-ultra.dts | 6 - arch/arm64/boot/dts/amlogic/meson-g12b-odroid.dtsi | 6 - .../boot/dts/amlogic/meson-g12b-radxa-zero2.dts | 6 - arch/arm64/boot/dts/amlogic/meson-g12b-s922x.dtsi | 2 + arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi | 6 - arch/arm64/boot/dts/amlogic/meson-sm1-ac2xx.dtsi | 4 - .../arm64/boot/dts/amlogic/meson-sm1-bananapi.dtsi | 4 - .../boot/dts/amlogic/meson-sm1-khadas-vim3l.dts | 4 - arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi | 4 - .../dts/amlogic/meson-sm1-s905d3-libretech-cc.dts | 4 - arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts | 4 - arch/arm64/boot/dts/amlogic/meson-sm1.dtsi | 1 + arch/arm64/boot/dts/arm/morello.dtsi | 22 +- arch/arm64/boot/dts/broadcom/bcm2712.dtsi | 8 +- arch/arm64/boot/dts/freescale/imx8mm.dtsi | 4 - arch/arm64/boot/dts/freescale/imx8mn.dtsi | 4 - arch/arm64/boot/dts/freescale/imx8mp.dtsi | 4 - arch/arm64/boot/dts/freescale/imx8mq.dtsi | 4 - .../boot/dts/microchip/sparx5_pcb_common.dtsi | 2 + arch/arm64/boot/dts/qcom/msm8916.dtsi | 8 - arch/arm64/boot/dts/qcom/msm8939.dtsi | 24 +- arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts | 6 + arch/arm64/boot/dts/qcom/qdu1000.dtsi | 8 +- 50 files changed, 210 insertions(+), 514 deletions(-) --- base-commit: a2cc6ff5ec8f91bc463fd3b0c26b61166a07eb11 change-id: 20250403-dt-cpu-schema-48e66c7f6a90 Best regards,