* [PATCH v2 01/17] arm64: dts: allwinner: h5/h6: Drop spurious 'clock-latency-ns' properties
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
@ 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)
` (11 subsequent siblings)
12 siblings, 0 replies; 22+ 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] 22+ messages in thread* [PATCH v2 02/17] arm64: dts: broadcom: bcm2712: Use "l2-cache" for L2 cache node names
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
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-10 15:47 ` [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies Rob Herring (Arm)
` (10 subsequent siblings)
12 siblings, 0 replies; 22+ 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] 22+ messages in thread* [PATCH v2 03/17] arm64: dts: microchip: sparx5: Fix CPU node "enable-method" property dependencies
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
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)
` (9 subsequent siblings)
12 siblings, 1 reply; 22+ 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] 22+ 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; 22+ 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] 22+ 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; 22+ 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] 22+ 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; 22+ 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] 22+ messages in thread
* [PATCH v2 04/17] arm64: dts: qcom: qdu1000: Fix qcom,freq-domain
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
` (8 subsequent siblings)
12 siblings, 0 replies; 22+ 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] 22+ messages in thread* [PATCH v2 05/17] arm64: dts: qcom: msm8939: Fix CPU node "enable-method" property dependencies
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
` (7 subsequent siblings)
12 siblings, 1 reply; 22+ 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] 22+ 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; 22+ 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] 22+ messages in thread
* [PATCH v2 06/17] arm64: dts: qcom: msm8992-lg-h815: Fix CPU node "enable-method" property dependencies
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
` (6 subsequent siblings)
12 siblings, 1 reply; 22+ 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] 22+ 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; 22+ 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] 22+ messages in thread
* [PATCH v2 07/17] arm: dts: qcom: sdx55/sdx65: Fix CPU power-domain-names
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
` (5 subsequent siblings)
12 siblings, 1 reply; 22+ 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] 22+ 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; 22+ 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] 22+ messages in thread
* [PATCH v2 08/17] arm/arm64: dts: imx: Drop redundant CPU "clock-latency"
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
` (4 subsequent siblings)
12 siblings, 1 reply; 22+ 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] 22+ 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; 22+ 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] 22+ messages in thread
* [PATCH v2 09/17] arm: dts: qcom: ipq4019: Drop redundant CPU "clock-latency"
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
` (3 subsequent siblings)
12 siblings, 0 replies; 22+ 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] 22+ messages in thread* [PATCH v2 10/17] arm: dts: rockchip: Drop redundant CPU "clock-latency"
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
` (2 subsequent siblings)
12 siblings, 0 replies; 22+ 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] 22+ messages in thread* [PATCH v2 11/17] arm64: dts: amlogic: Drop redundant CPU "clock-latency"
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
2025-04-10 15:47 ` [PATCH v2 13/17] dt-bindings: arm/cpus: Re-wrap 'description' entries Rob Herring (Arm)
12 siblings, 1 reply; 22+ 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] 22+ 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; 22+ 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] 22+ 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; 22+ 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] 22+ messages in thread
* [PATCH v2 12/17] dt-bindings: arm/cpus: Add schemas for "enable-method" dependencies
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
12 siblings, 0 replies; 22+ 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] 22+ messages in thread* [PATCH v2 13/17] dt-bindings: arm/cpus: Re-wrap 'description' entries
[not found] <20250410-dt-cpu-schema-v2-0-63d7dc9ddd0a@kernel.org>
` (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)
12 siblings, 0 replies; 22+ 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] 22+ messages in thread