devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] arm64: dts: renesas: r9a09g077: Fixup RZ/T2H SoC DTSI
@ 2025-06-13 13:56 Prabhakar
  2025-06-13 13:56 ` [PATCH 1/3] arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from timer node Prabhakar
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Prabhakar @ 2025-06-13 13:56 UTC (permalink / raw)
  To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley
  Cc: linux-renesas-soc, devicetree, linux-kernel, Prabhakar, Biju Das,
	Fabrizio Castro, Lad Prabhakar

From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Hi All,

This patch series addresses a few issues in the RZ/T2H SoC Device Tree,
patches apply on top of v10 series [1] which is yet to be merged.

@Geert, feel free to squash them in the original series [1] if you prefer.

[1] https://lore.kernel.org/all/20250523142417.2840797-1-thierry.bultel.yh@bp.renesas.com/

Cheers,
Prabhakar

Lad Prabhakar (3):
  arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from
    timer node
  arm64: dts: renesas: r9a09g077: Add missing hypervisor virtual timer
    IRQ
  arm64: dts: renesas: r9a09g077: Sort extal_clk node by name

 arch/arm64/boot/dts/renesas/r9a09g077.dtsi | 24 ++++++++++++----------
 1 file changed, 13 insertions(+), 11 deletions(-)

-- 
2.49.0


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

* [PATCH 1/3] arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from timer node
  2025-06-13 13:56 [PATCH 0/3] arm64: dts: renesas: r9a09g077: Fixup RZ/T2H SoC DTSI Prabhakar
@ 2025-06-13 13:56 ` Prabhakar
  2025-06-19 12:25   ` Geert Uytterhoeven
  2025-06-13 13:56 ` [PATCH 2/3] arm64: dts: renesas: r9a09g077: Add missing hypervisor virtual timer IRQ Prabhakar
  2025-06-13 13:56 ` [PATCH 3/3] arm64: dts: renesas: r9a09g077: Sort extal_clk node by name Prabhakar
  2 siblings, 1 reply; 7+ messages in thread
From: Prabhakar @ 2025-06-13 13:56 UTC (permalink / raw)
  To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley
  Cc: linux-renesas-soc, devicetree, linux-kernel, Prabhakar, Biju Das,
	Fabrizio Castro, Lad Prabhakar

From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

The RZ/T2H SoC uses GICv3, which does not require a CPU mask in the
`interrupts-extended` property of the ARMv8 timer node. The CPU mask
macro `GIC_CPU_MASK_SIMPLE()` is only applicable to pre-GICv3 systems.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 arch/arm64/boot/dts/renesas/r9a09g077.dtsi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/r9a09g077.dtsi b/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
index b531e393d8c4..c756a7c3cda2 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
@@ -274,9 +274,9 @@ sdhi1_vqmmc: vqmmc-regulator {
 
 	timer {
 		compatible = "arm,armv8-timer";
-		interrupts-extended = <&gic GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
-				      <&gic GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
-				      <&gic GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
-				      <&gic GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>;
+		interrupts-extended = <&gic GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>,
+				      <&gic GIC_PPI 14 IRQ_TYPE_LEVEL_LOW>,
+				      <&gic GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>,
+				      <&gic GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>;
 	};
 };
-- 
2.49.0


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

* [PATCH 2/3] arm64: dts: renesas: r9a09g077: Add missing hypervisor virtual timer IRQ
  2025-06-13 13:56 [PATCH 0/3] arm64: dts: renesas: r9a09g077: Fixup RZ/T2H SoC DTSI Prabhakar
  2025-06-13 13:56 ` [PATCH 1/3] arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from timer node Prabhakar
