Linux SOC development
 help / color / mirror / Atom feed
* [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs
@ 2025-04-28 12:06 Alexandre Torgue
  2025-04-28 12:06 ` Alexandre Torgue
                   ` (7 more replies)
  0 siblings, 8 replies; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:06 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

Dear Arm maintainers,

Please consider this first round of STM32 DT fixes for v6.15.

Thanks
Alex

Original cover-letter message:

Fix GIC compatible and register access for STM32MP2
This serie replaces the original stm32mp25 patch:
 "arm64: dts: st: Adjust interrupt-controller for aarch64"

Christian Bruel (6):
  arm64: dts: st: Adjust interrupt-controller for stm32mp25 SoCs
  arm64: dts: st: Use 128kB size for aliased GIC400 register access on
    stm32mp25 SoCs
  arm64: dts: st: Adjust interrupt-controller for stm32mp21 SoCs
  arm64: dts: st: Use 128kB size for aliased GIC400 register access on
    stm32mp21 SoCs
  arm64: dts: st: Adjust interrupt-controller for stm32mp23 SoCs
  arm64: dts: st: Use 128kB size for aliased GIC400 register access on
    stm32mp23 SoCs

 arch/arm64/boot/dts/st/stm32mp211.dtsi | 8 ++++----
 arch/arm64/boot/dts/st/stm32mp231.dtsi | 9 ++++-----
 arch/arm64/boot/dts/st/stm32mp251.dtsi | 9 ++++-----
 3 files changed, 12 insertions(+), 14 deletions(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
@ 2025-04-28 12:06 ` Alexandre Torgue
  2025-04-29 16:43   ` patchwork-bot+linux-soc
  2025-04-28 12:06 ` [PATCH 1/6] arm64: dts: st: Adjust interrupt-controller for stm32mp25 SoCs Alexandre Torgue
                   ` (6 subsequent siblings)
  7 siblings, 1 reply; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:06 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

Dear Arm maintainers,

Please consider this first round of STM32 DT fixes for v6.15.

Thanks
Alex

Original cover-letter message:

Fix GIC compatible and register access for STM32MP2
This serie replaces the original stm32mp25 patch:
 "arm64: dts: st: Adjust interrupt-controller for aarch64"

Christian Bruel (6):
  arm64: dts: st: Adjust interrupt-controller for stm32mp25 SoCs
  arm64: dts: st: Use 128kB size for aliased GIC400 register access on
    stm32mp25 SoCs
  arm64: dts: st: Adjust interrupt-controller for stm32mp21 SoCs
  arm64: dts: st: Use 128kB size for aliased GIC400 register access on
    stm32mp21 SoCs
  arm64: dts: st: Adjust interrupt-controller for stm32mp23 SoCs
  arm64: dts: st: Use 128kB size for aliased GIC400 register access on
    stm32mp23 SoCs

 arch/arm64/boot/dts/st/stm32mp211.dtsi | 8 ++++----
 arch/arm64/boot/dts/st/stm32mp231.dtsi | 9 ++++-----
 arch/arm64/boot/dts/st/stm32mp251.dtsi | 9 ++++-----
 3 files changed, 12 insertions(+), 14 deletions(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH 1/6] arm64: dts: st: Adjust interrupt-controller for stm32mp25 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
  2025-04-28 12:06 ` Alexandre Torgue
@ 2025-04-28 12:06 ` Alexandre Torgue
  2025-04-28 12:06 ` [PATCH 2/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:06 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

From: Christian Bruel <christian.bruel@foss.st.com>

Use gic-400 compatible and remove address-cells = <1> on aarch64

Fixes: 5d30d03aaf785 ("arm64: dts: st: introduce stm32mp25 SoCs family")
Signed-off-by: Christian Bruel <christian.bruel@foss.st.com>
Link: https://lore.kernel.org/r/20250415111654.2103767-2-christian.bruel@foss.st.com
Signed-off-by: Alexandre Torgue <alexandre.torgue@foss.st.com>

diff --git a/arch/arm64/boot/dts/st/stm32mp251.dtsi b/arch/arm64/boot/dts/st/stm32mp251.dtsi
index f3c6cdfd7008..379e290313dc 100644
--- a/arch/arm64/boot/dts/st/stm32mp251.dtsi
+++ b/arch/arm64/boot/dts/st/stm32mp251.dtsi
@@ -115,9 +115,8 @@ scmi_vdda18adc: regulator@7 {
 	};
 
 	intc: interrupt-controller@4ac00000 {
-		compatible = "arm,cortex-a7-gic";
+		compatible = "arm,gic-400";
 		#interrupt-cells = <3>;
-		#address-cells = <1>;
 		interrupt-controller;
 		reg = <0x0 0x4ac10000 0x0 0x1000>,
 		      <0x0 0x4ac20000 0x0 0x2000>,
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 2/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on stm32mp25 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
  2025-04-28 12:06 ` Alexandre Torgue
  2025-04-28 12:06 ` [PATCH 1/6] arm64: dts: st: Adjust interrupt-controller for stm32mp25 SoCs Alexandre Torgue
@ 2025-04-28 12:06 ` Alexandre Torgue
  2025-04-28 12:07 ` [PATCH 3/6] arm64: dts: st: Adjust interrupt-controller for stm32mp21 SoCs Alexandre Torgue
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:06 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

From: Christian Bruel <christian.bruel@foss.st.com>

Adjust the size of 8kB GIC regions to 128kB so that each 4kB is mapped 16
times over a 64kB region.
The offset is then adjusted in the irq-gic driver.

see commit 12e14066f4835 ("irqchip/GIC: Add workaround for aliased GIC400")

Fixes: 5d30d03aaf785 ("arm64: dts: st: introduce stm32mp25 SoCs family")
Suggested-by: Marc Zyngier <maz@kernel.org>
Signed-off-by: Christian Bruel <christian.bruel@foss.st.com>
Acked-by: Marc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20250415111654.2103767-3-christian.bruel@foss.st.com
Signed-off-by: Alexandre Torgue <alexandre.torgue@foss.st.com>

diff --git a/arch/arm64/boot/dts/st/stm32mp251.dtsi b/arch/arm64/boot/dts/st/stm32mp251.dtsi
index 379e290313dc..87110f91e489 100644
--- a/arch/arm64/boot/dts/st/stm32mp251.dtsi
+++ b/arch/arm64/boot/dts/st/stm32mp251.dtsi
@@ -119,9 +119,9 @@ intc: interrupt-controller@4ac00000 {
 		#interrupt-cells = <3>;
 		interrupt-controller;
 		reg = <0x0 0x4ac10000 0x0 0x1000>,
-		      <0x0 0x4ac20000 0x0 0x2000>,
-		      <0x0 0x4ac40000 0x0 0x2000>,
-		      <0x0 0x4ac60000 0x0 0x2000>;
+		      <0x0 0x4ac20000 0x0 0x20000>,
+		      <0x0 0x4ac40000 0x0 0x20000>,
+		      <0x0 0x4ac60000 0x0 0x20000>;
 	};
 
 	psci {
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 3/6] arm64: dts: st: Adjust interrupt-controller for stm32mp21 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
                   ` (2 preceding siblings ...)
  2025-04-28 12:06 ` [PATCH 2/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
@ 2025-04-28 12:07 ` Alexandre Torgue
  2025-04-28 12:07 ` [PATCH 4/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:07 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

From: Christian Bruel <christian.bruel@foss.st.com>

Use gic-400 compatible for aarch64

Fixes: 7a57b1bb1afbf ("arm64: dts: st: introduce stm32mp21 SoCs family")
Signed-off-by: Christian Bruel <christian.bruel@foss.st.com>
Link: https://lore.kernel.org/r/20250415111654.2103767-4-christian.bruel@foss.st.com
Signed-off-by: Alexandre Torgue <alexandre.torgue@foss.st.com>

diff --git a/arch/arm64/boot/dts/st/stm32mp211.dtsi b/arch/arm64/boot/dts/st/stm32mp211.dtsi
index 6dd1377f3e1d..52a8209471b8 100644
--- a/arch/arm64/boot/dts/st/stm32mp211.dtsi
+++ b/arch/arm64/boot/dts/st/stm32mp211.dtsi
@@ -116,7 +116,7 @@ syscfg: syscon@44230000 {
 		};
 
 		intc: interrupt-controller@4ac10000 {
-			compatible = "arm,cortex-a7-gic";
+			compatible = "arm,gic-400";
 			reg = <0x4ac10000 0x0 0x1000>,
 			      <0x4ac20000 0x0 0x2000>,
 			      <0x4ac40000 0x0 0x2000>,
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 4/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on stm32mp21 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
                   ` (3 preceding siblings ...)
  2025-04-28 12:07 ` [PATCH 3/6] arm64: dts: st: Adjust interrupt-controller for stm32mp21 SoCs Alexandre Torgue
@ 2025-04-28 12:07 ` Alexandre Torgue
  2025-04-28 12:07 ` [PATCH 5/6] arm64: dts: st: Adjust interrupt-controller for stm32mp23 SoCs Alexandre Torgue
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:07 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

From: Christian Bruel <christian.bruel@foss.st.com>

Adjust the size of 8kB GIC regions to 128kB so that each 4kB is mapped
16 times over a 64kB region.
The offset is then adjusted in the irq-gic driver.

see commit 12e14066f4835 ("irqchip/GIC: Add workaround for aliased GIC400")

Fixes: 7a57b1bb1afbf ("arm64: dts: st: introduce stm32mp21 SoCs family")
Suggested-by: Marc Zyngier <maz@kernel.org>
Signed-off-by: Christian Bruel <christian.bruel@foss.st.com>
Link: https://lore.kernel.org/r/20250415111654.2103767-5-christian.bruel@foss.st.com
Signed-off-by: Alexandre Torgue <alexandre.torgue@foss.st.com>

diff --git a/arch/arm64/boot/dts/st/stm32mp211.dtsi b/arch/arm64/boot/dts/st/stm32mp211.dtsi
index 52a8209471b8..bf888d60cd4f 100644
--- a/arch/arm64/boot/dts/st/stm32mp211.dtsi
+++ b/arch/arm64/boot/dts/st/stm32mp211.dtsi
@@ -118,9 +118,9 @@ syscfg: syscon@44230000 {
 		intc: interrupt-controller@4ac10000 {
 			compatible = "arm,gic-400";
 			reg = <0x4ac10000 0x0 0x1000>,
-			      <0x4ac20000 0x0 0x2000>,
-			      <0x4ac40000 0x0 0x2000>,
-			      <0x4ac60000 0x0 0x2000>;
+			      <0x4ac20000 0x0 0x20000>,
+			      <0x4ac40000 0x0 0x20000>,
+			      <0x4ac60000 0x0 0x20000>;
 			      #interrupt-cells = <3>;
 			      interrupt-controller;
 		};
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 5/6] arm64: dts: st: Adjust interrupt-controller for stm32mp23 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
                   ` (4 preceding siblings ...)
  2025-04-28 12:07 ` [PATCH 4/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
@ 2025-04-28 12:07 ` Alexandre Torgue
  2025-04-28 12:07 ` [PATCH 6/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
  2025-04-29 16:43 ` [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs patchwork-bot+linux-soc
  7 siblings, 0 replies; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:07 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

From: Christian Bruel <christian.bruel@foss.st.com>

Use gic-400 compatible and remove address-cells = <1> for aarch64

Fixes: e9b03ef21386e ("arm64: dts: st: introduce stm32mp23 SoCs family")
Signed-off-by: Christian Bruel <christian.bruel@foss.st.com>
Link: https://lore.kernel.org/r/20250415111654.2103767-6-christian.bruel@foss.st.com
Signed-off-by: Alexandre Torgue <alexandre.torgue@foss.st.com>

diff --git a/arch/arm64/boot/dts/st/stm32mp231.dtsi b/arch/arm64/boot/dts/st/stm32mp231.dtsi
index 8820d219a33e..3f73cf1d443d 100644
--- a/arch/arm64/boot/dts/st/stm32mp231.dtsi
+++ b/arch/arm64/boot/dts/st/stm32mp231.dtsi
@@ -1201,13 +1201,12 @@ exti2: interrupt-controller@46230000 {
 		};
 
 		intc: interrupt-controller@4ac10000 {
-			compatible = "arm,cortex-a7-gic";
+			compatible = "arm,gic-400";
 			reg = <0x4ac10000 0x1000>,
 			      <0x4ac20000 0x2000>,
 			      <0x4ac40000 0x2000>,
 			      <0x4ac60000 0x2000>;
 			#interrupt-cells = <3>;
-			#address-cells = <1>;
 			interrupt-controller;
 		};
 	};
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 6/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on stm32mp23 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
                   ` (5 preceding siblings ...)
  2025-04-28 12:07 ` [PATCH 5/6] arm64: dts: st: Adjust interrupt-controller for stm32mp23 SoCs Alexandre Torgue
@ 2025-04-28 12:07 ` Alexandre Torgue
  2025-04-29 16:43 ` [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs patchwork-bot+linux-soc
  7 siblings, 0 replies; 10+ messages in thread
From: Alexandre Torgue @ 2025-04-28 12:07 UTC (permalink / raw)
  To: Arnd Bergmann, Kevin Hilman, soc, arm; +Cc: Alexandre Torgue

From: Christian Bruel <christian.bruel@foss.st.com>

Adjust the size of 8kB GIC regions to 128kB so that each 4kB is mapped 16
times over a 64kB region.
The offset is then adjusted in the irq-gic driver.

see commit 12e14066f4835 ("irqchip/GIC: Add workaround for aliased GIC400")

Fixes: e9b03ef21386e ("arm64: dts: st: introduce stm32mp23 SoCs family")
Suggested-by: Marc Zyngier <maz@kernel.org>
Signed-off-by: Christian Bruel <christian.bruel@foss.st.com>
Link: https://lore.kernel.org/r/20250415111654.2103767-7-christian.bruel@foss.st.com
Signed-off-by: Alexandre Torgue <alexandre.torgue@foss.st.com>

diff --git a/arch/arm64/boot/dts/st/stm32mp231.dtsi b/arch/arm64/boot/dts/st/stm32mp231.dtsi
index 3f73cf1d443d..75697acd1345 100644
--- a/arch/arm64/boot/dts/st/stm32mp231.dtsi
+++ b/arch/arm64/boot/dts/st/stm32mp231.dtsi
@@ -1203,9 +1203,9 @@ exti2: interrupt-controller@46230000 {
 		intc: interrupt-controller@4ac10000 {
 			compatible = "arm,gic-400";
 			reg = <0x4ac10000 0x1000>,
-			      <0x4ac20000 0x2000>,
-			      <0x4ac40000 0x2000>,
-			      <0x4ac60000 0x2000>;
+			      <0x4ac20000 0x20000>,
+			      <0x4ac40000 0x20000>,
+			      <0x4ac60000 0x20000>;
 			#interrupt-cells = <3>;
 			interrupt-controller;
 		};
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs
  2025-04-28 12:06 ` Alexandre Torgue
@ 2025-04-29 16:43   ` patchwork-bot+linux-soc
  0 siblings, 0 replies; 10+ messages in thread
From: patchwork-bot+linux-soc @ 2025-04-29 16:43 UTC (permalink / raw)
  To: Alexandre Torgue; +Cc: soc

Hello:

This series was applied to soc/soc.git (arm/fixes)
by Arnd Bergmann <arnd@arndb.de>:

On Mon, 28 Apr 2025 14:06:57 +0200 you wrote:
> Dear Arm maintainers,
> 
> Please consider this first round of STM32 DT fixes for v6.15.
> 
> Thanks
> Alex
> 
> [...]

Here is the summary with links:
  - [1/6] arm64: dts: st: Adjust interrupt-controller for stm32mp25 SoCs
    https://git.kernel.org/soc/soc/c/de2b2107d5a4
  - [2/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on stm32mp25 SoCs
    https://git.kernel.org/soc/soc/c/06c231fe953a
  - [3/6] arm64: dts: st: Adjust interrupt-controller for stm32mp21 SoCs
    https://git.kernel.org/soc/soc/c/02dc83f09c72

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs
  2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
                   ` (6 preceding siblings ...)
  2025-04-28 12:07 ` [PATCH 6/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
@ 2025-04-29 16:43 ` patchwork-bot+linux-soc
  7 siblings, 0 replies; 10+ messages in thread
From: patchwork-bot+linux-soc @ 2025-04-29 16:43 UTC (permalink / raw)
  To: Alexandre Torgue; +Cc: soc

Hello:

This series was applied to soc/soc.git (arm/fixes)
by Arnd Bergmann <arnd@arndb.de>:

On Mon, 28 Apr 2025 14:06:56 +0200 you wrote:
> Dear Arm maintainers,
> 
> Please consider this first round of STM32 DT fixes for v6.15.
> 
> Thanks
> Alex
> 
> [...]

Here is the summary with links:
  - [4/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on stm32mp21 SoCs
    https://git.kernel.org/soc/soc/c/1bc229e9bb9c
  - [5/6] arm64: dts: st: Adjust interrupt-controller for stm32mp23 SoCs
    https://git.kernel.org/soc/soc/c/3a1e1082097b
  - [6/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on stm32mp23 SoCs
    https://git.kernel.org/soc/soc/c/2ef5c66cba61

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2025-04-29 16:43 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-28 12:06 [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs Alexandre Torgue
2025-04-28 12:06 ` Alexandre Torgue
2025-04-29 16:43   ` patchwork-bot+linux-soc
2025-04-28 12:06 ` [PATCH 1/6] arm64: dts: st: Adjust interrupt-controller for stm32mp25 SoCs Alexandre Torgue
2025-04-28 12:06 ` [PATCH 2/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
2025-04-28 12:07 ` [PATCH 3/6] arm64: dts: st: Adjust interrupt-controller for stm32mp21 SoCs Alexandre Torgue
2025-04-28 12:07 ` [PATCH 4/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
2025-04-28 12:07 ` [PATCH 5/6] arm64: dts: st: Adjust interrupt-controller for stm32mp23 SoCs Alexandre Torgue
2025-04-28 12:07 ` [PATCH 6/6] arm64: dts: st: Use 128kB size for aliased GIC400 register access on " Alexandre Torgue
2025-04-29 16:43 ` [PATCH 0/6] Fix interrupt controller node for STM32MP2 SoCs patchwork-bot+linux-soc

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox