| Message ID | 20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org |
|---|---|
| Headers | show |
| Series | Arm cpu schema clean-ups | expand |
On Thu, Apr 10, 2025 at 10:47:24AM -0500, 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> > Reviewed-by: Daniel Machon <daniel.machon@microchip.com> > Tested-by: Daniel Machon <daniel.machon@microchip.com> This is already applied, guess I forgot to merge it into the branch that appears in linux next. I'll do that now..
On Fri, Apr 11, 2025 at 03:26:50PM -0500, Rob Herring wrote: > On Fri, Apr 11, 2025 at 11:22 AM Conor Dooley <conor@kernel.org> wrote: > > > > On Thu, Apr 10, 2025 at 10:47:24AM -0500, 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> > > > Reviewed-by: Daniel Machon <daniel.machon@microchip.com> > > > Tested-by: Daniel Machon <daniel.machon@microchip.com> > > > > This is already applied, guess I forgot to merge it into the branch that > > appears in linux next. I'll do that now.. > > Sometimes I check next, but in this case I just looked at replies for > which there were none. I dislike submitting dts changes because it's a > range of AWOL maintainers, only applying around some rcN (so up to 2 > months later), silently applying, and applied but never in linux-next > (until in soc tree). Let's add "send the b4 ty email" to the list of things that I did not do, but thought that I had done.
On Thu, 10 Apr 2025 10:47: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, thanks! [07/17] arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names commit: ae33b874fc81bfb60bdda5a350c0635f98e6d747 Best regards,
On Thu, 10 Apr 2025 10:47: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, thanks! [10/17] arm: dts: rockchip: Drop redundant CPU "clock-latency" commit: 709a25f7a433d53dc9f0daf7cf5657f0671c5026 Best regards,
On Thu, Apr 10, 2025 at 10:50 AM Rob Herring (Arm) <robh@kernel.org> 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> > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > --- Ping! > 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 { > > -- > 2.47.2 > >
On Mon, May 5, 2025 at 1:43 PM Rob Herring <robh@kernel.org> wrote: > > On Thu, Apr 10, 2025 at 10:50 AM Rob Herring (Arm) <robh@kernel.org> 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> > > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > > --- > > Ping! NM, I see it now in linux-next.
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. --- v2: - Drop applied "arm64: dts: morello: Fix-up cache nodes" - Rework enable-method schema - Drop "arm: dts: qcom: msm8916: Move "qcom,acc" and "qcom,saw" to 32-bit .dtsi" - Keep qcom,saw and qcom,acc properties on msm8939 - Fix qcom,saw2.yaml example - Fix power-domain-names to be "perf" on qcom sdx55/65 Link to v1: https://lore.kernel.org/all/20250403-dt-cpu-schema-v1-0-076be7171a85@kernel.org/ Signed-off-by: "Rob Herring (Arm)" <robh@kernel.org> To: Krzysztof Kozlowski <krzk+dt@kernel.org> To: Conor Dooley <conor+dt@kernel.org> To: Chen-Yu Tsai <wens@csie.org> To: Jernej Skrabec <jernej.skrabec@gmail.com> To: Samuel Holland <samuel@sholland.org> To: Conor Dooley <conor@kernel.org> To: Nicolas Ferre <nicolas.ferre@microchip.com> To: Claudiu Beznea <claudiu.beznea@tuxon.dev> To: Steen Hegelund <Steen.Hegelund@microchip.com> To: Daniel Machon <daniel.machon@microchip.com> To: UNGLinuxDriver@microchip.com To: Bjorn Andersson <andersson@kernel.org> To: Konrad Dybcio <konradybcio@kernel.org> To: Shawn Guo <shawnguo@kernel.org> To: Sascha Hauer <s.hauer@pengutronix.de> To: Pengutronix Kernel Team <kernel@pengutronix.de> To: Fabio Estevam <festevam@gmail.com> To: Heiko Stuebner <heiko@sntech.de> To: Neil Armstrong <neil.armstrong@linaro.org> To: Kevin Hilman <khilman@baylibre.com> To: Jerome Brunet <jbrunet@baylibre.com> To: Martin Blumenstingl <martin.blumenstingl@googlemail.com> To: Geert Uytterhoeven <geert+renesas@glider.be> To: Magnus Damm <magnus.damm@gmail.com> To: Lorenzo Pieralisi <lpieralisi@kernel.org> To: Andy Gross <agross@kernel.org> To: Thomas Bogendoerfer <tsbogend@alpha.franken.de> To: Viresh Kumar <vireshk@kernel.org> To: Nishanth Menon <nm@ti.com> To: Stephen Boyd <sboyd@kernel.org> To: zhouyanjie@wanyeetech.com To: Matthias Brugger <matthias.bgg@gmail.com> To: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> To: "Rafael J. Wysocki" <rafael@kernel.org> To: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> To: Stephan Gerhold <stephan.gerhold@linaro.org> Cc: devicetree@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-sunxi@lists.linux.dev Cc: linux-kernel@vger.kernel.org Cc: linux-arm-msm@vger.kernel.org Cc: imx@lists.linux.dev Cc: linux-rockchip@lists.infradead.org Cc: linux-amlogic@lists.infradead.org Cc: linux-renesas-soc@vger.kernel.org Cc: linux-mips@vger.kernel.org Cc: linux-pm@vger.kernel.org Cc: linux-mediatek@lists.infradead.org --- Rob Herring (Arm) (17): 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: 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: 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 | 229 +++++++++++-------- .../bindings/cpufreq/cpufreq-mediatek.txt | 250 --------------------- Documentation/devicetree/bindings/mips/cpus.yaml | 3 +- Documentation/devicetree/bindings/opp/opp-v1.yaml | 18 +- .../devicetree/bindings/soc/qcom/qcom,saw2.yaml | 3 +- arch/arm/boot/dts/nxp/imx/imx7s.dtsi | 1 - arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi | 4 - 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/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/msm8939.dtsi | 8 + arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts | 6 + arch/arm64/boot/dts/qcom/qdu1000.dtsi | 8 +- 48 files changed, 202 insertions(+), 480 deletions(-) --- base-commit: 0af2f6be1b4281385b618cb86ad946eded089ac8 change-id: 20250403-dt-cpu-schema-48e66c7f6a90 Best regards,