Hi Simon, On Thu, Mar 22, 2018 at 11:24 AM, Simon Horman <horms+renesas@xxxxxxxxxxxx> wrote: > Sort subnodes of the soc node. > - The primary key is the bus address. > - The secondary key is the IP block. > - The tertiary key is the node name. > > This is part of an ongoing effort to provide consistent node > order in the DT of Renesas SoCs to improve maintainability. > > This should not have any run-time effect. > > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> Thanks for your patch! Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> using dtx_diff, and "grep '@.*{', which tells me > --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi > pciec0: pcie@fe000000 { > @@ -1860,6 +1838,43 @@ > /* placeholder */ > }; > > + gic: interrupt-controller@f1010000 { Probably gic should be before pciec0. Currently it's after pciec1, which has a lower address than pciec1. 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