RE: [v4 3/6] dt-bindings: fsl-qdma: Add NXP Layerscpae qDMA controller bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Rob Herring [mailto:robh@xxxxxxxxxx]
> Sent: 2018年5月24日 3:59
> To: Wen He <wen.he_1@xxxxxxx>
> Cc: dmaengine@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; Leo Li
> <leoyang.li@xxxxxxx>; Jiafei Pan <jiafei.pan@xxxxxxx>; Jiaheng Fan
> <jiaheng.fan@xxxxxxx>; Vinod <vkoul@xxxxxxxxxx>
> Subject: Re: [v4 3/6] dt-bindings: fsl-qdma: Add NXP Layerscpae qDMA
> controller bindings
> 
> Updated Vinod's email...
> 
> On Mon, May 21, 2018 at 12:52 AM, Wen He <wen.he_1@xxxxxxx> wrote:
> > Hi Rob,
> >
> > Please see my comments inline.
> >
> > Best Regards,
> > Wen
> >
> >> -----Original Message-----
> >> From: Rob Herring [mailto:robh@xxxxxxxxxx]
> >> Sent: 2018年5月19日 5:26
> >> To: Wen He <wen.he_1@xxxxxxx>
> >> Cc: vinod.koul@xxxxxxxxx; dmaengine@xxxxxxxxxxxxxxx;
> >> devicetree@xxxxxxxxxxxxxxx; Leo Li <leoyang.li@xxxxxxx>; Jiafei Pan
> >> <jiafei.pan@xxxxxxx>; Jiaheng Fan <jiaheng.fan@xxxxxxx>
> >> Subject: Re: [v4 3/6] dt-bindings: fsl-qdma: Add NXP Layerscpae qDMA
> >> controller bindings
> >>
> >> On Mon, May 14, 2018 at 08:03:04PM +0800, Wen He wrote:
> >> > Document the devicetree bindings for NXP Layerscape qDMA controller
> >> > which could be found on NXP QorIQ Layerscape SoCs.
> >> >
> >> > Signed-off-by: Wen He <wen.he_1@xxxxxxx>
> >> > ---
> >> > change in v4:
> >> >     - Rewrite the bindings document that follows generic DMA
> >> > bindings file
> >> >
> >> > change in v3:
> >> >     - no change
> >> >
> >> > change in v2:
> >> >     - Remove indentation
> >> >     - Add "Should be" before 'fsl,ls1021a-qdma'
> >> >     - Replace 'channels' by 'dma-channels'
> >> >     - Replace 'qdma@8390000' by 'dma-controller@8390000'
> >> >
> >> >  Documentation/devicetree/bindings/dma/fsl-qdma.txt |   41
> >> ++++++++++++++++++++
> >> >  1 files changed, 41 insertions(+), 0 deletions(-)  create mode
> >> > 100644 Documentation/devicetree/bindings/dma/fsl-qdma.txt
> >> >
> >> > diff --git a/Documentation/devicetree/bindings/dma/fsl-qdma.txt
> >> > b/Documentation/devicetree/bindings/dma/fsl-qdma.txt
> >> > new file mode 100644
> >> > index 0000000..368c4e7
> >> > --- /dev/null
> >> > +++ b/Documentation/devicetree/bindings/dma/fsl-qdma.txt
> >> > @@ -0,0 +1,41 @@
> >> > +NXP Layerscape SoC qDMA Controller
> >> > +==================================
> >> > +
> >> > +This device follows the generic DMA bindings defined in dma/dma.txt.
> >> > +
> >> > +Required properties:
> >> > +
> >> > +- compatible:              Must be one of
> >> > +                    "fsl,ls1021a-qdma": for LS1021A Board
> >> > +                    "fsl,ls1043a-qdma": for ls1043A Board
> >> > +                    "fsl,ls1046a-qdma": for ls1046A Board
> >> > +- reg:                     Should contain the register's base
> address and length.
> >> > +- interrupts:              Should contain a reference to the
> interrupt used by
> >> this
> >> > +                   device.
> >> > +- interrupt-names: Should contain interrupt names:
> >> > +                    "qdma-error": the error interrupt
> >> > +                    "qdma-queue": the queue interrupt
> >> > +- queues:          Should contain number of queues supported.
> >>
> >> Needs a vendor prefix.
> >>
> >
> > Does means: The queues filed need a vendor prefix ?
> > like 'fsl-queues' ? right?
> 
> No, vendor prefixes end with a comma: fsl,queues
> 
> Rob

Done.
Thanks for your review, the issue will next version fix.

Best Regards,
Wen
��.n��������+%������w��{.n��������)�)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥




[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux