[U-Boot] [PATCH 1/7] tegra: fdt: add apbdma node

Allen Martin amartin at nvidia.com
Sat Jan 12 07:49:45 CET 2013


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


More information about the U-Boot mailing list