@ 2025-06-13 13:56 ` Prabhakar
  2025-06-19 12:28   ` Geert Uytterhoeven
  2025-06-13 13:56 ` [PATCH 3/3] arm64: dts: renesas: r9a09g077: Sort extal_clk node by name Prabhakar
  2 siblings, 1 reply; 7+ messages in thread
From: Prabhakar @ 2025-06-13 13:56 UTC (permalink / raw)
  To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley
  Cc: linux-renesas-soc, devicetree, linux-kernel, Prabhakar, Biju Das,
	Fabrizio Castro, Lad Prabhakar

From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Add the missing fifth interrupt to the device node that represents the
ARM architected timer.  While at it, add an interrupt-names property for
clarity.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 arch/arm64/boot/dts/renesas/r9a09g077.dtsi | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r9a09g077.dtsi b/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
index c756a7c3cda2..7a79db19aac6 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
@@ -277,6 +277,8 @@ timer {
 		interrupts-extended = <&gic GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>,
 				      <&gic GIC_PPI 14 IRQ_TYPE_LEVEL_LOW>,
 				      <&gic GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>,
-				      <&gic GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>;
+				      <&gic GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>,
+				      <&gic GIC_PPI 12 IRQ_TYPE_LEVEL_LOW>;
+		interrupt-names = "sec-phys", "phys", "virt", "hyp-phys", "hyp-virt";
 	};
 };
-- 
2.49.0


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

* [PATCH 3/3] arm64: dts: renesas: r9a09g077: Sort extal_clk node by name
  2025-06-13 13:56 [PATCH 0/3] arm64: dts: renesas: r9a09g077: Fixup RZ/T2H SoC DTSI Prabhakar
  2025-06-13 13:56 ` [PATCH 1/3] arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from timer node Prabhakar
  2025-06-13 13:56 ` [PATCH 2/3] arm64: dts: renesas: r9a09g077: Add missing hypervisor virtual timer IRQ Prabhakar
@ 2025-06-13 13:56 ` Prabhakar
  2025-06-19 12:29   ` Geert Uytterhoeven
  2 siblings, 1 reply; 7+ messages in thread
From: Prabhakar @ 2025-06-13 13:56 UTC (permalink / raw)
  To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley
  Cc: linux-renesas-soc, devicetree, linux-kernel, Prabhakar, Biju Das,
	Fabrizio Castro, Lad Prabhakar

From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Reorder the `extal_clk` node in the RZ/T2H SoC DTSI to maintain
consistent sorting by node name.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 arch/arm64/boot/dts/renesas/r9a09g077.dtsi | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/r9a09g077.dtsi b/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
index 7a79db19aac6..1508f581cb66 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a09g077.dtsi
@@ -13,13 +13,6 @@ / {
 	#address-cells = <2>;
 	#size-cells = <2>;
 
-	extal_clk: extal {
-		compatible = "fixed-clock";
-		#clock-cells = <0>;
-		/* This value must be overridden by the board */
-		clock-frequency = <0>;
-	};
-
 	cpus {
 		#address-cells = <1>;
 		#size-cells = <0>;
@@ -64,6 +57,13 @@ cpu3: cpu@300 {
 		};
 	};
 
+	extal_clk: extal {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		/* This value must be overridden by the board */
+		clock-frequency = <0>;
+	};
+
 	psci {
 		compatible = "arm,psci-1.0", "arm,psci-0.2";
 		method = "smc";
-- 
2.49.0


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

* Re: [PATCH 1/3] arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from timer node
  2025-06-13 13:56 ` [PATCH 1/3] arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from timer node Prabhakar
@ 2025-06-19 12:25   ` Geert Uytterhoeven
  0 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2025-06-19 12:25 UTC (permalink / raw)
  To: Prabhakar
  Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	linux-renesas-soc, devicetree, linux-kernel, Biju Das,
	Fabrizio Castro, Lad Prabhakar

On Fri, 13 Jun 2025 at 17:26, Prabhakar <prabhakar.csengg@gmail.com> wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>
> The RZ/T2H SoC uses GICv3, which does not require a CPU mask in the
> `interrupts-extended` property of the ARMv8 timer node. The CPU mask
> macro `GIC_CPU_MASK_SIMPLE()` is only applicable to pre-GICv3 systems.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

JFTR
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH 2/3] arm64: dts: renesas: r9a09g077: Add missing hypervisor virtual timer IRQ
  2025-06-13 13:56 ` [PATCH 2/3] arm64: dts: renesas: r9a09g077: Add missing hypervisor virtual timer IRQ Prabhakar
@ 2025-06-19 12:28   ` Geert Uytterhoeven
  0 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2025-06-19 12:28 UTC (permalink / raw)
  To: Prabhakar
  Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	linux-renesas-soc, devicetree, linux-kernel, Biju Das,
	Fabrizio Castro, Lad Prabhakar

On Fri, 13 Jun 2025 at 17:15, Prabhakar <prabhakar.csengg@gmail.com> wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>
> Add the missing fifth interrupt to the device node that represents the
> ARM architected timer.  While at it, add an interrupt-names property for
> clarity.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

JFTR
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH 3/3] arm64: dts: renesas: r9a09g077: Sort extal_clk node by name
  2025-06-13 13:56 ` [PATCH 3/3] arm64: dts: renesas: r9a09g077: Sort extal_clk node by name Prabhakar
@ 2025-06-19 12:29   ` Geert Uytterhoeven
  0 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2025-06-19 12:29 UTC (permalink / raw)
  To: Prabhakar
  Cc: Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	linux-renesas-soc, devicetree, linux-kernel, Biju Das,
	Fabrizio Castro, Lad Prabhakar

On Fri, 13 Jun 2025 at 17:41, Prabhakar <prabhakar.csengg@gmail.com> wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>
> Reorder the `extal_clk` node in the RZ/T2H SoC DTSI to maintain
> consistent sorting by node name.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

JFTR
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

end of thread, other threads:[~2025-06-19 12:29 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-13 13:56 [PATCH 0/3] arm64: dts: renesas: r9a09g077: Fixup RZ/T2H SoC DTSI Prabhakar
2025-06-13 13:56 ` [PATCH 1/3] arm64: dts: renesas: r9a09g077: Remove GIC_CPU_MASK_SIMPLE() from timer node Prabhakar
2025-06-19 12:25   ` Geert Uytterhoeven
2025-06-13 13:56 ` [PATCH 2/3] arm64: dts: renesas: r9a09g077: Add missing hypervisor virtual timer IRQ Prabhakar
2025-06-19 12:28   ` Geert Uytterhoeven
2025-06-13 13:56 ` [PATCH 3/3] arm64: dts: renesas: r9a09g077: Sort extal_clk node by name Prabhakar
2025-06-19 12:29   ` Geert Uytterhoeven

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).