[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20170519012957.q3dr73qharbwtvc3@rob-hp-laptop>
Date: Thu, 18 May 2017 20:29:57 -0500
From: Rob Herring <robh@...nel.org>
To: sean.wang@...iatek.com
Cc: dan.j.williams@...el.com, vinod.koul@...el.com,
mark.rutland@....com, dmaengine@...r.kernel.org,
devicetree@...r.kernel.org, linux-mediatek@...ts.infradead.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
keyhaede@...il.com
Subject: Re: [PATCH 1/2] dt-bindings: dmaengine: Add Mediatek High-Speed DMA
controller bindings
On Mon, May 15, 2017 at 02:59:26PM +0800, sean.wang@...iatek.com wrote:
> From: Sean Wang <sean.wang@...iatek.com>
>
> Document the devicetree bindings for Mediatek High-Speed DMA controller
> which could be found on MT7623 SoC or other similar Mediatek SoCs.
>
> Signed-off-by: Sean Wang <sean.wang@...iatek.com>
> ---
> .../devicetree/bindings/dma/mtk-hsdma.txt | 30 ++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/dma/mtk-hsdma.txt
>
> diff --git a/Documentation/devicetree/bindings/dma/mtk-hsdma.txt b/Documentation/devicetree/bindings/dma/mtk-hsdma.txt
> new file mode 100644
> index 0000000..62cdda4
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/dma/mtk-hsdma.txt
> @@ -0,0 +1,30 @@
> +Mediatek High-Speed DMA Controller
> +==================================
> +
> +This driver follows the generic DMA bindings defined in dma/dma.txt.
> +
> +Required properties:
> +
> +- compatible: Must be one of
> + "mediatek,mt7623-hsdma".
> +- reg: Should contain the register's base address and length.
> +- interrupts: Should contain a reference to the interrupt used by this
> + device.
> +- clocks: Should be the clock specifiers corresponding to the entry in
> + clock-names property.
> +- clock-names: Should contain "hsdma" entries.
> +- #dma-cells: The length of the DMA specifier, must be <1>. This one cell
> + in dmas property of a client device represents the channel
> + number.
> +Example:
> +
> + hsdma: hsdma@...07000 {
dma-controller@...
> + compatible = "mediatek,mt7623-hsdma";
> + reg = <0 0x1b007000 0 0x1000>;
> + interrupts = <GIC_SPI 98 IRQ_TYPE_LEVEL_LOW>;
> + clocks = <ðsys CLK_ETHSYS_HSDMA>;
> + clock-names = "hsdma";
> + status = "okay";
Don't need to show status in examples.
With those,
Acked-by: Rob Herring <robh@...nel.org>
Powered by blists - more mailing lists