On Wed, May 9, 2018 at 6:44 PM, Sudeep Holla <sudeep.holla@xxxxxxx> wrote: > It is a bit unorthodox to just include a file in the middle of a another > DTS file, it breaks the pattern from other device trees and also makes > it really hard to reference things across the files with phandles. > > Restructure the include for the Juno/RTSM motherboards to happen at the > top of the file, reference the target nodes directly, and indent the > motherboard .dtsi files to reflect their actual depth in the hierarchy. > > This is a purely syntactic change that result in the same DTB files from > the DTS/DTSI files. This is based on similar patch from Linus Walleij > for ARM Vexpress platforms. > > Cc: Liviu Dudau <liviu.dudau@xxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx> > Signed-off-by: Sudeep Holla <sudeep.holla@xxxxxxx> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Thanks for doing this, it is much more readable for me now. Sorry for slow review :/ Yours, Linus Walleij -- 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
