On Tue, 6 May 2008 08:37:16 -0500
Kumar Gala <[EMAIL PROTECTED]> wrote:

/;
> > +
> > +/ {
> > +   model = "ASP8347E";
> > +   compatible = "ASP8347E";
> 
> "analogue-and-micro, ASP8347E";

Fair enough.

> > +   memory {
> > +           device_type = "memory";
> > +           reg = <0x00000000 0x8000000>;   // 128MB at 0
> 
> is memory really fixed on this board or does the bootloader set this  
> if, dynamic make it <0 0>

It's fixed as far as I know.


> > +   [EMAIL PROTECTED] {
> > +           #address-cells = <1>;
> > +           #size-cells = <1>;
> > +           device_type = "soc";
> > +           ranges = <0x0 0xff000000 0xff100000>;
> 
> this is wrong, size should be like 0x100000

Agreed - typo !

> > +           enet0: [EMAIL PROTECTED] {
> > +                   cell-index = <0>;
> > +                   device_type = "network";
> > +                   model = "TSEC";
> > +                   compatible = "gianfar";
> > +                   reg = <0x24000 0x1000>;
> > +                   local-mac-address = [ 00 08 e5 11 32 33 ];
> > +                   interrupts = <32 0x8 33 0x8 34 0x8>;
> > +                   interrupt-parent = <&ipic>;
> > +                   phy-handle = <&phy0>;
> > +                   linux,network-index = <0>;
> 
> you shouldn't need this anymore. we should be using the aliases.


Aliases sound good to me. Don't know too much about how this should be defined
in a dts though, do you have an example I could use as a base for reference ?

Cheers.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to