[PATCH 15/16] Add device tree for Ebony

[Josh Boyer]

On Wed, 2007-02-14 at 09:14 -0700, Yoder Stuart-B08248 wrote:
> I realize the dts is preliminary, but saw some stuff and 
> thought I would comment.
> 
> > -----Original Message-----
> > From: linuxppc-dev-bounces+b08248=freescale.com at ozlabs.org 
> > [mailto:linuxppc-dev-bounces+b08248=freescale.com at ozlabs.org] 
> > On Behalf Of David Gibson
> 
> [snip]
> 
> > +	UIC0: interrupt-controller { /* UIC0 */
> > +		device_type = "interrupt-controller";
> > +		compatible = "ibm,uic440gp", "ibm,uic";
> > +		interrupt-controller;
> > +		cell-index = <0>;
> > +		dcr-reg = <0c0 009>;
> > +		#address-cells = <0>;
> > +		#size-cells = <0>;
> > +		#interrupt-cells = <2>;
> > +
> > +		UIC1: interrupt-controller at 1 { /* UIC1 */
> > +			device_type = "interrupt-controller";
> > +			compatible = "ibm,uic440gp", "ibm,uic";
> > +			interrupt-controller;
> > +			cell-index = <1>;
> > +			dcr-reg = <0d0 009>;
> > +			#address-cells = <0>;
> > +			#size-cells = <0>;
> > +			#interrupt-cells = <2>;
> > +			interrupts = <1e 4 1f 4>; /* cascade */
> > +		};
> > +	};
> 
> Did you mean to have UIC1 nested under UIC0 or should
> they be at the same level?

No, they are cascaded in hardware.  I think having UIC1 under UCI0 is a
correct representation.

> 
> > +	plb {
> > +		device_type = "soc";
> > +		compatible = "ibm,plb-440gp", "ibm,plb4";
> > +		ranges;
> > +
> > +		POB0: opb at 0 {
> > +			device_type = "soc";
> > +			compatible = "ibm,opb-440gp", "ibm,opb";
> > +			ranges;
> > +			dcr-reg = <090 00b>;
> > +			interrupt-parent = <&UIC1>;
> > +			interrupts = <7 4>;
> > +			clock-frequency = <3ef1480>; // FIXME: 66MHz
> 
> Hmm.  There are two "soc" devices here, one nested under the
> first??
> 
> I'm assuming these are two levels of busses the opb bus is attached
> to the plb bus.  Is the "soc" device_type the right way to 
> do this?

Right, OPB hangs off of PLB in this case.  I dunno if "soc" is the right
device type for them though.

josh