[PATCH 1/5 v3] Add the explanation and a sample of RapidIO OF node to the document of booting-without-of.txt file.

2007-07-26 Thread Zhang Wei
Add the explanation and a sample of RapidIO OF node to the document of 
booting-without-of.txt file.

Signed-off-by: Zhang Wei <[EMAIL PROTECTED]>
---
 Documentation/powerpc/booting-without-of.txt |   37 ++
 1 files changed, 37 insertions(+), 0 deletions(-)

diff --git a/Documentation/powerpc/booting-without-of.txt 
b/Documentation/powerpc/booting-without-of.txt
index 76733a3..d89b805 100644
--- a/Documentation/powerpc/booting-without-of.txt
+++ b/Documentation/powerpc/booting-without-of.txt
@@ -1824,6 +1824,43 @@ platforms are moved over to use the 
flattened-device-tree model.
fsl,has-rstcr;
};
 
+   l) RapidIO
+
+   RapidIO is a definition of a system interconnect. This node add
+   the support for RapidIO processor in kernel. The node name is
+   suggested to be 'rapidio'.
+
+   Required properties:
+
+- compatible : Using "fsl,rapidio-delta" for Freescale PowerPC
+  RapidIO controller.
+- #address-cells : Address representation for "rapidio" devices.
+  This field represents the number of cells needed to represent
+  the RapidIO address of the registers.
+  See 1) above for more details on defining #address-cells.
+- reg : Offset and length of the register set for the device
+- ranges : Should be defined as specified in 1) to describe the
+  translation of addresses for memory mapped RapidIO memory
+  space.
+- interrupts : binding interrupts for this device node,
+  please follow below orders:
+  .
+
+  Example:
+
+   [EMAIL PROTECTED] {
+   compatible = "fsl,rapidio-delta";
+   #address-cells = <2>;
+   #size-cells = <2>;
+   reg = ;
+   ranges = <0 0 c000 2000>;
+   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>;
+   };
+
More devices will be defined as this spec matures.
 
 VII - Specifying interrupt information for devices
-- 
1.5.1

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 1/5 v3] Add the explanation and a sample of RapidIO OF node to the document of booting-without-of.txt file.

2007-07-27 Thread Kumar Gala


On Jul 26, 2007, at 3:42 AM, Zhang Wei wrote:

Add the explanation and a sample of RapidIO OF node to the document  
of booting-without-of.txt file.


Signed-off-by: Zhang Wei <[EMAIL PROTECTED]>
---
 Documentation/powerpc/booting-without-of.txt |   37 +++ 
+++

 1 files changed, 37 insertions(+), 0 deletions(-)

diff --git a/Documentation/powerpc/booting-without-of.txt b/ 
Documentation/powerpc/booting-without-of.txt

index 76733a3..d89b805 100644
--- a/Documentation/powerpc/booting-without-of.txt
+++ b/Documentation/powerpc/booting-without-of.txt
@@ -1824,6 +1824,43 @@ platforms are moved over to use the  
flattened-device-tree model.

fsl,has-rstcr;
};

+   l) RapidIO
+
+   RapidIO is a definition of a system interconnect. This node add
+   the support for RapidIO processor in kernel. The node name is
+   suggested to be 'rapidio'.
+
+   Required properties:
+
+- compatible : Using "fsl,rapidio-delta" for Freescale PowerPC
+  RapidIO controller.
+- #address-cells : Address representation for "rapidio" devices.
+  This field represents the number of cells needed to represent
+  the RapidIO address of the registers.


Can you explain this a little further.  I'm a bit confused by  
'RapidIO address of the registers'.



+  See 1) above for more details on defining #address-cells.
+- reg : Offset and length of the register set for the device
+- ranges : Should be defined as specified in 1) to describe the
+  translation of addresses for memory mapped RapidIO memory
+  space.
+- interrupts : binding interrupts for this device node,
+  please follow below orders:
+  .
+
+  Example:
+
+   [EMAIL PROTECTED] {
+   compatible = "fsl,rapidio-delta";
+   #address-cells = <2>;
+   #size-cells = <2>;
+   reg = ;
+   ranges = <0 0 c000 2000>;
+   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>;
+   };
+
More devices will be defined as this spec matures.

 VII - Specifying interrupt information for devices
--
1.5.1


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 1/5 v3] Add the explanation and a sample of RapidIO OF node to the document of booting-without-of.txt file.

2007-07-27 Thread Zhang Wei-r63237
Hi, Kumar, 
> +   RapidIO is a definition of a system interconnect. This node add
> > +   the support for RapidIO processor in kernel. The node name is
> > +   suggested to be 'rapidio'.
> > +
> > +   Required properties:
> > +
> > +- compatible : Using "fsl,rapidio-delta" for Freescale PowerPC
> > +  RapidIO controller.
> > +- #address-cells : Address representation for 
> "rapidio" devices.
> > +  This field represents the number of cells needed to represent
> > +  the RapidIO address of the registers.
> 
> Can you explain this a little further.  I'm a bit confused by  
> 'RapidIO address of the registers'.
> 
I want to present "This field represents the number of cells [needed to
represent the RapidIO address] of the registers."
Maybe I should remove 'of the registers' to be more clear.

Thanks!
-zw
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 1/5 v3] Add the explanation and a sample of RapidIO OF node to the document of booting-without-of.txt file.

2007-07-31 Thread Segher Boessenkool

+- #address-cells : Address representation for

"rapidio" devices.

+  This field represents the number of cells needed to represent
+  the RapidIO address of the registers.


Can you explain this a little further.  I'm a bit confused by
'RapidIO address of the registers'.


I want to present "This field represents the number of cells [needed to
represent the RapidIO address] of the registers."
Maybe I should remove 'of the registers' to be more clear.


This is completely content-free anyway; the semantics of
#address-cells (and #size-cells, which you forgot) are
already defined in the base OF spec; what you _should_ be
defining here is a) the required value of #address-cells
for a rapidio bus; and b) how addresses on that bus are
represented (simply as a 64-bit integer, encoded as a
pair of 32-bit integers as usual; but it needs to be said).


Segher

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 1/5 v3] Add the explanation and a sample of RapidIO OF node to the document of booting-without-of.txt file.

2007-07-31 Thread Segher Boessenkool

+   l) RapidIO


"FSL PowerPC bridge RapidIO" or something like that -- you
aren't doing a _generic_ rapidio binding here.


+   RapidIO is a definition of a system interconnect. This node add
+   the support for RapidIO processor in kernel. The node name is
+   suggested to be 'rapidio'.
+
+   Required properties:
+
+- compatible : Using "fsl,rapidio-delta" for Freescale PowerPC
+  RapidIO controller.
+- #address-cells : Address representation for "rapidio" devices.
+  This field represents the number of cells needed to represent
+  the RapidIO address of the registers.
+  See 1) above for more details on defining #address-cells.
+- reg : Offset and length of the register set for the device
+- ranges : Should be defined as specified in 1) to describe the
+  translation of addresses for memory mapped RapidIO memory
+  space.
+- interrupts : binding interrupts for this device node,
+  please follow below orders:
+  .


_All_ of these are "as specified in 1)".  You should be saying
the specifics for your device here; for example, what register
blocks of your controller are described in "reg", and in what
order?


Segher

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/