Re: [PATCH v2 1/2] dt-bindings: milbeaut-m10v-hdmac: Add Socionext Milbeaut HDMAC bindings
On 04-09-19, 01:18, Jassi Brar wrote: > On Wed, Sep 4, 2019 at 12:51 AM Vinod Koul wrote: > > > > On 18-08-19, 00:17, jassisinghb...@gmail.com wrote: > > > From: Jassi Brar > > > > > > Document the devicetree bindings for Socionext Milbeaut HDMAC > > > controller. Controller has upto 8 floating channels, that need > > > a predefined slave-id to work from a set of slaves. > > > > > > Signed-off-by: Jassi Brar > > > --- > > > .../bindings/dma/milbeaut-m10v-hdmac.txt | 32 +++ > > > 1 file changed, 32 insertions(+) > > > create mode 100644 > > > Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > > > > > diff --git > > > a/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > > b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > > new file mode 100644 > > > index ..f0960724f1c7 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > > @@ -0,0 +1,32 @@ > > > +* Milbeaut AHB DMA Controller > > > + > > > +Milbeaut AHB DMA controller has transfer capability bellow. > > > > s/bellow/below: > > > > > + - device to memory transfer > > > + - memory to device transfer > > > + > > > +Required property: > > > +- compatible: Should be "socionext,milbeaut-m10v-hdmac" > > > +- reg: Should contain DMA registers location and length. > > > +- interrupts: Should contain all of the per-channel DMA interrupts. > > > + Number of channels is configurable - 2, 4 or 8, so > > > + the number of interrupts specfied should be {2,4,8}. > > > > s/specfied/specified > > > Hi Vinod, > Do you want me to spin yet another revision for the two types in text? Yes that would be easier for me Thanks -- ~Vinod
Re: [PATCH v2 1/2] dt-bindings: milbeaut-m10v-hdmac: Add Socionext Milbeaut HDMAC bindings
On Wed, Sep 4, 2019 at 12:51 AM Vinod Koul wrote: > > On 18-08-19, 00:17, jassisinghb...@gmail.com wrote: > > From: Jassi Brar > > > > Document the devicetree bindings for Socionext Milbeaut HDMAC > > controller. Controller has upto 8 floating channels, that need > > a predefined slave-id to work from a set of slaves. > > > > Signed-off-by: Jassi Brar > > --- > > .../bindings/dma/milbeaut-m10v-hdmac.txt | 32 +++ > > 1 file changed, 32 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > > > diff --git a/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > new file mode 100644 > > index ..f0960724f1c7 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > @@ -0,0 +1,32 @@ > > +* Milbeaut AHB DMA Controller > > + > > +Milbeaut AHB DMA controller has transfer capability bellow. > > s/bellow/below: > > > + - device to memory transfer > > + - memory to device transfer > > + > > +Required property: > > +- compatible: Should be "socionext,milbeaut-m10v-hdmac" > > +- reg: Should contain DMA registers location and length. > > +- interrupts: Should contain all of the per-channel DMA interrupts. > > + Number of channels is configurable - 2, 4 or 8, so > > + the number of interrupts specfied should be {2,4,8}. > > s/specfied/specified > Hi Vinod, Do you want me to spin yet another revision for the two types in text? thnx
Re: [PATCH v2 1/2] dt-bindings: milbeaut-m10v-hdmac: Add Socionext Milbeaut HDMAC bindings
On 18-08-19, 00:17, jassisinghb...@gmail.com wrote: > From: Jassi Brar > > Document the devicetree bindings for Socionext Milbeaut HDMAC > controller. Controller has upto 8 floating channels, that need > a predefined slave-id to work from a set of slaves. > > Signed-off-by: Jassi Brar > --- > .../bindings/dma/milbeaut-m10v-hdmac.txt | 32 +++ > 1 file changed, 32 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > > diff --git a/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > new file mode 100644 > index ..f0960724f1c7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > @@ -0,0 +1,32 @@ > +* Milbeaut AHB DMA Controller > + > +Milbeaut AHB DMA controller has transfer capability bellow. s/bellow/below: > + - device to memory transfer > + - memory to device transfer > + > +Required property: > +- compatible: Should be "socionext,milbeaut-m10v-hdmac" > +- reg: Should contain DMA registers location and length. > +- interrupts: Should contain all of the per-channel DMA interrupts. > + Number of channels is configurable - 2, 4 or 8, so > + the number of interrupts specfied should be {2,4,8}. s/specfied/specified -- ~Vinod
Re: [PATCH v2 1/2] dt-bindings: milbeaut-m10v-hdmac: Add Socionext Milbeaut HDMAC bindings
On Sun, 18 Aug 2019 00:17:54 -0500, jassisinghb...@gmail.com wrote: > From: Jassi Brar > > Document the devicetree bindings for Socionext Milbeaut HDMAC > controller. Controller has upto 8 floating channels, that need > a predefined slave-id to work from a set of slaves. > > Signed-off-by: Jassi Brar > --- > .../bindings/dma/milbeaut-m10v-hdmac.txt | 32 +++ > 1 file changed, 32 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt > Reviewed-by: Rob Herring
[PATCH v2 1/2] dt-bindings: milbeaut-m10v-hdmac: Add Socionext Milbeaut HDMAC bindings
From: Jassi Brar Document the devicetree bindings for Socionext Milbeaut HDMAC controller. Controller has upto 8 floating channels, that need a predefined slave-id to work from a set of slaves. Signed-off-by: Jassi Brar --- .../bindings/dma/milbeaut-m10v-hdmac.txt | 32 +++ 1 file changed, 32 insertions(+) create mode 100644 Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt diff --git a/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt new file mode 100644 index ..f0960724f1c7 --- /dev/null +++ b/Documentation/devicetree/bindings/dma/milbeaut-m10v-hdmac.txt @@ -0,0 +1,32 @@ +* Milbeaut AHB DMA Controller + +Milbeaut AHB DMA controller has transfer capability bellow. + - device to memory transfer + - memory to device transfer + +Required property: +- compatible: Should be "socionext,milbeaut-m10v-hdmac" +- reg: Should contain DMA registers location and length. +- interrupts: Should contain all of the per-channel DMA interrupts. + Number of channels is configurable - 2, 4 or 8, so + the number of interrupts specfied should be {2,4,8}. +- #dma-cells: Should be 1. Specify the ID of the slave. +- clocks: Phandle to the clock used by the HDMAC module. + + +Example: + + hdmac1: dma-controller@1e11 { + compatible = "socionext,milbeaut-m10v-hdmac"; + reg = <0x1e11 0x1>; + interrupts = <0 132 4>, +<0 133 4>, +<0 134 4>, +<0 135 4>, +<0 136 4>, +<0 137 4>, +<0 138 4>, +<0 139 4>; + #dma-cells = <1>; + clocks = <_clk>; + }; -- 2.17.1