Hi Biju, On Tue, Nov 27, 2018 at 1:05 PM Biju Das <biju.das@xxxxxxxxxxxxxx> wrote: > Add a device node for the Watchdog Timer (RWDT) controller on the Renesas > RZ/G1N (r8a7744) SoC. > > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> Thanks for your patch! > --- a/arch/arm/boot/dts/r8a7744.dtsi > +++ b/arch/arm/boot/dts/r8a7744.dtsi > @@ -293,6 +293,16 @@ > reg = <0 0xe6160000 0 0x100>; > }; > > + rwdt: watchdog@e6020000 { Please preserve sort order (by address, per group of devices), by inserting before gpio@e6050000. 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