Hello Geert, > Subject: Re: [PATCH v2 2/4] ARM: dts: r8a7745: Add PWM SoC support > > On Mon, Dec 18, 2017 at 7:06 PM, Fabrizio Castro > <fabrizio.castro@xxxxxxxxxxxxxx> wrote: > > Add the definitions for pwm[0123456] to the SoC .dtsi. > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > Reviewed-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > > --- a/arch/arm/boot/dts/r8a7745.dtsi > > +++ b/arch/arm/boot/dts/r8a7745.dtsi > > @@ -968,6 +968,83 @@ > > status = "disabled"; > > }; > > > > + pwm0: pwm@e6e30000 { > > + compatible = "renesas,pwm-r8a7745", > > + "renesas,pwm-rcar"; > > I think this can fit on one line. do you think this deserves a v3? Thanks, Fab > > 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 [https://www2.renesas.eu/media/email/unicef_2017.jpg] This Christmas, instead of sending out cards, Renesas Electronics Europe have decided to support Unicef with a donation. For further details click here<https://www.unicef.org/> to find out about the valuable work they do, helping children all over the world. We would like to take this opportunity to wish you a Merry Christmas and a prosperous New Year. Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.