lkml.org 
[lkml]   [2011]   [Sep]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRE: [PATCH 4/4] powerpc/fsl: Document rapidio node binding-information
Date
Thanks Kumar! I'll update these SRIO patches following all your comments.

Regards,

Liu Gang
-----Original Message-----
From: Kumar Gala [mailto:galak@kernel.crashing.org]
Sent: Thursday, September 29, 2011 10:21 PM
To: Liu Gang-B34182
Cc: linuxppc-dev@lists.ozlabs.org; akpm@linux-foundation.org; linux-kernel@vger.kernel.org; Li Yang-R58472; Gala Kumar-B11780; Zang Roy-R61911; Li Yang-R58472; Jin Qing
Subject: Re: [PATCH 4/4] powerpc/fsl: Document rapidio node binding-information


On Sep 28, 2011, at 9:29 PM, Liu Gang wrote:

> This document is created for powerpc rapidio and rmu nodes in dts
> file. These nodes can support two rapidio ports and message units. In
> addition, It explicates the properties and gives examples about rapidio and rmu nodes.
>
> Signed-off-by: Li Yang <leoli@freescale.com>
> Signed-off-by: Jin Qing <b24347@freescale.com>
> Signed-off-by: Liu Gang <Gang.Liu@freescale.com>
> ---
> .../devicetree/bindings/powerpc/fsl/srio.txt | 85 ++++++++++++++++++++
> 1 files changed, 85 insertions(+), 0 deletions(-) create mode 100644
> Documentation/devicetree/bindings/powerpc/fsl/srio.txt

You need to fix this for 80 char column line wrap.

>
> diff --git a/Documentation/devicetree/bindings/powerpc/fsl/srio.txt
> b/Documentation/devicetree/bindings/powerpc/fsl/srio.txt
> new file mode 100644
> index 0000000..01f2da1
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/powerpc/fsl/srio.txt
> @@ -0,0 +1,85 @@
> +* Freescale Rapidio Controller
> +
> +Rapidio port node:
> +Properties:
> + - compatible: "fsl,rapidio-delta".
> + "fsl,rapidio-delta" should be listed for any chip whose rapidio controller is compatible.
> + At first, rapidio controller was introduced with the version of delta and has no revision
> + register. Rapidio driver and controller were matched by "fsl,rapidio-delta". After the
> + addition of two revision registers in rapidio controller, we can read some revision and
> + configuration information about rapidio controller IP block, and the compatible with
> + "fsl,rapidio-delta" was still used.

Lets go ahead and change this from 'fsl,rapdio-delta' to 'fsl,srio' since you're changing how the binding is defined completely. This way old .dts will fail graceful with the new code.

> +
> + - reg: For devices compatible with "fsl,rapidio-delta", should contain the address and
> + the length about all the rapidio controller's registers.
> +
> + - ranges: Should be defined according to the u-boot settings about SRIO. Describe the memory
> + mapped I/O space used by the rapidio controller.
> +
> + - interrupts: Interrupt mapping for rapidio IRQ. Three interrupts in the group, and starting
> + with SRIO error/port-write IRQ, an error interrupt and with interrupt type 1. The other
> + two interrupts are doorbell outbound IRQ and doorbell inbound IRQ, and they are external
> + interrupts.

be clear about the order of the interrupts

> +
> + - fsl,rio-num-ports: The number of rapidio ports supported by this controller.
> +
> + - fsl,liodn: The logical I/O device number for the PAMU to be correctly configured for SRIO
> + accesses. This property is added in SRIO node by u-boot and usually used by hypervisor.
> + The number of elements may either be 2 or 4 LIODN values. For HW that only supports LIODNs
> + for ALL memory & maintenance transactions we have 2 cells.

We need to add something like:

In this case, the first cell is the LIODN associated with the port 1 and the second LIODN is associated with port 2.


> For HW that has separate LIODNs
> + for memory & maintenance transaction we utilize 4 cells.

In this case, the first pair of cells are associated with port 1. The first cell is the LIODN associated with memory transactions, the second cell is the LIODN associated with maintenance transactions. The 3 and 4 cells follow the same pattern for port 2.

> +
> + - rmu-handle: The phandle for the rmu connected to this rapidio controller.
> +
> +Example:
> +
> + rapidio: rapidio@ffe0c0000 {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + compatible = "fsl,rapidio-delta";
> + interrupts = <
> + 16 2 1 11 /* err_irq */
> + 56 2 0 0 /* bell_outb_irq */
> + 57 2 0 0>;/* bell_inb_irq */
> + fsl,rio-num-ports = <2>;
> + rmu-handle = <&rmu>;
> + };
> +
> +Message unit node:
> +Properties:
> + - compatible: "fsl,rmu".
> + "fsl,rmu" should be listed for any chip whose message unit is compatible. In addition,
> + RMAN will replace RMU for rapidio message transaction in some chips using DPAA architecture.
> + Then instead of RMU node, RMAN node will be used in dts file and the compatible property
> + "fsl,rmu" should be replaced.
> +
> + - reg: Registers mapping for message unit.
> +
> + - interrupts: Interrupt mapping for message unit controller. Every message
> + unit controller has two external interrupts: message outbound IRQ and
> + message inbound IRQ.

be clear about the order of the interrupts being TX first and RX second.

> +
> + - fsl,liodn: The logical I/O device number for rmuliodnr and added by u-boot.
> +
> +Example:
> +
> + rmu: rmu@d3000 {
> + fsl,liodn = <0xc8>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "fsl,rmu";
> + reg = <0xd3000 0x200>;
> +
> + message-unit@0 {
> + reg = <0x0 0x100>;
> + interrupts = <
> + 60 2 0 0 /* msg1_tx_irq */
> + 61 2 0 0>;/* msg1_rx_irq */
> + };
> + message-unit@1 {

should be message-unit@100

> + reg = <0x100 0x100>;
> + interrupts = <
> + 62 2 0 0 /* msg2_tx_irq */
> + 63 2 0 0>;/* msg2_rx_irq */
> + };
> + };
> --
> 1.7.3.1
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe
> linux-kernel" in the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/





\
 
 \ /
  Last update: 2011-09-30 09:39    [W:0.053 / U:0.728 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site