* [PATCH v2 01/17] arm64: dts: allwinner: h5/h6: Drop spurious 'clock-latency-ns' properties
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 02/17] arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names Rob Herring (Arm)
` (20 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Andre Przywara
'clock-latency-ns' is not a valid property for CPU nodes. It belongs in
OPP table (which has it). Drop them from the CPU nodes.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Andre Przywara <andre.przywara@arm.com>
Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>
---
arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi | 4 ----
arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 4 ----
2 files changed, 8 deletions(-)
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi
index d3caf27b6a55..48802bf02f3b 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi
@@ -16,7 +16,6 @@ cpu0: cpu@0 {
reg = <0>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
};
@@ -26,7 +25,6 @@ cpu1: cpu@1 {
reg = <1>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
};
@@ -36,7 +34,6 @@ cpu2: cpu@2 {
reg = <2>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
};
@@ -46,7 +43,6 @@ cpu3: cpu@3 {
reg = <3>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
};
};
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
index 2301c59b41b1..73e8604315c5 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
@@ -27,7 +27,6 @@ cpu0: cpu@0 {
reg = <0>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
i-cache-size = <0x8000>;
i-cache-line-size = <64>;
@@ -44,7 +43,6 @@ cpu1: cpu@1 {
reg = <1>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
i-cache-size = <0x8000>;
i-cache-line-size = <64>;
@@ -61,7 +59,6 @@ cpu2: cpu@2 {
reg = <2>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
i-cache-size = <0x8000>;
i-cache-line-size = <64>;
@@ -78,7 +75,6 @@ cpu3: cpu@3 {
reg = <3>;
enable-method = "psci";
clocks = <&ccu CLK_CPUX>;
- clock-latency-ns = <244144>; /* 8 32k periods */
#cooling-cells = <2>;
i-cache-size = <0x8000>;
i-cache-line-size = <64>;
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 02/17] arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 01/17] arm64: dts: allwinner: h5/h6: Drop spurious 'clock-latency-ns' properties Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-11 23:37 ` Florian Fainelli
2025-04-10 15:47 ` [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies Rob Herring (Arm)
` (19 subsequent siblings)
21 siblings, 1 reply; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Philippe Mathieu-Daudé
There's no need include the CPU number in the L2 cache node names as
the names are local to the CPU nodes. The documented node name is
also just "l2-cache".
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
arch/arm64/boot/dts/broadcom/bcm2712.dtsi | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm64/boot/dts/broadcom/bcm2712.dtsi b/arch/arm64/boot/dts/broadcom/bcm2712.dtsi
index 9e610a89a337..ad0cac8e4444 100644
--- a/arch/arm64/boot/dts/broadcom/bcm2712.dtsi
+++ b/arch/arm64/boot/dts/broadcom/bcm2712.dtsi
@@ -64,7 +64,7 @@ cpu0: cpu@0 {
i-cache-sets = <256>; // 64KiB(size)/64(line-size)=1024ways/4-way set
next-level-cache = <&l2_cache_l0>;
- l2_cache_l0: l2-cache-l0 {
+ l2_cache_l0: l2-cache {
compatible = "cache";
cache-size = <0x80000>;
cache-line-size = <64>;
@@ -88,7 +88,7 @@ cpu1: cpu@1 {
i-cache-sets = <256>; // 64KiB(size)/64(line-size)=1024ways/4-way set
next-level-cache = <&l2_cache_l1>;
- l2_cache_l1: l2-cache-l1 {
+ l2_cache_l1: l2-cache {
compatible = "cache";
cache-size = <0x80000>;
cache-line-size = <64>;
@@ -112,7 +112,7 @@ cpu2: cpu@2 {
i-cache-sets = <256>; // 64KiB(size)/64(line-size)=1024ways/4-way set
next-level-cache = <&l2_cache_l2>;
- l2_cache_l2: l2-cache-l2 {
+ l2_cache_l2: l2-cache {
compatible = "cache";
cache-size = <0x80000>;
cache-line-size = <64>;
@@ -136,7 +136,7 @@ cpu3: cpu@3 {
i-cache-sets = <256>; // 64KiB(size)/64(line-size)=1024ways/4-way set
next-level-cache = <&l2_cache_l3>;
- l2_cache_l3: l2-cache-l3 {
+ l2_cache_l3: l2-cache {
compatible = "cache";
cache-size = <0x80000>;
cache-line-size = <64>;
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: [PATCH v2 02/17] arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names
2025-04-10 15:47 ` [PATCH v2 02/17] arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names Rob Herring (Arm)
@ 2025-04-11 23:37 ` Florian Fainelli
2025-04-14 13:12 ` Rob Herring
0 siblings, 1 reply; 34+ messages in thread
From: Florian Fainelli @ 2025-04-11 23:37 UTC (permalink / raw)
To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
Chen-Yu Tsai, Jernej Skrabec, Samuel Holland, Conor Dooley,
Nicolas Ferre, Claudiu Beznea, Steen Hegelund, Daniel Machon,
UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Philippe Mathieu-Daudé
On 4/10/25 08:47, Rob Herring (Arm) wrote:
> There's no need include the CPU number in the L2 cache node names as
> the names are local to the CPU nodes. The documented node name is
> also just "l2-cache".
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Not sure how you had intended for me to pick up that patch without
copying the maintainers.
Applied nonetheless, thanks!
--
Florian
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH v2 02/17] arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names
2025-04-11 23:37 ` Florian Fainelli
@ 2025-04-14 13:12 ` Rob Herring
0 siblings, 0 replies; 34+ messages in thread
From: Rob Herring @ 2025-04-14 13:12 UTC (permalink / raw)
To: Florian Fainelli
Cc: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold, devicetree, linux-arm-kernel,
linux-sunxi, linux-kernel, linux-arm-msm, imx, linux-rockchip,
linux-amlogic, linux-renesas-soc, linux-mips, linux-pm,
linux-mediatek, Philippe Mathieu-Daudé
On Fri, Apr 11, 2025 at 6:37 PM Florian Fainelli <f.fainelli@gmail.com> wrote:
>
> On 4/10/25 08:47, Rob Herring (Arm) wrote:
> > There's no need include the CPU number in the L2 cache node names as
> > the names are local to the CPU nodes. The documented node name is
> > also just "l2-cache".
> >
> > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> > Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>
> Not sure how you had intended for me to pick up that patch without
> copying the maintainers.
Looks like there is a problem in MAINTAINERS. This matches what b4 runs:
$ git show cca91c99fe14 | scripts/get_maintainer.pl --nogit
--nogit-fallback --nogit-chief-penguins --norolestats
Rob Herring <robh@kernel.org>
Krzysztof Kozlowski <krzk+dt@kernel.org>
Conor Dooley <conor+dt@kernel.org>
devicetree@vger.kernel.org
linux-kernel@vger.kernel.org
> Applied nonetheless, thanks!
Thanks.
Rob
^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 01/17] arm64: dts: allwinner: h5/h6: Drop spurious 'clock-latency-ns' properties Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 02/17] arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-11 16:21 ` Conor Dooley
2025-04-10 15:47 ` [PATCH v2 04/17] arm64: dts: qcom: qdu1000: Fix qcom,freq-domain Rob Herring (Arm)
` (18 subsequent siblings)
21 siblings, 1 reply; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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>
---
arch/arm64/boot/dts/microchip/sparx5_pcb_common.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/microchip/sparx5_pcb_common.dtsi b/arch/arm64/boot/dts/microchip/sparx5_pcb_common.dtsi
index 32bb76b3202a..83bf5c81b5f7 100644
--- a/arch/arm64/boot/dts/microchip/sparx5_pcb_common.dtsi
+++ b/arch/arm64/boot/dts/microchip/sparx5_pcb_common.dtsi
@@ -12,10 +12,12 @@ &psci {
&cpu0 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&cpu1 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&uart0 {
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies
2025-04-10 15:47 ` [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies Rob Herring (Arm)
@ 2025-04-11 16:21 ` Conor Dooley
2025-04-11 20:26 ` Rob Herring
0 siblings, 1 reply; 34+ messages in thread
From: Conor Dooley @ 2025-04-11 16:21 UTC (permalink / raw)
To: Rob Herring (Arm)
Cc: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Nicolas Ferre, Claudiu Beznea, Steen Hegelund,
Daniel Machon, UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, devicetree, linux-arm-kernel, linux-sunxi,
linux-kernel, linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
[-- Attachment #1: Type: text/plain, Size: 525 bytes --]
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..
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies
2025-04-11 16:21 ` Conor Dooley
@ 2025-04-11 20:26 ` Rob Herring
2025-04-14 17:02 ` Conor Dooley
0 siblings, 1 reply; 34+ messages in thread
From: Rob Herring @ 2025-04-11 20:26 UTC (permalink / raw)
To: Conor Dooley
Cc: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Nicolas Ferre, Claudiu Beznea, Steen Hegelund,
Daniel Machon, UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, devicetree, linux-arm-kernel, linux-sunxi,
linux-kernel, linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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).
Rob
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies
2025-04-11 20:26 ` Rob Herring
@ 2025-04-14 17:02 ` Conor Dooley
0 siblings, 0 replies; 34+ messages in thread
From: Conor Dooley @ 2025-04-14 17:02 UTC (permalink / raw)
To: Rob Herring
Cc: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Nicolas Ferre, Claudiu Beznea, Steen Hegelund,
Daniel Machon, UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, devicetree, linux-arm-kernel, linux-sunxi,
linux-kernel, linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
[-- Attachment #1: Type: text/plain, Size: 1138 bytes --]
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.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2 04/17] arm64: dts: qcom: qdu1000: Fix qcom,freq-domain
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (2 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 05/17] arm64: dts: qcom: msm8939: Fix CPU node "enable-method" property dependencies Rob Herring (Arm)
` (17 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Konrad Dybcio
The correct property name is 'qcom,freq-domain', not
'qcom,freq-domains'.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/qdu1000.dtsi | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/qdu1000.dtsi b/arch/arm64/boot/dts/qcom/qdu1000.dtsi
index f973aa8f7477..7c8d78fd7ebf 100644
--- a/arch/arm64/boot/dts/qcom/qdu1000.dtsi
+++ b/arch/arm64/boot/dts/qcom/qdu1000.dtsi
@@ -47,7 +47,7 @@ cpu0: cpu@0 {
enable-method = "psci";
power-domains = <&cpu_pd0>;
power-domain-names = "psci";
- qcom,freq-domains = <&cpufreq_hw 0>;
+ qcom,freq-domain = <&cpufreq_hw 0>;
next-level-cache = <&l2_0>;
l2_0: l2-cache {
compatible = "cache";
@@ -70,7 +70,7 @@ cpu1: cpu@100 {
enable-method = "psci";
power-domains = <&cpu_pd1>;
power-domain-names = "psci";
- qcom,freq-domains = <&cpufreq_hw 0>;
+ qcom,freq-domain = <&cpufreq_hw 0>;
next-level-cache = <&l2_100>;
l2_100: l2-cache {
compatible = "cache";
@@ -88,7 +88,7 @@ cpu2: cpu@200 {
enable-method = "psci";
power-domains = <&cpu_pd2>;
power-domain-names = "psci";
- qcom,freq-domains = <&cpufreq_hw 0>;
+ qcom,freq-domain = <&cpufreq_hw 0>;
next-level-cache = <&l2_200>;
l2_200: l2-cache {
compatible = "cache";
@@ -106,7 +106,7 @@ cpu3: cpu@300 {
enable-method = "psci";
power-domains = <&cpu_pd3>;
power-domain-names = "psci";
- qcom,freq-domains = <&cpufreq_hw 0>;
+ qcom,freq-domain = <&cpufreq_hw 0>;
next-level-cache = <&l2_300>;
l2_300: l2-cache {
compatible = "cache";
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 05/17] arm64: dts: qcom: msm8939: Fix CPU node "enable-method" property dependencies
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (3 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 04/17] arm64: dts: qcom: qdu1000: Fix qcom,freq-domain Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-11 9:15 ` Konrad Dybcio
2025-04-10 15:47 ` [PATCH v2 06/17] arm64: dts: qcom: msm8992-lg-h815: " Rob Herring (Arm)
` (16 subsequent siblings)
21 siblings, 1 reply; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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>
---
v2:
- Keep qcom,saw and qcom,acc properties
---
arch/arm64/boot/dts/qcom/msm8939.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/msm8939.dtsi b/arch/arm64/boot/dts/qcom/msm8939.dtsi
index 7cd5660de1b3..72f01953e65c 100644
--- a/arch/arm64/boot/dts/qcom/msm8939.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8939.dtsi
@@ -46,6 +46,7 @@ cpu0: cpu@100 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x100>;
next-level-cache = <&l2_1>;
qcom,acc = <&acc0>;
@@ -64,6 +65,7 @@ cpu1: cpu@101 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x101>;
next-level-cache = <&l2_1>;
qcom,acc = <&acc1>;
@@ -77,6 +79,7 @@ cpu2: cpu@102 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x102>;
next-level-cache = <&l2_1>;
qcom,acc = <&acc2>;
@@ -90,6 +93,7 @@ cpu3: cpu@103 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x103>;
next-level-cache = <&l2_1>;
qcom,acc = <&acc3>;
@@ -103,6 +107,7 @@ cpu4: cpu@0 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x0>;
qcom,acc = <&acc4>;
qcom,saw = <&saw4>;
@@ -121,6 +126,7 @@ cpu5: cpu@1 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x1>;
next-level-cache = <&l2_0>;
qcom,acc = <&acc5>;
@@ -134,6 +140,7 @@ cpu6: cpu@2 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x2>;
next-level-cache = <&l2_0>;
qcom,acc = <&acc6>;
@@ -147,6 +154,7 @@ cpu7: cpu@3 {
compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
reg = <0x3>;
next-level-cache = <&l2_0>;
qcom,acc = <&acc7>;
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: [PATCH v2 05/17] arm64: dts: qcom: msm8939: Fix CPU node "enable-method" property dependencies
2025-04-10 15:47 ` [PATCH v2 05/17] arm64: dts: qcom: msm8939: Fix CPU node "enable-method" property dependencies Rob Herring (Arm)
@ 2025-04-11 9:15 ` Konrad Dybcio
0 siblings, 0 replies; 34+ messages in thread
From: Konrad Dybcio @ 2025-04-11 9:15 UTC (permalink / raw)
To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
Chen-Yu Tsai, Jernej Skrabec, Samuel Holland, Conor Dooley,
Nicolas Ferre, Claudiu Beznea, Steen Hegelund, Daniel Machon,
UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
On 4/10/25 5:47 PM, 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>
> ---
Acked-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Konrad
^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2 06/17] arm64: dts: qcom: msm8992-lg-h815: Fix CPU node "enable-method" property dependencies
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (4 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 05/17] arm64: dts: qcom: msm8939: Fix CPU node "enable-method" property dependencies Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-11 9:16 ` Konrad Dybcio
2025-04-10 15:47 ` [PATCH v2 07/17] arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names Rob Herring (Arm)
` (15 subsequent siblings)
21 siblings, 1 reply; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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>
---
arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts b/arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts
index 4520d5d51a29..6a231afad85d 100644
--- a/arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts
+++ b/arch/arm64/boot/dts/qcom/msm8992-lg-h815.dts
@@ -93,26 +93,32 @@ key-vol-up {
&cpu0 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&cpu1 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&cpu2 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&cpu3 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&cpu4 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&cpu5 {
enable-method = "spin-table";
+ cpu-release-addr = /bits/ 64 <0>;
};
&pm8994_resin {
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: [PATCH v2 06/17] arm64: dts: qcom: msm8992-lg-h815: Fix CPU node "enable-method" property dependencies
2025-04-10 15:47 ` [PATCH v2 06/17] arm64: dts: qcom: msm8992-lg-h815: " Rob Herring (Arm)
@ 2025-04-11 9:16 ` Konrad Dybcio
0 siblings, 0 replies; 34+ messages in thread
From: Konrad Dybcio @ 2025-04-11 9:16 UTC (permalink / raw)
To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
Chen-Yu Tsai, Jernej Skrabec, Samuel Holland, Conor Dooley,
Nicolas Ferre, Claudiu Beznea, Steen Hegelund, Daniel Machon,
UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
On 4/10/25 5:47 PM, 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>
> ---
Acked-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Konrad
^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2 07/17] arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (5 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 06/17] arm64: dts: qcom: msm8992-lg-h815: " Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-11 9:16 ` Konrad Dybcio
2025-04-10 15:47 ` [PATCH v2 08/17] arm/arm64: dts: imx: Drop redundant CPU "clock-latency" Rob Herring (Arm)
` (14 subsequent siblings)
21 siblings, 1 reply; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
"rpmhpd" is not documented nor used anywhere. The power-domain is used
for performance scaling (cpufreq), so "perf" is the correct name to use.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
v2:
- Use "perf" instead of "psci"
---
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..20fdae9825e0 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 = "perf";
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..c8e312dcd26b 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 = "perf";
operating-points-v2 = <&cpu_opp_table>;
};
};
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: [PATCH v2 07/17] arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names
2025-04-10 15:47 ` [PATCH v2 07/17] arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names Rob Herring (Arm)
@ 2025-04-11 9:16 ` Konrad Dybcio
0 siblings, 0 replies; 34+ messages in thread
From: Konrad Dybcio @ 2025-04-11 9:16 UTC (permalink / raw)
To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
Chen-Yu Tsai, Jernej Skrabec, Samuel Holland, Conor Dooley,
Nicolas Ferre, Claudiu Beznea, Steen Hegelund, Daniel Machon,
UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
On 4/10/25 5:47 PM, Rob Herring (Arm) wrote:
> "rpmhpd" is not documented nor used anywhere. The power-domain is used
> for performance scaling (cpufreq), so "perf" is the correct name to use.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Konrad
^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2 08/17] arm/arm64: dts: imx: Drop redundant CPU "clock-latency"
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (6 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 07/17] arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-05-07 7:33 ` Peng Fan
2025-04-10 15:47 ` [PATCH v2 09/17] arm: dts: qcom: ipq4019: " Rob Herring (Arm)
` (13 subsequent siblings)
21 siblings, 1 reply; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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 tables have values of 150000, so it can be
removed.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
arch/arm/boot/dts/nxp/imx/imx7s.dtsi | 1 -
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 ----
5 files changed, 17 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx7s.dtsi b/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
index 2629968001a7..9235dd7e93bb 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
@@ -73,7 +73,6 @@ cpu0: cpu@0 {
device_type = "cpu";
reg = <0>;
clock-frequency = <792000000>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clks IMX7D_CLK_ARM>;
cpu-idle-states = <&cpu_sleep_wait>;
operating-points-v2 = <&cpu0_opp_table>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index 4de3bf22902b..cfebaa01217e 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -62,7 +62,6 @@ A53_0: cpu@0 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x0>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MM_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -83,7 +82,6 @@ A53_1: cpu@1 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x1>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MM_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -102,7 +100,6 @@ A53_2: cpu@2 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x2>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MM_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -121,7 +118,6 @@ A53_3: cpu@3 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x3>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MM_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
index a5f9cfb46e5d..848ba5e46ee6 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
@@ -62,7 +62,6 @@ A53_0: cpu@0 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x0>;
- clock-latency = <61036>;
clocks = <&clk IMX8MN_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -83,7 +82,6 @@ A53_1: cpu@1 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x1>;
- clock-latency = <61036>;
clocks = <&clk IMX8MN_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -102,7 +100,6 @@ A53_2: cpu@2 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x2>;
- clock-latency = <61036>;
clocks = <&clk IMX8MN_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -121,7 +118,6 @@ A53_3: cpu@3 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x3>;
- clock-latency = <61036>;
clocks = <&clk IMX8MN_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
index ce6793b2d57e..f8afdba71c36 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
@@ -65,7 +65,6 @@ A53_0: cpu@0 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x0>;
- clock-latency = <61036>;
clocks = <&clk IMX8MP_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -86,7 +85,6 @@ A53_1: cpu@1 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x1>;
- clock-latency = <61036>;
clocks = <&clk IMX8MP_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -105,7 +103,6 @@ A53_2: cpu@2 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x2>;
- clock-latency = <61036>;
clocks = <&clk IMX8MP_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -124,7 +121,6 @@ A53_3: cpu@3 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x3>;
- clock-latency = <61036>;
clocks = <&clk IMX8MP_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index d51de8d899b2..d27b824995eb 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -106,7 +106,6 @@ A53_0: cpu@0 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x0>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MQ_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -126,7 +125,6 @@ A53_1: cpu@1 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x1>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MQ_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -144,7 +142,6 @@ A53_2: cpu@2 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x2>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MQ_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
@@ -162,7 +159,6 @@ A53_3: cpu@3 {
device_type = "cpu";
compatible = "arm,cortex-a53";
reg = <0x3>;
- clock-latency = <61036>; /* two CLK32 periods */
clocks = <&clk IMX8MQ_CLK_ARM>;
enable-method = "psci";
i-cache-size = <0x8000>;
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: [PATCH v2 08/17] arm/arm64: dts: imx: Drop redundant CPU "clock-latency"
2025-04-10 15:47 ` [PATCH v2 08/17] arm/arm64: dts: imx: Drop redundant CPU "clock-latency" Rob Herring (Arm)
@ 2025-05-07 7:33 ` Peng Fan
0 siblings, 0 replies; 34+ messages in thread
From: Peng Fan @ 2025-05-07 7:33 UTC (permalink / raw)
To: Rob Herring (Arm)
Cc: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold, devicetree, linux-arm-kernel,
linux-sunxi, linux-kernel, linux-arm-msm, imx, linux-rockchip,
linux-amlogic, linux-renesas-soc, linux-mips, linux-pm,
linux-mediatek
On Thu, Apr 10, 2025 at 10:47:29AM -0500, 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 tables have values of 150000, so it can be
>removed.
>
>Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
>---
> arch/arm/boot/dts/nxp/imx/imx7s.dtsi | 1 -
> 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 ----
> 5 files changed, 17 deletions(-)
>
Acked-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2 09/17] arm: dts: qcom: ipq4019: Drop redundant CPU "clock-latency"
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (7 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 08/17] arm/arm64: dts: imx: Drop redundant CPU "clock-latency" Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 10/17] arm: dts: rockchip: " Rob Herring (Arm)
` (12 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Konrad Dybcio
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>
---
arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi | 4 ----
1 file changed, 4 deletions(-)
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
index 06b20c196faf..fceb2f5f5482 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
@@ -53,7 +53,6 @@ cpu@0 {
reg = <0x0>;
clocks = <&gcc GCC_APPS_CLK_SRC>;
clock-frequency = <0>;
- clock-latency = <256000>;
operating-points-v2 = <&cpu0_opp_table>;
};
@@ -67,7 +66,6 @@ cpu@1 {
reg = <0x1>;
clocks = <&gcc GCC_APPS_CLK_SRC>;
clock-frequency = <0>;
- clock-latency = <256000>;
operating-points-v2 = <&cpu0_opp_table>;
};
@@ -81,7 +79,6 @@ cpu@2 {
reg = <0x2>;
clocks = <&gcc GCC_APPS_CLK_SRC>;
clock-frequency = <0>;
- clock-latency = <256000>;
operating-points-v2 = <&cpu0_opp_table>;
};
@@ -95,7 +92,6 @@ cpu@3 {
reg = <0x3>;
clocks = <&gcc GCC_APPS_CLK_SRC>;
clock-frequency = <0>;
- clock-latency = <256000>;
operating-points-v2 = <&cpu0_opp_table>;
};
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 10/17] arm: dts: rockchip: Drop redundant CPU "clock-latency"
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (8 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 09/17] arm: dts: qcom: ipq4019: " Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 11/17] arm64: dts: amlogic: " Rob Herring (Arm)
` (11 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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/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 -
5 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/arch/arm/boot/dts/rockchip/rk3128.dtsi b/arch/arm/boot/dts/rockchip/rk3128.dtsi
index d4572146d135..c49099954c28 100644
--- a/arch/arm/boot/dts/rockchip/rk3128.dtsi
+++ b/arch/arm/boot/dts/rockchip/rk3128.dtsi
@@ -48,7 +48,6 @@ cpu0: cpu@f00 {
device_type = "cpu";
compatible = "arm,cortex-a7";
reg = <0xf00>;
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
resets = <&cru SRST_CORE0>;
operating-points-v2 = <&cpu_opp_table>;
@@ -87,31 +86,38 @@ cpu_opp_table: opp-table-0 {
opp-216000000 {
opp-hz = /bits/ 64 <216000000>;
opp-microvolt = <950000 950000 1325000>;
+ clock-latency-ns = <40000>;
};
opp-408000000 {
opp-hz = /bits/ 64 <408000000>;
opp-microvolt = <950000 950000 1325000>;
+ clock-latency-ns = <40000>;
};
opp-600000000 {
opp-hz = /bits/ 64 <600000000>;
opp-microvolt = <950000 950000 1325000>;
+ clock-latency-ns = <40000>;
};
opp-696000000 {
opp-hz = /bits/ 64 <696000000>;
opp-microvolt = <975000 975000 1325000>;
+ clock-latency-ns = <40000>;
};
opp-816000000 {
opp-hz = /bits/ 64 <816000000>;
opp-microvolt = <1075000 1075000 1325000>;
opp-suspend;
+ clock-latency-ns = <40000>;
};
opp-1008000000 {
opp-hz = /bits/ 64 <1008000000>;
opp-microvolt = <1200000 1200000 1325000>;
+ clock-latency-ns = <40000>;
};
opp-1200000000 {
opp-hz = /bits/ 64 <1200000000>;
opp-microvolt = <1325000 1325000 1325000>;
+ clock-latency-ns = <40000>;
};
};
diff --git a/arch/arm/boot/dts/rockchip/rk3188.dtsi b/arch/arm/boot/dts/rockchip/rk3188.dtsi
index 44b54af0bbf9..850bd6e67895 100644
--- a/arch/arm/boot/dts/rockchip/rk3188.dtsi
+++ b/arch/arm/boot/dts/rockchip/rk3188.dtsi
@@ -23,7 +23,6 @@ cpu0: cpu@0 {
compatible = "arm,cortex-a9";
next-level-cache = <&L2>;
reg = <0x0>;
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
operating-points-v2 = <&cpu0_opp_table>;
resets = <&cru SRST_CORE0>;
diff --git a/arch/arm/boot/dts/rockchip/rk322x.dtsi b/arch/arm/boot/dts/rockchip/rk322x.dtsi
index 96421355c274..cd11a018105b 100644
--- a/arch/arm/boot/dts/rockchip/rk322x.dtsi
+++ b/arch/arm/boot/dts/rockchip/rk322x.dtsi
@@ -36,7 +36,6 @@ cpu0: cpu@f00 {
resets = <&cru SRST_CORE0>;
operating-points-v2 = <&cpu0_opp_table>;
#cooling-cells = <2>; /* min followed by max */
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
enable-method = "psci";
};
diff --git a/arch/arm/boot/dts/rockchip/rk3288.dtsi b/arch/arm/boot/dts/rockchip/rk3288.dtsi
index 3f1d640afafa..42d705b544ec 100644
--- a/arch/arm/boot/dts/rockchip/rk3288.dtsi
+++ b/arch/arm/boot/dts/rockchip/rk3288.dtsi
@@ -70,7 +70,6 @@ cpu0: cpu@500 {
resets = <&cru SRST_CORE0>;
operating-points-v2 = <&cpu_opp_table>;
#cooling-cells = <2>; /* min followed by max */
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
dynamic-power-coefficient = <370>;
};
@@ -81,7 +80,6 @@ cpu1: cpu@501 {
resets = <&cru SRST_CORE1>;
operating-points-v2 = <&cpu_opp_table>;
#cooling-cells = <2>; /* min followed by max */
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
dynamic-power-coefficient = <370>;
};
@@ -92,7 +90,6 @@ cpu2: cpu@502 {
resets = <&cru SRST_CORE2>;
operating-points-v2 = <&cpu_opp_table>;
#cooling-cells = <2>; /* min followed by max */
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
dynamic-power-coefficient = <370>;
};
@@ -103,7 +100,6 @@ cpu3: cpu@503 {
resets = <&cru SRST_CORE3>;
operating-points-v2 = <&cpu_opp_table>;
#cooling-cells = <2>; /* min followed by max */
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
dynamic-power-coefficient = <370>;
};
@@ -116,6 +112,7 @@ cpu_opp_table: opp-table-0 {
opp-126000000 {
opp-hz = /bits/ 64 <126000000>;
opp-microvolt = <900000>;
+ clock-latency-ns = <40000>;
};
opp-216000000 {
opp-hz = /bits/ 64 <216000000>;
diff --git a/arch/arm/boot/dts/rockchip/rv1108.dtsi b/arch/arm/boot/dts/rockchip/rv1108.dtsi
index f3291f3bbc6f..42a4d72597a5 100644
--- a/arch/arm/boot/dts/rockchip/rv1108.dtsi
+++ b/arch/arm/boot/dts/rockchip/rv1108.dtsi
@@ -32,7 +32,6 @@ cpu0: cpu@f00 {
device_type = "cpu";
compatible = "arm,cortex-a7";
reg = <0xf00>;
- clock-latency = <40000>;
clocks = <&cru ARMCLK>;
#cooling-cells = <2>; /* min followed by max */
dynamic-power-coefficient = <75>;
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 11/17] arm64: dts: amlogic: Drop redundant CPU "clock-latency"
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (9 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 10/17] arm: dts: rockchip: " Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-05-05 18:43 ` Rob Herring
2025-04-10 15:47 ` [PATCH v2 12/17] dt-bindings: arm/cpus: Add schemas for "enable-method" dependencies Rob Herring (Arm)
` (10 subsequent siblings)
21 siblings, 1 reply; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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>
---
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
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: [PATCH v2 11/17] arm64: dts: amlogic: Drop redundant CPU "clock-latency"
2025-04-10 15:47 ` [PATCH v2 11/17] arm64: dts: amlogic: " Rob Herring (Arm)
@ 2025-05-05 18:43 ` Rob Herring
2025-05-05 18:50 ` Rob Herring
0 siblings, 1 reply; 34+ messages in thread
From: Rob Herring @ 2025-05-05 18:43 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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
>
>
^ permalink raw reply [flat|nested] 34+ messages in thread* Re: [PATCH v2 11/17] arm64: dts: amlogic: Drop redundant CPU "clock-latency"
2025-05-05 18:43 ` Rob Herring
@ 2025-05-05 18:50 ` Rob Herring
0 siblings, 0 replies; 34+ messages in thread
From: Rob Herring @ 2025-05-05 18:50 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek
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.
^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2 12/17] dt-bindings: arm/cpus: Add schemas for "enable-method" dependencies
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (10 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 11/17] arm64: dts: amlogic: " Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 13/17] dt-bindings: arm/cpus: Re-wrap 'description' entries Rob Herring (Arm)
` (9 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Sudeep Holla
Replace the prose for properties dependent on specific "enable-method"
values with schemas defining the same requirements.
Both "qcom,acc" and "qcom,saw" properties appear to be required for any
of the Qualcomm enable-method values, so the schema is a bit simpler
than what the text said. The properties are also needed on some Qualcomm
platforms with other enable-method values. It's limited to Cortex A53
based platforms so use that to disable the properties. The references
to arm/msm/qcom,saw2.txt and arm/msm/qcom,kpss-acc.txt are out of date,
so just drop them.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
---
v2:
- Only disallow qcom,acc and qcom,saw on !cortex-a53
- Fix example in qcom,saw2.yaml
---
Documentation/devicetree/bindings/arm/cpus.yaml | 91 ++++++++++++++--------
.../devicetree/bindings/soc/qcom/qcom,saw2.yaml | 3 +-
2 files changed, 60 insertions(+), 34 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
index 2e666b2a4dcd..351be2f77581 100644
--- a/Documentation/devicetree/bindings/arm/cpus.yaml
+++ b/Documentation/devicetree/bindings/arm/cpus.yaml
@@ -273,8 +273,6 @@ properties:
description:
The DT specification defines this as 64-bit always, but some 32-bit Arm
systems have used a 32-bit value which must be supported.
- Required for systems that have an "enable-method"
- property value of "spin-table".
cpu-idle-states:
$ref: /schemas/types.yaml#/definitions/phandle-array
@@ -333,24 +331,13 @@ properties:
qcom,saw:
$ref: /schemas/types.yaml#/definitions/phandle
- description: |
- Specifies the SAW* node associated with this CPU.
-
- Required for systems that have an "enable-method" property
- value of "qcom,kpss-acc-v1" or "qcom,kpss-acc-v2"
-
- * arm/msm/qcom,saw2.txt
+ description:
+ Specifies the SAW node associated with this CPU.
qcom,acc:
$ref: /schemas/types.yaml#/definitions/phandle
- description: |
- Specifies the ACC* node associated with this CPU.
-
- Required for systems that have an "enable-method" property
- value of "qcom,kpss-acc-v1", "qcom,kpss-acc-v2", "qcom,msm8226-smp" or
- "qcom,msm8916-smp".
-
- * arm/msm/qcom,kpss-acc.txt
+ description:
+ Specifies the ACC node associated with this CPU.
rockchip,pmu:
$ref: /schemas/types.yaml#/definitions/phandle
@@ -378,22 +365,60 @@ properties:
formed by encoding the target CPU id into the low bits of the
physical start address it should jump to.
-if:
- # If the enable-method property contains one of those values
- properties:
- enable-method:
- contains:
- enum:
- - brcm,bcm11351-cpu-method
- - brcm,bcm23550
- - brcm,bcm-nsp-smp
- # and if enable-method is present
- required:
- - enable-method
-
-then:
- required:
- - secondary-boot-reg
+allOf:
+ - if:
+ # If the enable-method property contains one of those values
+ properties:
+ enable-method:
+ contains:
+ enum:
+ - brcm,bcm11351-cpu-method
+ - brcm,bcm23550
+ - brcm,bcm-nsp-smp
+ # and if enable-method is present
+ required:
+ - enable-method
+ then:
+ required:
+ - secondary-boot-reg
+ - if:
+ properties:
+ enable-method:
+ enum:
+ - spin-table
+ - renesas,r9a06g032-smp
+ required:
+ - enable-method
+ then:
+ required:
+ - cpu-release-addr
+ - if:
+ properties:
+ enable-method:
+ enum:
+ - qcom,kpss-acc-v1
+ - qcom,kpss-acc-v2
+ - qcom,msm8226-smp
+ - qcom,msm8916-smp
+ required:
+ - enable-method
+ then:
+ required:
+ - qcom,acc
+ - qcom,saw
+ - if:
+ # 2 Qualcomm platforms bootloaders need qcom,acc and qcom,saw yet use
+ # "spin-table" or "psci" enable-methods. Disallowing the properties for
+ # all other CPUs is the best we can do as there's not any way to
+ # distinguish these Qualcomm platforms.
+ not:
+ properties:
+ compatible:
+ const: arm,cortex-a53
+ then:
+ properties:
+ qcom,acc: false
+ qcom,saw: false
required:
- device_type
diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,saw2.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,saw2.yaml
index ca4bce817273..c2f1f5946cfa 100644
--- a/Documentation/devicetree/bindings/soc/qcom/qcom,saw2.yaml
+++ b/Documentation/devicetree/bindings/soc/qcom/qcom,saw2.yaml
@@ -73,9 +73,10 @@ examples:
#size-cells = <0>;
cpu@0 {
- compatible = "qcom,kryo";
+ compatible = "arm,cortex-a53";
device_type = "cpu";
enable-method = "qcom,kpss-acc-v2";
+ qcom,acc = <&acc0>;
qcom,saw = <&saw0>;
reg = <0x0>;
operating-points-v2 = <&cpu_opp_table>;
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 13/17] dt-bindings: arm/cpus: Re-wrap 'description' entries
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (11 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 12/17] dt-bindings: arm/cpus: Add schemas for "enable-method" dependencies Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 14/17] dt-bindings: Reference opp-v1 schema in CPU schemas Rob Herring (Arm)
` (8 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Sudeep Holla
Some of the 'description' entries have odd line wrapping and incorrect
YAML block modifiers. The 'description' entries should typically wrap
at 80 chars. Reformat the entries to follow that along with using '>'
modifiers as appropriate.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
---
Documentation/devicetree/bindings/arm/cpus.yaml | 85 +++++++++++--------------
1 file changed, 36 insertions(+), 49 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
index 351be2f77581..acf38b3518dd 100644
--- a/Documentation/devicetree/bindings/arm/cpus.yaml
+++ b/Documentation/devicetree/bindings/arm/cpus.yaml
@@ -10,9 +10,9 @@ maintainers:
- Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
description: |+
- The device tree allows to describe the layout of CPUs in a system through
- the "cpus" node, which in turn contains a number of subnodes (ie "cpu")
- defining properties for every cpu.
+ The device tree allows to describe the layout of CPUs in a system through the
+ "cpus" node, which in turn contains a number of subnodes (ie "cpu") defining
+ properties for every cpu.
Bindings for CPU nodes follow the Devicetree Specification, available from:
@@ -41,45 +41,40 @@ description: |+
properties:
reg:
maxItems: 1
- description: |
- Usage and definition depend on ARM architecture version and
- configuration:
+ description: >
+ Usage and definition depend on ARM architecture version and configuration:
- On uniprocessor ARM architectures previous to v7
- this property is required and must be set to 0.
+ On uniprocessor ARM architectures previous to v7 this property is required
+ and must be set to 0.
- On ARM 11 MPcore based systems this property is
- required and matches the CPUID[11:0] register bits.
+ On ARM 11 MPcore based systems this property is required and matches the
+ CPUID[11:0] register bits.
- Bits [11:0] in the reg cell must be set to
- bits [11:0] in CPU ID register.
+ Bits [11:0] in the reg cell must be set to bits [11:0] in CPU ID register.
All other bits in the reg cell must be set to 0.
- On 32-bit ARM v7 or later systems this property is
- required and matches the CPU MPIDR[23:0] register
- bits.
+ On 32-bit ARM v7 or later systems this property is required and matches
+ the CPU MPIDR[23:0] register bits.
- Bits [23:0] in the reg cell must be set to
- bits [23:0] in MPIDR.
+ Bits [23:0] in the reg cell must be set to bits [23:0] in MPIDR.
All other bits in the reg cell must be set to 0.
- On ARM v8 64-bit systems this property is required
- and matches the MPIDR_EL1 register affinity bits.
+ On ARM v8 64-bit systems this property is required and matches the
+ MPIDR_EL1 register affinity bits.
* If cpus node's #address-cells property is set to 2
- The first reg cell bits [7:0] must be set to
- bits [39:32] of MPIDR_EL1.
+ The first reg cell bits [7:0] must be set to bits [39:32] of
+ MPIDR_EL1.
- The second reg cell bits [23:0] must be set to
- bits [23:0] of MPIDR_EL1.
+ The second reg cell bits [23:0] must be set to bits [23:0] of
+ MPIDR_EL1.
* If cpus node's #address-cells property is set to 1
- The reg cell bits [23:0] must be set to bits [23:0]
- of MPIDR_EL1.
+ The reg cell bits [23:0] must be set to bits [23:0] of MPIDR_EL1.
All other bits in the reg cells must be set to 0.
@@ -278,29 +273,26 @@ properties:
$ref: /schemas/types.yaml#/definitions/phandle-array
items:
maxItems: 1
- description: |
- List of phandles to idle state nodes supported
- by this cpu (see ./idle-states.yaml).
+ description:
+ List of phandles to idle state nodes supported by this cpu (see
+ ./idle-states.yaml).
capacity-dmips-mhz:
description:
u32 value representing CPU capacity (see ../cpu/cpu-capacity.txt) in
- DMIPS/MHz, relative to highest capacity-dmips-mhz
- in the system.
+ DMIPS/MHz, relative to highest capacity-dmips-mhz in the system.
cci-control-port: true
dynamic-power-coefficient:
$ref: /schemas/types.yaml#/definitions/uint32
- description:
- A u32 value that represents the running time dynamic
- power coefficient in units of uW/MHz/V^2. The
- coefficient can either be calculated from power
+ description: >
+ A u32 value that represents the running time dynamic power coefficient in
+ units of uW/MHz/V^2. The coefficient can either be calculated from power
measurements or derived by analysis.
- The dynamic power consumption of the CPU is
- proportional to the square of the Voltage (V) and
- the clock frequency (f). The coefficient is used to
+ The dynamic power consumption of the CPU is proportional to the square of
+ the Voltage (V) and the clock frequency (f). The coefficient is used to
calculate the dynamic power as below -
Pdyn = dynamic-power-coefficient * V^2 * f
@@ -309,10 +301,6 @@ properties:
performance-domains:
maxItems: 1
- description:
- List of phandles and performance domain specifiers, as defined by
- bindings of the performance domain provider. See also
- dvfs/performance-domain.yaml.
power-domains:
description:
@@ -341,22 +329,21 @@ properties:
rockchip,pmu:
$ref: /schemas/types.yaml#/definitions/phandle
- description: |
+ description: >
Specifies the syscon node controlling the cpu core power domains.
- Optional for systems that have an "enable-method"
- property value of "rockchip,rk3066-smp"
- While optional, it is the preferred way to get access to
- the cpu-core power-domains.
+ Optional for systems that have an "enable-method" property value of
+ "rockchip,rk3066-smp". While optional, it is the preferred way to get
+ access to the cpu-core power-domains.
secondary-boot-reg:
$ref: /schemas/types.yaml#/definitions/uint32
- description: |
+ description: >
Required for systems that have an "enable-method" property value of
"brcm,bcm11351-cpu-method", "brcm,bcm23550" or "brcm,bcm-nsp-smp".
- This includes the following SoCs: |
- BCM11130, BCM11140, BCM11351, BCM28145, BCM28155, BCM21664, BCM23550
+ This includes the following SoCs:
+ BCM11130, BCM11140, BCM11351, BCM28145, BCM28155, BCM21664, BCM23550,
BCM58522, BCM58525, BCM58535, BCM58622, BCM58623, BCM58625, BCM88312
The secondary-boot-reg property is a u32 value that specifies the
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 14/17] dt-bindings: Reference opp-v1 schema in CPU schemas
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (12 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 13/17] dt-bindings: arm/cpus: Re-wrap 'description' entries Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 15/17] dt-bindings: arm/cpus: Add missing properties Rob Herring (Arm)
` (7 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Viresh Kumar, Sudeep Holla
The opp-v1 binding is only used in MIPS and arm32 CPU nodes, so add a
$ref to it in the CPU schemas and drop the "select".
As opp-v1 has long been deprecated, mark it as such.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
---
Documentation/devicetree/bindings/arm/cpus.yaml | 1 +
Documentation/devicetree/bindings/mips/cpus.yaml | 3 ++-
Documentation/devicetree/bindings/opp/opp-v1.yaml | 2 +-
3 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
index acf38b3518dd..5af1396d631f 100644
--- a/Documentation/devicetree/bindings/arm/cpus.yaml
+++ b/Documentation/devicetree/bindings/arm/cpus.yaml
@@ -353,6 +353,7 @@ properties:
physical start address it should jump to.
allOf:
+ - $ref: /schemas/opp/opp-v1.yaml#
- if:
# If the enable-method property contains one of those values
properties:
diff --git a/Documentation/devicetree/bindings/mips/cpus.yaml b/Documentation/devicetree/bindings/mips/cpus.yaml
index a85137add668..471373ad0cfb 100644
--- a/Documentation/devicetree/bindings/mips/cpus.yaml
+++ b/Documentation/devicetree/bindings/mips/cpus.yaml
@@ -50,6 +50,7 @@ properties:
device_type: true
allOf:
+ - $ref: /schemas/opp/opp-v1.yaml#
- if:
properties:
compatible:
@@ -68,7 +69,7 @@ required:
- compatible
- reg
-additionalProperties: false
+unevaluatedProperties: false
examples:
- |
diff --git a/Documentation/devicetree/bindings/opp/opp-v1.yaml b/Documentation/devicetree/bindings/opp/opp-v1.yaml
index 07e26c267815..1b59b103dab6 100644
--- a/Documentation/devicetree/bindings/opp/opp-v1.yaml
+++ b/Documentation/devicetree/bindings/opp/opp-v1.yaml
@@ -18,7 +18,7 @@ description: |+
This binding only supports voltage-frequency pairs.
-select: true
+deprecated: true
properties:
operating-points:
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 15/17] dt-bindings: arm/cpus: Add missing properties
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (13 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 14/17] dt-bindings: Reference opp-v1 schema in CPU schemas Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 16/17] dt-bindings: arm/cpus: Add power-domains constraints Rob Herring (Arm)
` (6 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Sudeep Holla, Viresh Kumar
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>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.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 5af1396d631f..3846f13ded5b 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
+
qcom,saw:
$ref: /schemas/types.yaml#/definitions/phandle
description:
@@ -327,6 +362,11 @@ properties:
description:
Specifies the ACC node associated with this CPU.
+ qcom,freq-domain:
+ description: Specifies the QCom CPUFREQ HW associated with the CPU.
+ $ref: /schemas/types.yaml#/definitions/phandle-array
+ maxItems: 1
+
rockchip,pmu:
$ref: /schemas/types.yaml#/definitions/phandle
description: >
@@ -352,7 +392,11 @@ properties:
formed by encoding the target CPU id into the low bits of the
physical start address it should jump to.
+ thermal-idle:
+ type: object
+
allOf:
+ - $ref: /schemas/cpu.yaml#
- $ref: /schemas/opp/opp-v1.yaml#
- if:
# If the enable-method property contains one of those values
@@ -416,7 +460,7 @@ required:
dependencies:
rockchip,pmu: [enable-method]
-additionalProperties: true
+unevaluatedProperties: false
examples:
- |
diff --git a/Documentation/devicetree/bindings/opp/opp-v1.yaml b/Documentation/devicetree/bindings/opp/opp-v1.yaml
index 1b59b103dab6..61c080e50859 100644
--- a/Documentation/devicetree/bindings/opp/opp-v1.yaml
+++ b/Documentation/devicetree/bindings/opp/opp-v1.yaml
@@ -21,6 +21,18 @@ description: |+
deprecated: true
properties:
+ clock-latency:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description:
+ The latency in nanoseconds for clock changes. Use OPP tables for new
+ designs instead.
+
+ voltage-tolerance:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ maximum: 10
+ description:
+ The voltage tolerance in percent. Use OPP tables for new designs instead.
+
operating-points:
$ref: /schemas/types.yaml#/definitions/uint32-matrix
items:
@@ -28,8 +40,12 @@ properties:
- description: Frequency in kHz
- description: Voltage for OPP in uV
+dependencies:
+ clock-latency: [ operating-points ]
+ voltage-tolerance: [ operating-points ]
additionalProperties: true
+
examples:
- |
cpus {
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 16/17] dt-bindings: arm/cpus: Add power-domains constraints
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (14 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 15/17] dt-bindings: arm/cpus: Add missing properties Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-10 15:47 ` [PATCH v2 17/17] dt-bindings: cpufreq: Drop redundant Mediatek binding Rob Herring (Arm)
` (5 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Ulf Hansson, Sudeep Holla
The "power-domains" and "power-domains-names" properties are missing any
constraints. Add the constraints and drop the generic descriptions.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
---
Documentation/devicetree/bindings/arm/cpus.yaml | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
index 3846f13ded5b..3d61313ca00e 100644
--- a/Documentation/devicetree/bindings/arm/cpus.yaml
+++ b/Documentation/devicetree/bindings/arm/cpus.yaml
@@ -313,19 +313,15 @@ properties:
maxItems: 1
power-domains:
- description:
- List of phandles and PM domain specifiers, as defined by bindings of the
- PM domain provider (see also ../power_domain.txt).
+ maxItems: 1
power-domain-names:
description:
- A list of power domain name strings sorted in the same order as the
- power-domains property.
-
For PSCI based platforms, the name corresponding to the index of the PSCI
PM domain provider, must be "psci". For SCMI based platforms, the name
corresponding to the index of an SCMI performance domain provider, must be
"perf".
+ enum: [ psci, perf, cpr ]
resets:
maxItems: 1
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* [PATCH v2 17/17] dt-bindings: cpufreq: Drop redundant Mediatek binding
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (15 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 16/17] dt-bindings: arm/cpus: Add power-domains constraints Rob Herring (Arm)
@ 2025-04-10 15:47 ` Rob Herring (Arm)
2025-04-11 8:37 ` (subset) [PATCH v2 00/17] Arm cpu schema clean-ups Neil Armstrong
` (4 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 15:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Geert Uytterhoeven,
Magnus Damm, Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer,
Viresh Kumar, Nishanth Menon, Stephen Boyd, zhouyanjie,
Matthias Brugger, AngeloGioacchino Del Regno, Rafael J. Wysocki,
Dmitry Baryshkov, Stephan Gerhold
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Sudeep Holla, Viresh Kumar
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>
Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
---
.../bindings/cpufreq/cpufreq-mediatek.txt | 250 ---------------------
1 file changed, 250 deletions(-)
diff --git a/Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.txt b/Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.txt
deleted file mode 100644
index e0a4ba599abc..000000000000
--- a/Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.txt
+++ /dev/null
@@ -1,250 +0,0 @@
-Binding for MediaTek's CPUFreq driver
-=====================================
-
-Required properties:
-- clocks: A list of phandle + clock-specifier pairs for the clocks listed in clock names.
-- clock-names: Should contain the following:
- "cpu" - The multiplexer for clock input of CPU cluster.
- "intermediate" - A parent of "cpu" clock which is used as "intermediate" clock
- source (usually MAINPLL) when the original CPU PLL is under
- transition and not stable yet.
- Please refer to Documentation/devicetree/bindings/clock/clock-bindings.txt for
- generic clock consumer properties.
-- operating-points-v2: Please refer to Documentation/devicetree/bindings/opp/opp-v2.yaml
- for detail.
-- proc-supply: Regulator for Vproc of CPU cluster.
-
-Optional properties:
-- sram-supply: Regulator for Vsram of CPU cluster. When present, the cpufreq driver
- needs to do "voltage tracking" to step by step scale up/down Vproc and
- Vsram to fit SoC specific needs. When absent, the voltage scaling
- flow is handled by hardware, hence no software "voltage tracking" is
- needed.
-- mediatek,cci:
- Used to confirm the link status between cpufreq and mediatek cci. Because
- cpufreq and mediatek cci could share the same regulator in some MediaTek SoCs.
- To prevent the issue of high frequency and low voltage, we need to use this
- property to make sure mediatek cci is ready.
- For details of mediatek cci, please refer to
- Documentation/devicetree/bindings/interconnect/mediatek,cci.yaml
-- #cooling-cells:
- For details, please refer to
- Documentation/devicetree/bindings/thermal/thermal-cooling-devices.yaml
-
-Example 1 (MT7623 SoC):
-
- cpu_opp_table: opp_table {
- compatible = "operating-points-v2";
- opp-shared;
-
- opp-598000000 {
- opp-hz = /bits/ 64 <598000000>;
- opp-microvolt = <1050000>;
- };
-
- opp-747500000 {
- opp-hz = /bits/ 64 <747500000>;
- opp-microvolt = <1050000>;
- };
-
- opp-1040000000 {
- opp-hz = /bits/ 64 <1040000000>;
- opp-microvolt = <1150000>;
- };
-
- opp-1196000000 {
- opp-hz = /bits/ 64 <1196000000>;
- opp-microvolt = <1200000>;
- };
-
- opp-1300000000 {
- opp-hz = /bits/ 64 <1300000000>;
- opp-microvolt = <1300000>;
- };
- };
-
- cpu0: cpu@0 {
- device_type = "cpu";
- compatible = "arm,cortex-a7";
- reg = <0x0>;
- clocks = <&infracfg CLK_INFRA_CPUSEL>,
- <&apmixedsys CLK_APMIXED_MAINPLL>;
- clock-names = "cpu", "intermediate";
- operating-points-v2 = <&cpu_opp_table>;
- #cooling-cells = <2>;
- };
- cpu@1 {
- device_type = "cpu";
- compatible = "arm,cortex-a7";
- reg = <0x1>;
- operating-points-v2 = <&cpu_opp_table>;
- };
- cpu@2 {
- device_type = "cpu";
- compatible = "arm,cortex-a7";
- reg = <0x2>;
- operating-points-v2 = <&cpu_opp_table>;
- };
- cpu@3 {
- device_type = "cpu";
- compatible = "arm,cortex-a7";
- reg = <0x3>;
- operating-points-v2 = <&cpu_opp_table>;
- };
-
-Example 2 (MT8173 SoC):
- cpu_opp_table_a: opp_table_a {
- compatible = "operating-points-v2";
- opp-shared;
-
- opp-507000000 {
- opp-hz = /bits/ 64 <507000000>;
- opp-microvolt = <859000>;
- };
-
- opp-702000000 {
- opp-hz = /bits/ 64 <702000000>;
- opp-microvolt = <908000>;
- };
-
- opp-1001000000 {
- opp-hz = /bits/ 64 <1001000000>;
- opp-microvolt = <983000>;
- };
-
- opp-1105000000 {
- opp-hz = /bits/ 64 <1105000000>;
- opp-microvolt = <1009000>;
- };
-
- opp-1183000000 {
- opp-hz = /bits/ 64 <1183000000>;
- opp-microvolt = <1028000>;
- };
-
- opp-1404000000 {
- opp-hz = /bits/ 64 <1404000000>;
- opp-microvolt = <1083000>;
- };
-
- opp-1508000000 {
- opp-hz = /bits/ 64 <1508000000>;
- opp-microvolt = <1109000>;
- };
-
- opp-1573000000 {
- opp-hz = /bits/ 64 <1573000000>;
- opp-microvolt = <1125000>;
- };
- };
-
- cpu_opp_table_b: opp_table_b {
- compatible = "operating-points-v2";
- opp-shared;
-
- opp-507000000 {
- opp-hz = /bits/ 64 <507000000>;
- opp-microvolt = <828000>;
- };
-
- opp-702000000 {
- opp-hz = /bits/ 64 <702000000>;
- opp-microvolt = <867000>;
- };
-
- opp-1001000000 {
- opp-hz = /bits/ 64 <1001000000>;
- opp-microvolt = <927000>;
- };
-
- opp-1209000000 {
- opp-hz = /bits/ 64 <1209000000>;
- opp-microvolt = <968000>;
- };
-
- opp-1404000000 {
- opp-hz = /bits/ 64 <1007000000>;
- opp-microvolt = <1028000>;
- };
-
- opp-1612000000 {
- opp-hz = /bits/ 64 <1612000000>;
- opp-microvolt = <1049000>;
- };
-
- opp-1807000000 {
- opp-hz = /bits/ 64 <1807000000>;
- opp-microvolt = <1089000>;
- };
-
- opp-1989000000 {
- opp-hz = /bits/ 64 <1989000000>;
- opp-microvolt = <1125000>;
- };
- };
-
- cpu0: cpu@0 {
- device_type = "cpu";
- compatible = "arm,cortex-a53";
- reg = <0x000>;
- enable-method = "psci";
- cpu-idle-states = <&CPU_SLEEP_0>;
- clocks = <&infracfg CLK_INFRA_CA53SEL>,
- <&apmixedsys CLK_APMIXED_MAINPLL>;
- clock-names = "cpu", "intermediate";
- operating-points-v2 = <&cpu_opp_table_a>;
- };
-
- cpu1: cpu@1 {
- device_type = "cpu";
- compatible = "arm,cortex-a53";
- reg = <0x001>;
- enable-method = "psci";
- cpu-idle-states = <&CPU_SLEEP_0>;
- clocks = <&infracfg CLK_INFRA_CA53SEL>,
- <&apmixedsys CLK_APMIXED_MAINPLL>;
- clock-names = "cpu", "intermediate";
- operating-points-v2 = <&cpu_opp_table_a>;
- };
-
- cpu2: cpu@100 {
- device_type = "cpu";
- compatible = "arm,cortex-a72";
- reg = <0x100>;
- enable-method = "psci";
- cpu-idle-states = <&CPU_SLEEP_0>;
- clocks = <&infracfg CLK_INFRA_CA72SEL>,
- <&apmixedsys CLK_APMIXED_MAINPLL>;
- clock-names = "cpu", "intermediate";
- operating-points-v2 = <&cpu_opp_table_b>;
- };
-
- cpu3: cpu@101 {
- device_type = "cpu";
- compatible = "arm,cortex-a72";
- reg = <0x101>;
- enable-method = "psci";
- cpu-idle-states = <&CPU_SLEEP_0>;
- clocks = <&infracfg CLK_INFRA_CA72SEL>,
- <&apmixedsys CLK_APMIXED_MAINPLL>;
- clock-names = "cpu", "intermediate";
- operating-points-v2 = <&cpu_opp_table_b>;
- };
-
- &cpu0 {
- proc-supply = <&mt6397_vpca15_reg>;
- };
-
- &cpu1 {
- proc-supply = <&mt6397_vpca15_reg>;
- };
-
- &cpu2 {
- proc-supply = <&da9211_vcpu_reg>;
- sram-supply = <&mt6397_vsramca7_reg>;
- };
-
- &cpu3 {
- proc-supply = <&da9211_vcpu_reg>;
- sram-supply = <&mt6397_vsramca7_reg>;
- };
--
2.47.2
^ permalink raw reply related [flat|nested] 34+ messages in thread* Re: (subset) [PATCH v2 00/17] Arm cpu schema clean-ups
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (16 preceding siblings ...)
2025-04-10 15:47 ` [PATCH v2 17/17] dt-bindings: cpufreq: Drop redundant Mediatek binding Rob Herring (Arm)
@ 2025-04-11 8:37 ` Neil Armstrong
2025-04-11 15:44 ` Chen-Yu Tsai
` (3 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Neil Armstrong @ 2025-04-11 8:37 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Heiko Stuebner, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, Rob Herring (Arm)
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Andre Przywara, Philippe Mathieu-Daudé, Konrad Dybcio,
Sudeep Holla, Viresh Kumar, Ulf Hansson
Hi,
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...
>
> [...]
Thanks, Applied to https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git (v6.16/arm64-dt)
[11/17] arm64: dts: amlogic: Drop redundant CPU "clock-latency"
https://git.kernel.org/amlogic/c/4bc28af2da876531e5183d25ae807e608c816d18
These changes has been applied on the intermediate git tree [1].
The v6.16/arm64-dt branch will then be sent via a formal Pull Request to the Linux SoC maintainers
for inclusion in their intermediate git branches in order to be sent to Linus during
the next merge window, or sooner if it's a set of fixes.
In the cases of fixes, those will be merged in the current release candidate
kernel and as soon they appear on the Linux master branch they will be
backported to the previous Stable and Long-Stable kernels [2].
The intermediate git branches are merged daily in the linux-next tree [3],
people are encouraged testing these pre-release kernels and report issues on the
relevant mailing-lists.
If problems are discovered on those changes, please submit a signed-off-by revert
patch followed by a corrective changeset.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git
[2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
[3] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
--
Neil
^ permalink raw reply [flat|nested] 34+ messages in thread* Re: (subset) [PATCH v2 00/17] Arm cpu schema clean-ups
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (17 preceding siblings ...)
2025-04-11 8:37 ` (subset) [PATCH v2 00/17] Arm cpu schema clean-ups Neil Armstrong
@ 2025-04-11 15:44 ` Chen-Yu Tsai
2025-04-15 2:52 ` Bjorn Andersson
` (2 subsequent siblings)
21 siblings, 0 replies; 34+ messages in thread
From: Chen-Yu Tsai @ 2025-04-11 15:44 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Jernej Skrabec, Samuel Holland,
Conor Dooley, Nicolas Ferre, Claudiu Beznea, Steen Hegelund,
Daniel Machon, UNGLinuxDriver, Bjorn Andersson, Konrad Dybcio,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, Rob Herring (Arm)
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Andre Przywara, Philippe Mathieu-Daudé, Konrad Dybcio,
Sudeep Holla, Viresh Kumar, Ulf Hansson
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 to dt-for-6.16 in git@github.com:linux-sunxi/linux-sunxi.git, thanks!
[01/17] arm64: dts: allwinner: h5/h6: Drop spurious 'clock-latency-ns' properties
commit: 4df05f4a5fead4e5fc7e3c39cae74e5c0dc5282a
Best regards,
--
Chen-Yu Tsai <wens@csie.org>
^ permalink raw reply [flat|nested] 34+ messages in thread* Re: (subset) [PATCH v2 00/17] Arm cpu schema clean-ups
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (18 preceding siblings ...)
2025-04-11 15:44 ` Chen-Yu Tsai
@ 2025-04-15 2:52 ` Bjorn Andersson
2025-04-26 21:46 ` Heiko Stuebner
2025-05-07 16:38 ` Bjorn Andersson
21 siblings, 0 replies; 34+ messages in thread
From: Bjorn Andersson @ 2025-04-15 2:52 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Konrad Dybcio,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, Rob Herring (Arm)
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Andre Przywara, Philippe Mathieu-Daudé, Konrad Dybcio,
Sudeep Holla, Viresh Kumar, Ulf Hansson
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,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 34+ messages in thread* Re: (subset) [PATCH v2 00/17] Arm cpu schema clean-ups
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (19 preceding siblings ...)
2025-04-15 2:52 ` Bjorn Andersson
@ 2025-04-26 21:46 ` Heiko Stuebner
2025-05-07 16:38 ` Bjorn Andersson
21 siblings, 0 replies; 34+ messages in thread
From: Heiko Stuebner @ 2025-04-26 21:46 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Bjorn Andersson,
Konrad Dybcio, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, Rob Herring (Arm)
Cc: Heiko Stuebner, devicetree, linux-arm-kernel, linux-sunxi,
linux-kernel, linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Andre Przywara, Philippe Mathieu-Daudé, Konrad Dybcio,
Sudeep Holla, Viresh Kumar, Ulf Hansson
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,
--
Heiko Stuebner <heiko@sntech.de>
^ permalink raw reply [flat|nested] 34+ messages in thread* Re: (subset) [PATCH v2 00/17] Arm cpu schema clean-ups
2025-04-10 15:47 [PATCH v2 00/17] Arm cpu schema clean-ups Rob Herring (Arm)
` (20 preceding siblings ...)
2025-04-26 21:46 ` Heiko Stuebner
@ 2025-05-07 16:38 ` Bjorn Andersson
21 siblings, 0 replies; 34+ messages in thread
From: Bjorn Andersson @ 2025-05-07 16:38 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Conor Dooley, Nicolas Ferre, Claudiu Beznea,
Steen Hegelund, Daniel Machon, UNGLinuxDriver, Konrad Dybcio,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Heiko Stuebner, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Geert Uytterhoeven, Magnus Damm,
Lorenzo Pieralisi, Andy Gross, Thomas Bogendoerfer, Viresh Kumar,
Nishanth Menon, Stephen Boyd, zhouyanjie, Matthias Brugger,
AngeloGioacchino Del Regno, Rafael J. Wysocki, Dmitry Baryshkov,
Stephan Gerhold, Rob Herring (Arm)
Cc: devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-arm-msm, imx, linux-rockchip, linux-amlogic,
linux-renesas-soc, linux-mips, linux-pm, linux-mediatek,
Andre Przywara, Philippe Mathieu-Daudé, Konrad Dybcio,
Sudeep Holla, Viresh Kumar, Ulf Hansson
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!
[09/17] arm: dts: qcom: ipq4019: Drop redundant CPU "clock-latency"
commit: 3ea267124573f24e67f0fe47c4a865f0f283f8fc
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 34+ messages in thread