From: Geert Uytterhoeven <geert@linux-m68k.org>
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>,
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, Conor Dooley <conor.dooley@microchip.com>
Subject: Re: [PATCH v2 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names
Date: Tue, 25 Jul 2023 10:52:24 +0200 [thread overview]
Message-ID: <CAMuHMdWXN2GWeSoG-5Ht66zaR8Y0EzUUh7_5+4Hmtf_YOGpVNQ@mail.gmail.com> (raw)
In-Reply-To: <20230724091927.123847-3-biju.das.jz@bp.renesas.com>
Hi Biju,
On Mon, Jul 24, 2023 at 11:19 AM Biju Das <biju.das.jz@bp.renesas.com> 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>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> ---
> v1->v2:
> * Added Ack from Conor Dooley.
Thanks for your patch!
> --- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> @@ -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
According to the documentation, there is no underflow interrupt for
channel 8?
The rest LGTM.
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
next prev parent reply other threads:[~2023-07-25 8:53 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-24 9:19 [PATCH v2 0/6] Add RZ/G2UL MTU3a support Biju Das
2023-07-24 9:19 ` [PATCH v2 1/6] dt-bindings: timer: renesas,rz-mtu3: Improve documentation Biju Das
2023-07-24 15:15 ` Geert Uytterhoeven
2023-07-24 15:24 ` Biju Das
2023-07-24 9:19 ` [PATCH v2 2/6] dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt names Biju Das
2023-07-25 8:52 ` Geert Uytterhoeven [this message]
2023-07-25 9:11 ` Biju Das
2023-07-25 9:32 ` Geert Uytterhoeven
2023-07-25 9:49 ` Biju Das
2023-07-24 9:19 ` [PATCH v2 3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC Biju Das
2023-07-24 15:26 ` Geert Uytterhoeven
2023-07-24 15:29 ` Biju Das
2023-07-24 9:19 ` [PATCH v2 4/6] arm64: dts: renesas: r9a07g044: Update overfow/underflow IRQ names for MTU3 channels Biju Das
2023-07-25 9:00 ` Geert Uytterhoeven
2023-07-25 9:14 ` Biju Das
2023-07-25 9:33 ` Geert Uytterhoeven
2023-07-24 9:19 ` [PATCH v2 5/6] arm64: dts: renesas: r9a07g043: Add MTU3a node Biju Das
2023-07-24 9:19 ` [PATCH v2 6/6] arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3 Biju Das
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAMuHMdWXN2GWeSoG-5Ht66zaR8Y0EzUUh7_5+4Hmtf_YOGpVNQ@mail.gmail.com \
--to=geert@linux-m68k.org \
--cc=biju.das.jz@bp.renesas.com \
--cc=conor+dt@kernel.org \
--cc=conor.dooley@microchip.com \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=fabrizio.castro.jz@renesas.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=lee@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=prabhakar.mahadev-lad.rj@bp.renesas.com \
--cc=robh+dt@kernel.org \
--cc=stable@kernel.org \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.