> -----Original Message----- > From: Radhey Shyam Pandey > Sent: Wednesday, June 20, 2018 7:13 PM > To: Andrea Merello <andrea.merello@xxxxxxxxx>; vkoul@xxxxxxxxxx; > dan.j.williams@xxxxxxxxx; Michal Simek <michals@xxxxxxxxxx>; Appana Durga > Kedareswara Rao <appanad@xxxxxxxxxx>; dmaengine@xxxxxxxxxxxxxxx > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: RE: [PATCH 3/6] dt-bindings: xilinx_dma: add required > xlnx,lengthregwidth property > > > > -----Original Message----- > > From: dmaengine-owner@xxxxxxxxxxxxxxx [mailto:dmaengine- > > owner@xxxxxxxxxxxxxxx] On Behalf Of Andrea Merello > > Sent: Wednesday, June 20, 2018 2:07 PM > > To: vkoul@xxxxxxxxxx; dan.j.williams@xxxxxxxxx; Michal Simek > > <michals@xxxxxxxxxx>; Appana Durga Kedareswara Rao > > <appanad@xxxxxxxxxx>; dmaengine@xxxxxxxxxxxxxxx > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > > Andrea Merello <andrea.merello@xxxxxxxxx> > > Subject: [PATCH 3/6] dt-bindings: xilinx_dma: add required > > xlnx,lengthregwidth property > > dt-bindings: dmaengine: xilinx_dma > > Please also include DT folks. > > > > The width of the "length register" cannot be autodetected, and it is now > > specified with a DT property. Add DOC for it. > > > > Signed-off-by: Andrea Merello <andrea.merello@xxxxxxxxx> > > --- > > Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt > > b/Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt > > index a2b8bfaec43c..acecdc5d8d47 100644 > > --- a/Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt > > +++ b/Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt > > @@ -36,6 +36,8 @@ Required properties: > > > > Required properties for VDMA: > > - xlnx,num-fstores: Should be the number of framebuffers as configured in > > h/w. > > +Required properties for AXI DMA: > > +- xlnx,lengthregwidth: Should be the width of the length register as > > configured in h/w. > > One suggestion to be inline with IP property naming we can rename > this prop to "xlnx,sg-length-width"? Please take a look at Xilinx tree > we have this feature added in the master branch. It would be good > to consolidate both implementations and upstream. Let me know > if there are any followup queries. It should be ok to cherrypick 3/6 and 4/6 (xlnx,sg-length-width) from Xilinx tree and include it in your v2 patch series. > > > > > Optional properties: > > - xlnx,include-sg: Tells configured for Scatter-mode in > > -- > > 2.17.1 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe dmaengine" in > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
