| 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, 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. > operating-points-v2 = <&cpu_opp_table>; > }; > }; > diff --git a/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi b/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi > index 6b23ee676c9e..bfd04e53c5a8 100644 > --- a/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi > +++ b/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi > @@ -58,7 +58,7 @@ cpu0: cpu@0 { > enable-method = "psci"; > clocks = <&apcs>; > power-domains = <&rpmhpd SDX65_CX_AO>; > - power-domain-names = "rpmhpd"; > + power-domain-names = "psci"; Ditto. > operating-points-v2 = <&cpu_opp_table>; > }; > }; > > -- > 2.47.2 > > Kind regards Uffe [1] drivers/cpufreq/airoha-cpufreq.c Documentation/devicetree/bindings/cpufreq/airoha,en7581-cpufreq.yaml drivers/pmdomain/mediatek/airoha-cpu-pmdomain.c
Il 04/04/25 04:59, Rob Herring (Arm) ha scritto: > The Arm CPU schema is missing a number of properties already in use. > This has gone unnoticed as extra properties have not been restricted. > Add a missing reference to cpu.yaml, and add all the missing properties. > > As "clock-latency" and "voltage-tolerance" are related to opp-v1, add > those properties to the opp-v1.yaml schema. > > With this, other properties can be prevented from creeping in with > 'unevaluatedProperties: false'. > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org> > --- > Documentation/devicetree/bindings/arm/cpus.yaml | 46 ++++++++++++++++++++++- > Documentation/devicetree/bindings/opp/opp-v1.yaml | 16 ++++++++ > 2 files changed, 61 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml > index 3d2b6286efb8..6f74ebfd38df 100644 > --- a/Documentation/devicetree/bindings/arm/cpus.yaml > +++ b/Documentation/devicetree/bindings/arm/cpus.yaml > @@ -299,6 +299,16 @@ properties: > > where voltage is in V, frequency is in MHz. > > + interconnects: > + minItems: 1 > + maxItems: 2 > + > + nvmem-cells: > + maxItems: 1 > + > + nvmem-cell-names: > + const: speed_grade > + > performance-domains: > maxItems: 1 > > @@ -317,6 +327,31 @@ properties: > corresponding to the index of an SCMI performance domain provider, must be > "perf". > > + resets: > + maxItems: 1 > + > + arm-supply: > + deprecated: true > + description: Use 'cpu-supply' instead > + > + cpu0-supply: > + deprecated: true > + description: Use 'cpu-supply' instead > + > + mem-supply: true > + > + proc-supply: > + deprecated: true > + description: Use 'cpu-supply' instead > + > + sram-supply: > + deprecated: true > + description: Use 'mem-supply' instead > + > + mediatek,cci: > + $ref: /schemas/types.yaml#/definitions/phandle > + description: Link to Mediatek Cache Coherent Interconnect s/Mediatek/MediaTek/g please :-) Anyway: Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Il 04/04/25 04:59, Rob Herring (Arm) ha scritto: > The Mediatek CPUFreq binding document just describes properties from > the CPU node which the driver uses. This is redundant as all the > properties are described in the arm/cpus.yaml schema. > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
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: > "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> > --- "psci" is what we want here, but these platforms require some more massaging.. These SoCs don't seem to have any PSCI idle states (deeper than WFI) described, which is no bueno, as they support some.. I'll try to improve this. 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
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,