mbox series

[0/2] dt-bindings: can: renesas,rcar-canfd: Make interrupt-names required

Message ID cover.1651512451.git.geert+renesas@glider.be (mailing list archive)
Headers show
Series dt-bindings: can: renesas,rcar-canfd: Make interrupt-names required | expand

Message

Geert Uytterhoeven May 2, 2022, 5:33 p.m. UTC
Hi all,

The Renesas R-Car CAN FD Controller always uses two or more interrupts.
Hence it makes sense to make the interrupt-names property a required
property, to make it easier to identify the individual interrupts, and
validate the mapping.

  - The first patch updates the various R-Car Gen3 and RZ/G2 DTS files
    to add interrupt-names properties, and is intended for the
    renesas-devel tree,
  - The second patch updates the CAN-FD DT bindings to mark the
    interrupt-names property required, and is intended for the DT or net
    tree.

Thanks!

Geert Uytterhoeven (2):
  arm64: dts: renesas: Add interrupt-names to CANFD nodes
  dt-bindings: can: renesas,rcar-canfd: Make interrupt-names required

 .../devicetree/bindings/net/can/renesas,rcar-canfd.yaml        | 3 ++-
 arch/arm64/boot/dts/renesas/r8a774a1.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a774b1.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a774c0.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a774e1.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77951.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77960.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77961.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77965.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77970.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77980.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77990.dtsi                      | 1 +
 arch/arm64/boot/dts/renesas/r8a77995.dtsi                      | 1 +
 13 files changed, 14 insertions(+), 1 deletion(-)

Comments

Marc Kleine-Budde May 2, 2022, 6:26 p.m. UTC | #1
On 02.05.2022 19:33:51, Geert Uytterhoeven wrote:
> 	Hi all,
> 
> The Renesas R-Car CAN FD Controller always uses two or more interrupts.
> Hence it makes sense to make the interrupt-names property a required
> property, to make it easier to identify the individual interrupts, and
> validate the mapping.
> 
>   - The first patch updates the various R-Car Gen3 and RZ/G2 DTS files
>     to add interrupt-names properties, and is intended for the
>     renesas-devel tree,
>   - The second patch updates the CAN-FD DT bindings to mark the
>     interrupt-names property required, and is intended for the DT or net
>     tree.
> 
> Thanks!

LGTM. Who takes this series?

Marc
Geert Uytterhoeven May 3, 2022, 7:25 a.m. UTC | #2
Hi Marc,

On Mon, May 2, 2022 at 8:27 PM Marc Kleine-Budde <mkl@pengutronix.de> wrote:
> On 02.05.2022 19:33:51, Geert Uytterhoeven wrote:
> > The Renesas R-Car CAN FD Controller always uses two or more interrupts.
> > Hence it makes sense to make the interrupt-names property a required
> > property, to make it easier to identify the individual interrupts, and
> > validate the mapping.
> >
> >   - The first patch updates the various R-Car Gen3 and RZ/G2 DTS files
> >     to add interrupt-names properties, and is intended for the
> >     renesas-devel tree,
> >   - The second patch updates the CAN-FD DT bindings to mark the
> >     interrupt-names property required, and is intended for the DT or net
> >     tree.
> >
> > Thanks!
>
> LGTM. Who takes this series?

I'll take the first patch.

The second patch is up to you and the DT maintainers.

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
Marc Kleine-Budde May 3, 2022, 8:13 a.m. UTC | #3
On 03.05.2022 09:25:45, Geert Uytterhoeven wrote:
> Hi Marc,
> 
> On Mon, May 2, 2022 at 8:27 PM Marc Kleine-Budde <mkl@pengutronix.de> wrote:
> > On 02.05.2022 19:33:51, Geert Uytterhoeven wrote:
> > > The Renesas R-Car CAN FD Controller always uses two or more interrupts.
> > > Hence it makes sense to make the interrupt-names property a required
> > > property, to make it easier to identify the individual interrupts, and
> > > validate the mapping.
> > >
> > >   - The first patch updates the various R-Car Gen3 and RZ/G2 DTS files
> > >     to add interrupt-names properties, and is intended for the
> > >     renesas-devel tree,
> > >   - The second patch updates the CAN-FD DT bindings to mark the
> > >     interrupt-names property required, and is intended for the DT or net
> > >     tree.
> > >
> > > Thanks!
> >
> > LGTM. Who takes this series?
> 
> I'll take the first patch.
> 
> The second patch is up to you and the DT maintainers.

Sounds good.

regards,
Marc