From mboxrd@z Thu Jan 1 00:00:00 1970 From: Allen Martin Date: Fri, 11 Jan 2013 22:49:45 -0800 Subject: [U-Boot] [PATCH 1/7] tegra: fdt: add apbdma node In-Reply-To: <50F0FDAD.2000502@wwwdotorg.org> References: <1357929877-18671-1-git-send-email-amartin@nvidia.com> <1357929877-18671-2-git-send-email-amartin@nvidia.com> <50F0A997.3080603@wwwdotorg.org> <20130112031900.GA27011@badger> <50F0FDAD.2000502@wwwdotorg.org> Message-ID: <20130112064944.GA21751@badger> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, Jan 11, 2013 at 10:07:41PM -0800, Stephen Warren wrote: > On 01/11/2013 08:19 PM, Allen Martin wrote: > > On Fri, Jan 11, 2013 at 04:08:55PM -0800, Stephen Warren wrote: > >> On 01/11/2013 11:44 AM, Allen Martin wrote: > >>> Add apbdma node for tegra20 and tegra30, copied directly from tegra > >>> Linux dtsi files. > >> > >>> diff --git a/arch/arm/dts/tegra20.dtsi b/arch/arm/dts/tegra20.dtsi > >> > >>> + apbdma: dma { > >>> + compatible = "nvidia,tegra20-apbdma"; > >>> + reg = <0x6000a000 0x1200>; > >> ... > >>> intc: interrupt-controller at 50041000 { > >> > >> Can the nodes be kept sorted by address? > >> > > > > They're already unsorted. I'll add a new patch to sort them, then add > > this one on top. > > Ah yes. > > You can use the kernel as a reference, but FYI the order I have > attempted to impose there is: > > 1) Any nodes that already exist in any /include/d file, in the order > they appear in the /include/d file. > > 2) Any nodes with a reg property, in order of their address. > > 3) Any nodes without a reg property, alphabetically by node name. > > If U-Boot follows the same rules, diff'ing the .dts files between U-Boot > and the kernel should be easy. Ok, thanks, I'll try to follow the same rules. BTW: cache-controller and interrupt-controller are out of order in the kernel dtsi files. -Allen -- nvpublic