Re: [PATCH 2/5] Add RapidIO sector to MPC8641HPCN board dts file.

[Phil Terry <pterry@micromemory.com>]

On Tue, 2007-06-12 at 17:02 +0800, Zhang Wei wrote:
> Add RapidIO sector to the MPC8641HPCN board dts file.
> 
> Signed-off-by: Zhang Wei <wei.zhang@freescale.com>
> ---
>  arch/powerpc/boot/dts/mpc8641_hpcn.dts |   13 +++++++++++++
>  1 files changed, 13 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/powerpc/boot/dts/mpc8641_hpcn.dts b/arch/powerpc/boot/dts/mpc8641_hpcn.dts
> index 04626b1..e2ce06e 100644
> --- a/arch/powerpc/boot/dts/mpc8641_hpcn.dts
> +++ b/arch/powerpc/boot/dts/mpc8641_hpcn.dts
> @@ -329,6 +329,19 @@
>  				>;
>  		};
>  
> +		srio@c0000 {
> +			device_type = "rapidio";
> +			compatible = "fsl,rapidio-v1.0";
> +			#address-cells = <2>;
> +			reg = <c0000 20000>;
> +			ranges = <0 0 c0000000 20000000>;

Don't understand the range setup... The code uses c0000000 as the start
and 20000000 as the size for the rapidio law. So whats the 0 0 for? 
At first I thought address-cells 2 means that the range numbers are
pairs expressing 64-bit numbers, eg start 0 0, size c0000000 20000000
but thats not right...

Sorry if I'm asking stupid questions....

And thanks for the patch, just what I need.

BTW do you have any notes/documentation on the space allocation routines
intended use. I'm trying to work it out but I don't see any actual users
of it yet... I need something like it for my distributed DMA driver,
hence the interest.

Cheers
Phil

> +			interrupt-parent = <&mpic>;
> +			/* err_irq bell_outb_irq bell_inb_irq
> +						msg1_tx_irq msg1_rx_irq
> +						msg2_tx_irq msg2_rx_irq */
> +			interrupts = <30 2 31 2 32 2 35 2 36 2 37 2 38 2>;
> +		};
> +
>  		mpic: pic@40000 {
>  			clock-frequency = <0>;
>  			interrupt-controller;