[PATCH 2/5] Warp Base Platform - dts
David Gibson
david at gibson.dropbear.id.au
Fri Jan 11 16:32:31 EST 2008
On Fri, Jan 11, 2008 at 12:21:20AM -0500, Sean MacLennan wrote:
> David Gibson wrote:
> > On Thu, Jan 10, 2008 at 06:59:03PM -0500, Sean MacLennan wrote:
> >
> >> New version with recommended changes. Device types removed. FPGA
> >> moved to correct bus.
> >>
> >
> > [snip]
> >
> >> + fpga at 2,0 {
> >> + compatible = "pika,fpga";
> >> + reg = <0 80000000 2200>;
> >>
> >
> > Your reg property doesn't match your node name here...
> >
> Is the following correct?
>
> fpga at 2,0 {
> compatible = "pika,fpga";
> reg = <2 80000000 2200>;
> interrupts = <18 8>;
> interrupt-parent = <&UIC0>;
> };
No. You'd need
fpga at 2,80000000 {
reg = <2 80000000 2200>;
...
};
If, indeed, the fpga really has 32 bit address lines and requires the
high one to be 1...
Or possibly what you actually want is:
fpga at 2,0 {
reg = <2 0 2200>;
...
};
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
More information about the Linuxppc-dev
mailing list