Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1385459

Re: [PATCH v2 1/2] Documentation: DT: vdma: Add clock support for vdma

From Rob Herring <robh@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH v2 1/2] Documentation: DT: vdma: Add clock support for vdma
Date 2016-04-22 21:40 +0200
Message-ID <rqP5M-7go-15@gated-at.bofh.it> (permalink)
References <rpYNP-7fp-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Apr 20, 2016 at 05:13:18PM +0530, Kedareswara rao Appana wrote:
> This patch updates the binding doc with clock description
> for vdma.
> 
> Signed-off-by: Kedareswara rao Appana <appanad@xilinx.com>
> ---
> Changes for v2:
> --> Listed down all the clocks supported by the h/w
>     as suggested by the Datta.
> --> Used IP clock names instead of shortcut clock names.
> 
>  Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt b/Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt
> index fcc2b65..afe9eb7 100644
> --- a/Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt
> +++ b/Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt
> @@ -21,6 +21,11 @@ Required properties:
>  - dma-channel child node: Should have at least one channel and can have up to
>  	two channels per device. This node specifies the properties of each
>  	DMA channel (see child node properties below).
> +- clocks: Input clock specifier. Refer to common clock bindings.
> +- clock-names: List of input clocks "s_axi_lite_aclk", "m_axi_mm2s_aclk"
> +	       "m_axi_s2mm_aclk", "m_axis_mm2s_aclk", "s_axis_s2mm_aclk"
> +	       (list of input cloks may vary based on the ip configuration.

s/cloks/clocks/

> +		see clock bindings for more info).

This does not make sense. The common clock binding is going to tell me 
more about how the clocks vary?

You need to define here how the clocks can vary.

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 1/2] Documentation: DT: vdma: Add clock support for vdma Kedareswara rao Appana <appana.durga.rao@xilinx.com> - 2016-04-20 13:50 +0200
  [PATCH v2 2/2] dmaengine: vdma: Add clock support Kedareswara rao Appana <appana.durga.rao@xilinx.com> - 2016-04-20 13:50 +0200
    Re: [PATCH v2 2/2] dmaengine: vdma: Add clock support Shubhrajyoti Datta <shubhrajyoti.datta@gmail.com> - 2016-04-20 14:00 +0200
    RE: [PATCH v2 2/2] dmaengine: vdma: Add clock support Appana Durga Kedareswara Rao <appana.durga.rao@xilinx.com> - 2016-04-20 17:00 +0200
      Re: [PATCH v2 2/2] dmaengine: vdma: Add clock support Sören Brinkmann <soren.brinkmann@xilinx.com> - 2016-04-20 18:20 +0200
    Re: [PATCH v2 2/2] dmaengine: vdma: Add clock support Sören Brinkmann <soren.brinkmann@xilinx.com> - 2016-04-20 17:00 +0200
  Re: [PATCH v2 1/2] Documentation: DT: vdma: Add clock support for  vdma Sören Brinkmann <soren.brinkmann@xilinx.com> - 2016-04-20 16:30 +0200
    RE: [PATCH v2 1/2] Documentation: DT: vdma: Add clock support for  vdma Appana Durga Kedareswara Rao <appana.durga.rao@xilinx.com> - 2016-04-20 17:00 +0200
  Re: [PATCH v2 1/2] Documentation: DT: vdma: Add clock support for  vdma Rob Herring <robh@kernel.org> - 2016-04-22 21:40 +0200
    RE: [PATCH v2 1/2] Documentation: DT: vdma: Add clock support for  vdma Appana Durga Kedareswara Rao <appana.durga.rao@xilinx.com> - 2016-04-23 07:40 +0200

csiph-web