mbox series

[v2,0/6] Add RZ/G2UL MTU3a support

Message ID 20230724091927.123847-1-biju.das.jz@bp.renesas.com
Headers show
Series Add RZ/G2UL MTU3a support | expand

Message

Biju Das July 24, 2023, 9:19 a.m. UTC
This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
Also it fixes overflow/underflow interrupt names.

v1->v2:
 * Added Ack tags from Conor Dooley for binding patches
 * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
 * Fixed build error reported by kernel test robot by replacing
   GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.

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(-)

Comments

Geert Uytterhoeven July 24, 2023, 3:26 p.m. UTC | #1
Hi Biju,

On Mon, Jul 24, 2023 at 11:19 AM Biju Das <biju.das.jz@bp.renesas.com> 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 for your patch!

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

> --- 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

As patch 5/6 applies to both RZ/G2UL and RZ/Five, I think you should
add "and RZ/Five", too.

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