On Thu, Jun 24, 2021 at 3:03 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > Add I2C{0,1,2,3} nodes to RZ/G2L (R9A07G044) SoC DTSI. > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > --- a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi > +++ b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi > @@ -89,6 +89,86 @@ > status = "disabled"; > }; > > + i2c0: i2c@10058000 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "renesas,riic-r9a07g044", "renesas,riic-rz"; > + reg = <0 0x10058000 0 0x400>; > + interrupts = <GIC_SPI 350 IRQ_TYPE_LEVEL_HIGH>, There's a double space after the first interrupt number (in each i2c node). With the above fixed: Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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