On 07/22/2013 12:55:38 AM, hongbo.zh...@freescale.com wrote:
From: Hongbo Zhang <hongbo.zh...@freescale.com>
This updates the discription of each type of DMA controller and its
channels,
it is preparation for adding another new DMA controller binding, also
fixes
some defects of indent for text alignment at the same time.
Signed-off-by: Hongbo Zhang <hongbo.zh...@freescale.com>
---
.../devicetree/bindings/powerpc/fsl/dma.txt | 56
+++++++++++---------
1 file changed, 30 insertions(+), 26 deletions(-)
diff --git a/Documentation/devicetree/bindings/powerpc/fsl/dma.txt
b/Documentation/devicetree/bindings/powerpc/fsl/dma.txt
index 2a4b4bc..0650171 100644
--- a/Documentation/devicetree/bindings/powerpc/fsl/dma.txt
+++ b/Documentation/devicetree/bindings/powerpc/fsl/dma.txt
@@ -1,33 +1,33 @@
-* Freescale 83xx DMA Controller
+* Freescale DMA Controllers
-Freescale PowerPC 83xx have on chip general purpose DMA controllers.
+** Freescale ELO DMA Controller
+ This is a little-endian DMA controller.
+ Used in Freescale PowerPC 83xx series, such as:
+ mpc8313, mpc8315, mpc8323, mpc8347, mpc8349, mpc8360, mpc8377,
mpc8378, mpc8379.
You don't need to list every single chip... What happens when new
chips come out (not so likely with mpc83xx, but more likely for
eloplus/elo3)? Do we keep updating this, or have a list that looks
complete but isn't?
My point in suggesting that some examples be given is just so that the
reader knows where to find a manual that documents a particular version
of "elo", and to give a rough idea of what product families use it.
-Freescale PowerPC 85xx/86xx have on chip general purpose DMA
controllers.
+** Freescale ELOPLUS DMA Controller
+ This is DMA controller with extended addresses and chaining.
+ Used in Freescale PowerPC 85xx/86xx and pxxx series chips, such
as:
+ [1] mpc8540, mpc8541, mpc8555, mpc8560, mpc8610, mpc8641,
+ [2] mpc8536, mpc8544, mpc8548, mpc8568, mpc8569, mpc8572, p1010,
p1020, p1021,
+ p1022, p1023, p2020, p2041, p3041, p4080, p5020, p5040, and
also bsc9131.
What do [1] and [2] signify here?
Oh, I see. It's weird for footnotes to come before the place they're
referenced...
Required properties:
-- compatible : compatible list, contains 2 entries, first is
- "fsl,CHIP-dma", where CHIP is the processor
- (mpc8540, mpc8540, etc.) and the second is
- "fsl,eloplus-dma"
+- compatible : compatible list, contains 2 entries for chips
in above
+ list[1], the first is "fsl,CHIP-dma", where
CHIP is the
+ processor and the second is "fsl,eloplus-dma".
contains
+ only one "fsl,eloplus-dma" for chips in above
list[2]
Don't encode this difference in the binding document. It wasn't on
purpose, but rather an artifact of factoring things out into dtsi
files. Just make it optional.
Why can't we just talk about what compatible must include, as I
suggested earlier, rather than what it must *be*?
- reg : <registers mapping for DMA general status reg>
- cell-index : controller index. 0 for controller @ 0x21000,
1 for controller @ 0xc000
-- ranges : Should be defined as specified in 1) to
describe the
- DMA controller channels.
+- ranges : physical address range of DMA controller
channels
ranges will not have physical addresses at this level. They'll only
become physical addresses when translated by the ccsr node's ranges.
-Scott
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
