Path: csiph.com!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From: Sumit Semwal <sumit.semwal@linaro.org>
Newsgroups: linux.kernel
Subject: Re: [PATCH] dma-buf: fix kernel-doc warning and typos
Date: Wed, 17 Aug 2016 05:10:02 +0200
Message-ID: <s6ZoS-3CS-9@gated-at.bofh.it>
References: <s6W7D-1el-7@gated-at.bofh.it>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Sender: robomod@news.nic.it
Approved: robomod@news.nic.it
Lines: 44
Organization: linux.* mail to news gateway
Xref: csiph.com linux.kernel:1464298

Hi Randy,

On 17 August 2016 at 05:01, Randy Dunlap <rdunlap@infradead.org> wrote:
> From: Randy Dunlap <rdunlap@infradead.org>
>
> Fix dma-buf kernel-doc warning and 2 minor typos in
> fence_array_create().
>
Thanks for your patch, I will queue it up!
> Fixes this warning:
> ..//drivers/dma-buf/fence-array.c:124: warning: No description found for parameter 'signal_on_any'
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc:     Sumit Semwal <sumit.semwal@linaro.org>
> Cc:     linux-media@vger.kernel.org
> Cc:     dri-devel@lists.freedesktop.org
> Cc:     linaro-mm-sig@lists.linaro.org
> ---
>  drivers/dma-buf/fence-array.c |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> --- lnx-48-rc2.orig/drivers/dma-buf/fence-array.c
> +++ lnx-48-rc2/drivers/dma-buf/fence-array.c
> @@ -106,14 +106,14 @@ const struct fence_ops fence_array_ops =
>   * @fences:            [in]    array containing the fences
>   * @context:           [in]    fence context to use
>   * @seqno:             [in]    sequence number to use
> - * @signal_on_any      [in]    signal on any fence in the array
> + * @signal_on_any:     [in]    signal on any fence in the array
>   *
>   * Allocate a fence_array object and initialize the base fence with fence_init().
>   * In case of error it returns NULL.
>   *
> - * The caller should allocte the fences array with num_fences size
> + * The caller should allocate the fences array with num_fences size
>   * and fill it with the fences it wants to add to the object. Ownership of this
> - * array is take and fence_put() is used on each fence on release.
> + * array is taken and fence_put() is used on each fence on release.
>   *
>   * If @signal_on_any is true the fence array signals if any fence in the array
>   * signals, otherwise it signals when all fences in the array signal.

Best,
Sumit.