* [PATCH 0/6] Add RZ/G2UL MTU3a support
@ 2023-07-20 13:10 Biju Das
2023-07-20 13:10 ` [PATCH 1/6] dt-bindings: timer: renesas,rz-mtu3: Improve documentation Biju Das
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Biju Das @ 2023-07-20 13:10 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: Biju Das, Daniel Lezcano, Thomas Gleixner, Geert Uytterhoeven,
Magnus Damm, linux-iio, linux-renesas-soc, devicetree,
Fabrizio Castro, Prabhakar Mahadev Lad
This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
Biju Das (6):
dt-bindings: timer: renesas,rz-mtu3: Improve documentation
dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
names
dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC
arm64: dts: renesas: r9a07g044: Update overfow/underflow IRQ names for
MTU3 channels
arm64: dts: renesas: r9a07g043: Add MTU3a node
arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3
.../bindings/timer/renesas,rz-mtu3.yaml | 67 +++++++++---------
arch/arm64/boot/dts/renesas/r9a07g043.dtsi | 70 +++++++++++++++++++
.../boot/dts/renesas/r9a07g043u11-smarc.dts | 11 +++
arch/arm64/boot/dts/renesas/r9a07g044.dtsi | 16 ++---
arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 16 ++---
.../dts/renesas/rzg2ul-smarc-pinfunction.dtsi | 6 ++
arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi | 13 ++++
7 files changed, 150 insertions(+), 49 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 1/6] dt-bindings: timer: renesas,rz-mtu3: Improve documentation
2023-07-20 13:10 [PATCH 0/6] Add RZ/G2UL MTU3a support Biju Das
@ 2023-07-20 13:10 ` Biju Das
2023-07-20 13:10 ` [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names Biju Das
2023-07-20 13:10 ` [PATCH 3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC Biju Das
2 siblings, 0 replies; 11+ messages in thread
From: Biju Das @ 2023-07-20 13:10 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: Biju Das, Daniel Lezcano, Thomas Gleixner, Geert Uytterhoeven,
Magnus Damm, linux-iio, linux-renesas-soc, devicetree,
Fabrizio Castro, Prabhakar Mahadev Lad, Pavel Machek,
Conor Dooley
Fix the documentation issues pointed by Pavel while backporting
it to 6.1.y-cip.
- Replace '32- bit'->'32-bit'
- Consistently remove '.' at the end of line for the specifications
- Replace ' (excluding MTU8)'-> '(excluding MTU8)'
Reported-by: Pavel Machek <pavel@denx.de>
Closes: https://patchwork.kernel.org/project/cip-dev/patch/20230606075235.183132-3-biju.das.jz@bp.renesas.com/
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
---
v1->v2:
* Added Ack by Conor Dooley
---
.../bindings/timer/renesas,rz-mtu3.yaml | 28 +++++++++----------
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
index bffdab0b0185..eb2d5ebe4df0 100644
--- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
+++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
@@ -11,8 +11,8 @@ maintainers:
description: |
This hardware block consists of eight 16-bit timer channels and one
- 32- bit timer channel. It supports the following specifications:
- - Pulse input/output: 28 lines max.
+ 32-bit timer channel. It supports the following specifications:
+ - Pulse input/output: 28 lines max
- Pulse input 3 lines
- Count clock 11 clocks for each channel (14 clocks for MTU0, 12 clocks
for MTU2, and 10 clocks for MTU5, four clocks for MTU1-MTU2 combination
@@ -23,11 +23,11 @@ description: |
- Input capture function (noise filter setting available)
- Counter-clearing operation
- Simultaneous writing to multiple timer counters (TCNT)
- (excluding MTU8).
+ (excluding MTU8)
- Simultaneous clearing on compare match or input capture
- (excluding MTU8).
+ (excluding MTU8)
- Simultaneous input and output to registers in synchronization with
- counter operations (excluding MTU8).
+ counter operations (excluding MTU8)
- Up to 12-phase PWM output in combination with synchronous operation
(excluding MTU8)
- [MTU0 MTU3, MTU4, MTU6, MTU7, and MTU8]
@@ -40,26 +40,26 @@ description: |
- [MTU3, MTU4, MTU6, and MTU7]
- Through interlocked operation of MTU3/4 and MTU6/7, the positive and
negative signals in six phases (12 phases in total) can be output in
- complementary PWM and reset-synchronized PWM operation.
+ complementary PWM and reset-synchronized PWM operation
- In complementary PWM mode, values can be transferred from buffer
registers to temporary registers at crests and troughs of the timer-
counter values or when the buffer registers (TGRD registers in MTU4
- and MTU7) are written to.
- - Double-buffering selectable in complementary PWM mode.
+ and MTU7) are written to
+ - Double-buffering selectable in complementary PWM mode
- [MTU3 and MTU4]
- Through interlocking with MTU0, a mode for driving AC synchronous
motors (brushless DC motors) by using complementary PWM output and
reset-synchronized PWM output is settable and allows the selection
- of two types of waveform output (chopping or level).
+ of two types of waveform output (chopping or level)
- [MTU5]
- - Capable of operation as a dead-time compensation counter.
+ - Capable of operation as a dead-time compensation counter
- [MTU0/MTU5, MTU1, MTU2, and MTU8]
- 32-bit phase counting mode specifiable by combining MTU1 and MTU2 and
- through interlocked operation with MTU0/MTU5 and MTU8.
+ through interlocked operation with MTU0/MTU5 and MTU8
- Interrupt-skipping function
- In complementary PWM mode, interrupts on crests and troughs of counter
values and triggers to start conversion by the A/D converter can be
- skipped.
+ skipped
- Interrupt sources: 43 sources.
- Buffer operation:
- Automatic transfer of register data (transfer from the buffer
@@ -68,9 +68,9 @@ description: |
- A/D converter start triggers can be generated
- A/D converter start request delaying function enables A/D converter
to be started with any desired timing and to be synchronized with
- PWM output.
+ PWM output
- Low power consumption function
- - The MTU3a can be placed in the module-stop state.
+ - The MTU3a can be placed in the module-stop state
There are two phase counting modes. 16-bit phase counting mode in which
MTU1 and MTU2 operate independently, and cascade connection 32-bit phase
--
2.25.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
2023-07-20 13:10 [PATCH 0/6] Add RZ/G2UL MTU3a support Biju Das
2023-07-20 13:10 ` [PATCH 1/6] dt-bindings: timer: renesas,rz-mtu3: Improve documentation Biju Das
@ 2023-07-20 13:10 ` Biju Das
2023-07-20 17:19 ` Conor Dooley
2023-07-20 13:10 ` [PATCH 3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC Biju Das
2 siblings, 1 reply; 11+ messages in thread
From: Biju Das @ 2023-07-20 13:10 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: Biju Das, Daniel Lezcano, Thomas Gleixner, Geert Uytterhoeven,
Magnus Damm, Lee Jones, linux-iio, linux-renesas-soc, devicetree,
Fabrizio Castro, Prabhakar Mahadev Lad, stable
As per R01UH0914EJ0130 Rev.1.30 HW manual the MTU3 overflow/underflow
interrupt names starts with 'tci' instead of 'tgi'.
Fix this documentation issue by replacing below overflow/underflow
interrupt names:
- tgiv0->tciv0
- tgiv1->tciv1
- tgiu1->tciu1
- tgiv2->tciv2
- tgiu2->tciu2
- tgiv3->tciv3
- tgiv4->tciv4
- tgiv6->tciv6
- tgiv7->tciv7
- tgiv8->tciv8
- tgiu8->tciu8
Fixes: 0a9d6b54297e ("dt-bindings: timer: Document RZ/G2L MTU3a bindings")
Cc: stable@kernel.org
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
.../bindings/timer/renesas,rz-mtu3.yaml | 38 +++++++++----------
1 file changed, 19 insertions(+), 19 deletions(-)
diff --git a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
index eb2d5ebe4df0..670a2ebaacdb 100644
--- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
+++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
@@ -169,27 +169,27 @@ properties:
- const: tgib0
- const: tgic0
- const: tgid0
- - const: tgiv0
+ - const: tciv0
- const: tgie0
- const: tgif0
- const: tgia1
- const: tgib1
- - const: tgiv1
- - const: tgiu1
+ - const: tciv1
+ - const: tciu1
- const: tgia2
- const: tgib2
- - const: tgiv2
- - const: tgiu2
+ - const: tciv2
+ - const: tciu2
- const: tgia3
- const: tgib3
- const: tgic3
- const: tgid3
- - const: tgiv3
+ - const: tciv3
- const: tgia4
- const: tgib4
- const: tgic4
- const: tgid4
- - const: tgiv4
+ - const: tciv4
- const: tgiu5
- const: tgiv5
- const: tgiw5
@@ -197,18 +197,18 @@ properties:
- const: tgib6
- const: tgic6
- const: tgid6
- - const: tgiv6
+ - const: tciv6
- const: tgia7
- const: tgib7
- const: tgic7
- const: tgid7
- - const: tgiv7
+ - const: tciv7
- const: tgia8
- const: tgib8
- const: tgic8
- const: tgid8
- - const: tgiv8
- - const: tgiu8
+ - const: tciv8
+ - const: tciu8
clocks:
maxItems: 1
@@ -285,16 +285,16 @@ examples:
<GIC_SPI 211 IRQ_TYPE_EDGE_RISING>,
<GIC_SPI 212 IRQ_TYPE_EDGE_RISING>,
<GIC_SPI 213 IRQ_TYPE_EDGE_RISING>;
- interrupt-names = "tgia0", "tgib0", "tgic0", "tgid0", "tgiv0", "tgie0",
+ interrupt-names = "tgia0", "tgib0", "tgic0", "tgid0", "tciv0", "tgie0",
"tgif0",
- "tgia1", "tgib1", "tgiv1", "tgiu1",
- "tgia2", "tgib2", "tgiv2", "tgiu2",
- "tgia3", "tgib3", "tgic3", "tgid3", "tgiv3",
- "tgia4", "tgib4", "tgic4", "tgid4", "tgiv4",
+ "tgia1", "tgib1", "tciv1", "tciu1",
+ "tgia2", "tgib2", "tciv2", "tciu2",
+ "tgia3", "tgib3", "tgic3", "tgid3", "tciv3",
+ "tgia4", "tgib4", "tgic4", "tgid4", "tciv4",
"tgiu5", "tgiv5", "tgiw5",
- "tgia6", "tgib6", "tgic6", "tgid6", "tgiv6",
- "tgia7", "tgib7", "tgic7", "tgid7", "tgiv7",
- "tgia8", "tgib8", "tgic8", "tgid8", "tgiv8", "tgiu8";
+ "tgia6", "tgib6", "tgic6", "tgid6", "tciv6",
+ "tgia7", "tgib7", "tgic7", "tgid7", "tciv7",
+ "tgia8", "tgib8", "tgic8", "tgid8", "tciv8", "tciu8";
clocks = <&cpg CPG_MOD R9A07G044_MTU_X_MCK_MTU3>;
power-domains = <&cpg>;
resets = <&cpg R9A07G044_MTU_X_PRESET_MTU3>;
--
2.25.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC
2023-07-20 13:10 [PATCH 0/6] Add RZ/G2UL MTU3a support Biju Das
2023-07-20 13:10 ` [PATCH 1/6] dt-bindings: timer: renesas,rz-mtu3: Improve documentation Biju Das
2023-07-20 13:10 ` [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names Biju Das
@ 2023-07-20 13:10 ` Biju Das
2023-07-20 17:16 ` Conor Dooley
2 siblings, 1 reply; 11+ messages in thread
From: Biju Das @ 2023-07-20 13:10 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: Biju Das, Daniel Lezcano, Thomas Gleixner, Geert Uytterhoeven,
Magnus Damm, linux-iio, linux-renesas-soc, devicetree,
Fabrizio Castro, Prabhakar Mahadev Lad
Add MTU3a binding documentation for Renesas RZ/G2UL SoC.
MTU3a block is identical to one found on RZ/G2L, so no driver changes are
required. The fallback compatible string "renesas,rz-mtu3" will be used
on RZ/G2UL.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
index 670a2ebaacdb..2269e0bf8818 100644
--- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
+++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
@@ -109,6 +109,7 @@ properties:
compatible:
items:
- enum:
+ - renesas,r9a07g043-mtu3 # RZ/G2UL
- renesas,r9a07g044-mtu3 # RZ/G2{L,LC}
- renesas,r9a07g054-mtu3 # RZ/V2L
- const: renesas,rz-mtu3
--
2.25.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC
2023-07-20 13:10 ` [PATCH 3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC Biju Das
@ 2023-07-20 17:16 ` Conor Dooley
0 siblings, 0 replies; 11+ messages in thread
From: Conor Dooley @ 2023-07-20 17:16 UTC (permalink / raw)
To: Biju Das
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Daniel Lezcano,
Thomas Gleixner, Geert Uytterhoeven, Magnus Damm, linux-iio,
linux-renesas-soc, devicetree, Fabrizio Castro,
Prabhakar Mahadev Lad
[-- Attachment #1: Type: text/plain, Size: 1161 bytes --]
On Thu, Jul 20, 2023 at 02:10:13PM +0100, Biju Das wrote:
> Add MTU3a binding documentation for Renesas RZ/G2UL SoC.
>
> MTU3a block is identical to one found on RZ/G2L, so no driver changes are
> required. The fallback compatible string "renesas,rz-mtu3" will be used
> on RZ/G2UL.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Thanks,
Conor.
> ---
> Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> index 670a2ebaacdb..2269e0bf8818 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> @@ -109,6 +109,7 @@ properties:
> compatible:
> items:
> - enum:
> + - renesas,r9a07g043-mtu3 # RZ/G2UL
> - renesas,r9a07g044-mtu3 # RZ/G2{L,LC}
> - renesas,r9a07g054-mtu3 # RZ/V2L
> - const: renesas,rz-mtu3
> --
> 2.25.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
2023-07-20 13:10 ` [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names Biju Das
@ 2023-07-20 17:19 ` Conor Dooley
2023-07-20 17:52 ` Biju Das
0 siblings, 1 reply; 11+ messages in thread
From: Conor Dooley @ 2023-07-20 17:19 UTC (permalink / raw)
To: Biju Das
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Daniel Lezcano,
Thomas Gleixner, Geert Uytterhoeven, Magnus Damm, Lee Jones,
linux-iio, linux-renesas-soc, devicetree, Fabrizio Castro,
Prabhakar Mahadev Lad, stable
[-- Attachment #1: Type: text/plain, Size: 731 bytes --]
On Thu, Jul 20, 2023 at 02:10:12PM +0100, Biju Das wrote:
> As per R01UH0914EJ0130 Rev.1.30 HW manual the MTU3 overflow/underflow
> interrupt names starts with 'tci' instead of 'tgi'.
>
> Fix this documentation issue by replacing below overflow/underflow
> interrupt names:
> - tgiv0->tciv0
> - tgiv1->tciv1
> - tgiu1->tciu1
> - tgiv2->tciv2
> - tgiu2->tciu2
> - tgiv3->tciv3
> - tgiv4->tciv4
> - tgiv6->tciv6
> - tgiv7->tciv7
> - tgiv8->tciv8
> - tgiu8->tciu8
>
> Fixes: 0a9d6b54297e ("dt-bindings: timer: Document RZ/G2L MTU3a bindings")
> Cc: stable@kernel.org
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
I assume this means that nothing is actually using these interrupt
names?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
2023-07-20 17:19 ` Conor Dooley
@ 2023-07-20 17:52 ` Biju Das
2023-07-20 17:59 ` Conor Dooley
0 siblings, 1 reply; 11+ messages in thread
From: Biju Das @ 2023-07-20 17:52 UTC (permalink / raw)
To: Conor Dooley
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Daniel Lezcano,
Thomas Gleixner, Geert Uytterhoeven, Magnus Damm, Lee Jones,
linux-iio@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
devicetree@vger.kernel.org, Fabrizio Castro,
Prabhakar Mahadev Lad, stable@kernel.org
Hi Conor Dooley,
Thanks for the feedback.
> -----Original Message-----
> From: Conor Dooley <conor@kernel.org>
> Sent: Thursday, July 20, 2023 6:19 PM
> To: Biju Das <biju.das.jz@bp.renesas.com>
> Cc: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley <conor+dt@kernel.org>;
> Daniel Lezcano <daniel.lezcano@linaro.org>; Thomas Gleixner
> <tglx@linutronix.de>; Geert Uytterhoeven <geert+renesas@glider.be>;
> Magnus Damm <magnus.damm@gmail.com>; Lee Jones <lee@kernel.org>; linux-
> iio@vger.kernel.org; linux-renesas-soc@vger.kernel.org;
> devicetree@vger.kernel.org; Fabrizio Castro
> <fabrizio.castro.jz@renesas.com>; Prabhakar Mahadev Lad
> <prabhakar.mahadev-lad.rj@bp.renesas.com>; stable@kernel.org
> Subject: Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix
> overflow/underflow interrupt names
>
> On Thu, Jul 20, 2023 at 02:10:12PM +0100, Biju Das wrote:
> > As per R01UH0914EJ0130 Rev.1.30 HW manual the MTU3 overflow/underflow
> > interrupt names starts with 'tci' instead of 'tgi'.
> >
> > Fix this documentation issue by replacing below overflow/underflow
> > interrupt names:
> > - tgiv0->tciv0
> > - tgiv1->tciv1
> > - tgiu1->tciu1
> > - tgiv2->tciv2
> > - tgiu2->tciu2
> > - tgiv3->tciv3
> > - tgiv4->tciv4
> > - tgiv6->tciv6
> > - tgiv7->tciv7
> > - tgiv8->tciv8
> > - tgiu8->tciu8
> >
> > Fixes: 0a9d6b54297e ("dt-bindings: timer: Document RZ/G2L MTU3a
> > bindings")
> > Cc: stable@kernel.org
> > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
>
> I assume this means that nothing is actually using these interrupt
> names?
Yes, That is correct. Corresponding driver doesn't have interrupt support.
Cheers,
Biju
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
2023-07-20 17:52 ` Biju Das
@ 2023-07-20 17:59 ` Conor Dooley
2023-07-24 8:04 ` Biju Das
0 siblings, 1 reply; 11+ messages in thread
From: Conor Dooley @ 2023-07-20 17:59 UTC (permalink / raw)
To: Biju Das
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Daniel Lezcano,
Thomas Gleixner, Geert Uytterhoeven, Magnus Damm, Lee Jones,
linux-iio@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
devicetree@vger.kernel.org, Fabrizio Castro,
Prabhakar Mahadev Lad, stable@kernel.org
[-- Attachment #1: Type: text/plain, Size: 2001 bytes --]
On Thu, Jul 20, 2023 at 05:52:00PM +0000, Biju Das wrote:
> > -----Original Message-----
> > From: Conor Dooley <conor@kernel.org>
> > Sent: Thursday, July 20, 2023 6:19 PM
> > To: Biju Das <biju.das.jz@bp.renesas.com>
> > Cc: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
> > <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley <conor+dt@kernel.org>;
> > Daniel Lezcano <daniel.lezcano@linaro.org>; Thomas Gleixner
> > <tglx@linutronix.de>; Geert Uytterhoeven <geert+renesas@glider.be>;
> > Magnus Damm <magnus.damm@gmail.com>; Lee Jones <lee@kernel.org>; linux-
> > iio@vger.kernel.org; linux-renesas-soc@vger.kernel.org;
> > devicetree@vger.kernel.org; Fabrizio Castro
> > <fabrizio.castro.jz@renesas.com>; Prabhakar Mahadev Lad
> > <prabhakar.mahadev-lad.rj@bp.renesas.com>; stable@kernel.org
btw, please trim this stuff :)
> > Subject: Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix
> > overflow/underflow interrupt names
> >
> > On Thu, Jul 20, 2023 at 02:10:12PM +0100, Biju Das wrote:
> > > As per R01UH0914EJ0130 Rev.1.30 HW manual the MTU3 overflow/underflow
> > > interrupt names starts with 'tci' instead of 'tgi'.
> > >
> > > Fix this documentation issue by replacing below overflow/underflow
> > > interrupt names:
> > > - tgiv0->tciv0
> > > - tgiv1->tciv1
> > > - tgiu1->tciu1
> > > - tgiv2->tciv2
> > > - tgiu2->tciu2
> > > - tgiv3->tciv3
> > > - tgiv4->tciv4
> > > - tgiv6->tciv6
> > > - tgiv7->tciv7
> > > - tgiv8->tciv8
> > > - tgiu8->tciu8
> > >
> > > Fixes: 0a9d6b54297e ("dt-bindings: timer: Document RZ/G2L MTU3a
> > > bindings")
> > > Cc: stable@kernel.org
> > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> >
> > I assume this means that nothing is actually using these interrupt
> > names?
>
> Yes, That is correct. Corresponding driver doesn't have interrupt support.
Okay, if it is not being used anywhere:
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Thanks,
Conor.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
2023-07-20 17:59 ` Conor Dooley
@ 2023-07-24 8:04 ` Biju Das
2023-07-24 8:15 ` Conor Dooley
0 siblings, 1 reply; 11+ messages in thread
From: Biju Das @ 2023-07-24 8:04 UTC (permalink / raw)
To: Conor Dooley
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Daniel Lezcano,
Thomas Gleixner, Geert Uytterhoeven, Magnus Damm, Lee Jones,
linux-iio@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
devicetree@vger.kernel.org, Fabrizio Castro,
Prabhakar Mahadev Lad, stable@kernel.org
Hi Conor Dooley,
> Subject: Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix
> overflow/underflow interrupt names
>
> On Thu, Jul 20, 2023 at 05:52:00PM +0000, Biju Das wrote:
> > > -----Original Message-----
> > > From: Conor Dooley <conor@kernel.org>
>
> > > Sent: Thursday, July 20, 2023 6:19 PM
> > > To: Biju Das <biju.das.jz@bp.renesas.com>
> > > Cc: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
> > > <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley
> > > <conor+dt@kernel.org>; Daniel Lezcano <daniel.lezcano@linaro.org>;
> > > Thomas Gleixner <tglx@linutronix.de>; Geert Uytterhoeven
> > > <geert+renesas@glider.be>; Magnus Damm <magnus.damm@gmail.com>; Lee
> > > Jones <lee@kernel.org>; linux- iio@vger.kernel.org;
> > > linux-renesas-soc@vger.kernel.org;
> > > devicetree@vger.kernel.org; Fabrizio Castro
> > > <fabrizio.castro.jz@renesas.com>; Prabhakar Mahadev Lad
> > > <prabhakar.mahadev-lad.rj@bp.renesas.com>; stable@kernel.org
>
> btw, please trim this stuff :)
You mean trim the recipient list?
Cheers,
Biju
>
> > > Subject: Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix
> > > overflow/underflow interrupt names
> > >
> > > On Thu, Jul 20, 2023 at 02:10:12PM +0100, Biju Das wrote:
> > > > As per R01UH0914EJ0130 Rev.1.30 HW manual the MTU3
> > > > overflow/underflow interrupt names starts with 'tci' instead of
> 'tgi'.
> > > >
> > > > Fix this documentation issue by replacing below overflow/underflow
> > > > interrupt names:
> > > > - tgiv0->tciv0
> > > > - tgiv1->tciv1
> > > > - tgiu1->tciu1
> > > > - tgiv2->tciv2
> > > > - tgiu2->tciu2
> > > > - tgiv3->tciv3
> > > > - tgiv4->tciv4
> > > > - tgiv6->tciv6
> > > > - tgiv7->tciv7
> > > > - tgiv8->tciv8
> > > > - tgiu8->tciu8
> > > >
> > > > Fixes: 0a9d6b54297e ("dt-bindings: timer: Document RZ/G2L MTU3a
> > > > bindings")
> > > > Cc: stable@kernel.org
> > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > >
> > > I assume this means that nothing is actually using these interrupt
> > > names?
> >
> > Yes, That is correct. Corresponding driver doesn't have interrupt
> support.
>
> Okay, if it is not being used anywhere:
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
>
> Thanks,
> Conor.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
2023-07-24 8:04 ` Biju Das
@ 2023-07-24 8:15 ` Conor Dooley
2023-07-24 8:21 ` Biju Das
0 siblings, 1 reply; 11+ messages in thread
From: Conor Dooley @ 2023-07-24 8:15 UTC (permalink / raw)
To: Biju Das
Cc: Conor Dooley, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Daniel Lezcano, Thomas Gleixner, Geert Uytterhoeven, Magnus Damm,
Lee Jones, linux-iio@vger.kernel.org,
linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
Fabrizio Castro, Prabhakar Mahadev Lad, stable@kernel.org
[-- Attachment #1: Type: text/plain, Size: 1247 bytes --]
On Mon, Jul 24, 2023 at 08:04:08AM +0000, Biju Das wrote:
> Hi Conor Dooley,
>
> > Subject: Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix
> > overflow/underflow interrupt names
> >
> > On Thu, Jul 20, 2023 at 05:52:00PM +0000, Biju Das wrote:
> > > > -----Original Message-----
> > > > From: Conor Dooley <conor@kernel.org>
> >
> > > > Sent: Thursday, July 20, 2023 6:19 PM
> > > > To: Biju Das <biju.das.jz@bp.renesas.com>
> > > > Cc: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
> > > > <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley
> > > > <conor+dt@kernel.org>; Daniel Lezcano <daniel.lezcano@linaro.org>;
> > > > Thomas Gleixner <tglx@linutronix.de>; Geert Uytterhoeven
> > > > <geert+renesas@glider.be>; Magnus Damm <magnus.damm@gmail.com>; Lee
> > > > Jones <lee@kernel.org>; linux- iio@vger.kernel.org;
> > > > linux-renesas-soc@vger.kernel.org;
> > > > devicetree@vger.kernel.org; Fabrizio Castro
> > > > <fabrizio.castro.jz@renesas.com>; Prabhakar Mahadev Lad
> > > > <prabhakar.mahadev-lad.rj@bp.renesas.com>; stable@kernel.org
> >
> > btw, please trim this stuff :)
>
> You mean trim the recipient list?
Not the actual recipient list, this junk that outlook adds to emails :)
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
2023-07-24 8:15 ` Conor Dooley
@ 2023-07-24 8:21 ` Biju Das
0 siblings, 0 replies; 11+ messages in thread
From: Biju Das @ 2023-07-24 8:21 UTC (permalink / raw)
To: Conor Dooley
Cc: Conor Dooley, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Daniel Lezcano, Thomas Gleixner, Geert Uytterhoeven, Magnus Damm,
Lee Jones, linux-iio@vger.kernel.org,
linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
Fabrizio Castro, Prabhakar Mahadev Lad, stable@kernel.org
Hi Conor Dooley,
> On Mon, Jul 24, 2023 at 08:04:08AM +0000, Biju Das wrote:
> > Hi Conor Dooley,
> >
> > > Subject: Re: [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix
> > > overflow/underflow interrupt names
> > >
> > > On Thu, Jul 20, 2023 at 05:52:00PM +0000, Biju Das wrote:
> > > > > -----Original Message-----
> > > > > From: Conor Dooley <conor@kernel.org>
> > >
> > >
> > > btw, please trim this stuff :)
> >
> > You mean trim the recipient list?
>
> Not the actual recipient list, this junk that outlook adds to emails :)
Ah OK got it, like above.
Cheers,
Biju
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2023-07-24 8:21 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-20 13:10 [PATCH 0/6] Add RZ/G2UL MTU3a support Biju Das
2023-07-20 13:10 ` [PATCH 1/6] dt-bindings: timer: renesas,rz-mtu3: Improve documentation Biju Das
2023-07-20 13:10 ` [PATCH 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names Biju Das
2023-07-20 17:19 ` Conor Dooley
2023-07-20 17:52 ` Biju Das
2023-07-20 17:59 ` Conor Dooley
2023-07-24 8:04 ` Biju Das
2023-07-24 8:15 ` Conor Dooley
2023-07-24 8:21 ` Biju Das
2023-07-20 13:10 ` [PATCH 3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC Biju Das
2023-07-20 17:16 ` Conor Dooley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox