Re: [PATCH 1/4] ARM: dts: Restructure Vexpress motherboard includes

2018-05-30 Thread Sudeep Holla
On 29/05/18 12:48, Linus Walleij wrote: > On Tue, May 29, 2018 at 11:25 AM, Sudeep Holla wrote: >> On 28/05/18 13:26, Linus Walleij wrote: > >>> This is a purely syntactic change that result in the same >>> DTB files from the DTS/DTSI files. > >> Is it different from v1/earlier version you had

Re: [PATCH 1/4] ARM: dts: Restructure Vexpress motherboard includes

2018-05-30 Thread Sudeep Holla
Hi Linus, On 28/05/18 13:26, Linus Walleij 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

Re: [PATCH 1/4] ARM: dts: Restructure Vexpress motherboard includes

2018-05-29 Thread Linus Walleij
On Tue, May 29, 2018 at 11:25 AM, Sudeep Holla wrote: > On 28/05/18 13:26, Linus Walleij wrote: >> This is a purely syntactic change that result in the same >> DTB files from the DTS/DTSI files. > Is it different from v1/earlier version you had posted ? Don't think so. Just resent it as part of

[PATCH 1/4] ARM: dts: Restructure Vexpress motherboard includes

2018-05-28 Thread Linus Walleij
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 Versatile Express motherboards to happen at the top of