On Thursday 15 March 2012, Viresh Kumar wrote:
> On 3/15/2012 4:20 PM, Stefan Roese wrote:
>
> > diff --git a/arch/arm/boot/dts/spear600.dtsi
> > b/arch/arm/boot/dts/spear600.dtsi
>
> > +/ {
> > + compatible = "st,spear600";
> > +
> > + cpus {
> > + cpu@0 {
> > + compatible = "arm,arm926ejs";
> > + };
> > + };
> > +
> > + memory {
> > + device_type = "memory";
> > + reg = <0 0x40000000>; /* Real value updated by U-Boot */
> > + };
> > +
> > + ahb {
>
> What is the significance of this name? As i see gpio/spi etc in this list,
> which are on apb bus.
That sounds like it should be corrected, the device tree should reflect
the physical hierarchy where possible, e.g.
ahb0 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "simple-bus";
ranges = <0xe0000000 0xe0000000 0x10000000>;
ethernet@e0800000 {
...
};
...
};
ahb1 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "simple-bus";
ranges = <0xd0000000 0xd0000000 0x10000000>;
ranges = <0xf0000000 0xf0000000 0x10000000>;
apb {
#address-cells = <1>;
#size-cells = <1>;
compatible = "simple-bus";
ranges = <0x0 0xf0000000 0x10000000>;
gpio@f0100000 {
/* note: reg relative to local ranges property
*/
reg = <0x100000 0x1000>;
...
};
...
};
...
i2c@d0200000 {
reg = <0xd0200000 0x1000>;
...
};
};
I don't have the data sheet, so the bus hierarchy above is completely made up,
but I guess you get the idea. If each bus has its own local register range, it
may be helpful to use the ranges property to remap the reg properties in the
way that I did in the apb example above.
Arnd
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
