Hi Simon, On Thu, Nov 23, 2017 at 10:46 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > On Thu, Nov 23, 2017 at 10:27:55AM +0100, Geert Uytterhoeven wrote: >> On Mon, Nov 20, 2017 at 12:01 PM, Simon Horman >> <horms+renesas@xxxxxxxxxxxx> wrote: >> > --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi >> > +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi >> > @@ -126,6 +126,86 @@ >> >> The node above is called "cpus". >> >> > }; >> > }; >> > >> > + pmu_a57 { >> >> > + }; >> > + >> > + pmu_a53 { >> >> > + }; >> > + >> > + timer { >> >> > + }; >> > + >> > + thermal-zones { >> >> > + }; >> > + >> > extal_clk: extal { >> > compatible = "fixed-clock"; >> > #clock-cells = <0>; >> >> Any sorting rules (alphabetically?) for nodes at the root level? > > Now would be a good time to decide on an alternate on an order as > I plan to submit similar patches for several other SoCs. > > I took used order they were already present (with other nodes) in the > cpu node. They appear to be sorted alphabetically. Should I keep that > sort order relative to existing nodes present in the root node? I think you should. 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html