On 07/29/2013 04:49 AM, hongbo.zhang@xxxxxxxxxxxxx wrote: > From: Hongbo Zhang <hongbo.zhang@xxxxxxxxxxxxx> > > This patch updates the discription of each type of DMA controller and its > channels, it is preparation for adding another new DMA controller binding, it > also fixes some defects of indent for text alignment at the same time. > diff --git a/Documentation/devicetree/bindings/powerpc/fsl/dma.txt b/Documentation/devicetree/bindings/powerpc/fsl/dma.txt > -- compatible : compatible list, contains 2 entries, first is > - "fsl,CHIP-dma", where CHIP is the processor > - (mpc8349, mpc8360, etc.) and the second is > - "fsl,elo-dma" > +- compatible : must include "fsl,elo-dma" Why remove the list of supported compatible values. Lately it seems that we're moving towards listing more/all the values rather than removing their documentation... > -- ranges : Should be defined as specified in 1) to describe the > - DMA controller channels. > +- ranges : describes the mapping between the address space of the > + DMA channels and the address space of the DMA controller What is "the address space of the DMA controller". Perhaps this should say "the CPU-visible address space" instead? -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html