RE: [PATCH 3/7] [v2] arm64: dts: add QorIQ LS1046A SoC support

[S.H. Xie]

> -----Original Message-----
> From: Shawn Guo [mailto:shawnguo@xxxxxxxxxx]
> Sent: Thursday, September 08, 2016 9:06 PM
> To: shh.xie@xxxxxxxxx
> Cc: devicetree@xxxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; mark.rutland@xxxxxxx;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; catalin.marinas@xxxxxxx;
> will.deacon@xxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Mihai Emilian Bantea
> <mihai.bantea@xxxxxxx>; C.H. Zhao <chenhui.zhao@xxxxxxx>; arnd@xxxxxxxx;
> S.H. Xie <shaohui.xie@xxxxxxx>; Z.Q. Hou <zhiqiang.hou@xxxxxxx>; M.H. Lian
> <minghuan.lian@xxxxxxx>; Vincent Hu <mingkai.hu@xxxxxxx>; Horia Geanta
> Neag <horia.geanta@xxxxxxx>; Q.Y. Gong <qianyu.gong@xxxxxxx>
> Subject: Re: [PATCH 3/7] [v2] arm64: dts: add QorIQ LS1046A SoC support
> 
> On Mon, Sep 05, 2016 at 06:01:31PM +0800, shh.xie@xxxxxxxxx wrote:
> > +	soc {
> > +		compatible = "simple-bus";
> > +		#address-cells = <2>;
> > +		#size-cells = <2>;
> > +		ranges;
> > +
> > +		clockgen: clocking@1ee1000 {
> > +			compatible = "fsl,ls1046a-clockgen";
> > +			reg = <0x0 0x1ee1000 0x0 0x1000>;
> > +			#clock-cells = <2>;
> > +			clocks = <&sysclk>;
> > +		};
> > +
> > +		scfg: scfg@1570000 {
> 
> Instead of having device node in arbitrary place, can we sort all the nodes under
> 'simple-bus' in order of unit-address?
[S.H] Will sort the nodes in order of unit-address in next version.

Thanks,
Shaohui