Linux IIO development
 help / color / mirror / Atom feed
* [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