linux-kernel - Re: [PATCH -next] dma: Add dev_id parameter description in request_dma

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <377b0acf906b9bcfbc1053ca07d64f14a6cf9a1e.camel@physik.fu-berlin.de>
Date: Fri, 19 Apr 2024 11:21:55 +0200
From: John Paul Adrian Glaubitz <glaubitz@...sik.fu-berlin.de>
To: Yang Li <yang.lee@...ux.alibaba.com>, ysato@...rs.sourceforge.jp, 
	dalias@...c.org
Cc: linux-sh@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH -next] dma: Add dev_id parameter description in
 request_dma_bycap

Hi Yang,

On Fri, 2024-04-19 at 17:02 +0800, Yang Li wrote:
> This patch adds the missing description for the dev_id parameter in the
> kernel documentation for the request_dma_bycap function.
> 
> Signed-off-by: Yang Li <yang.lee@...ux.alibaba.com>
> ---
>  arch/sh/drivers/dma/dma-api.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/sh/drivers/dma/dma-api.c b/arch/sh/drivers/dma/dma-api.c
> index 89cd4a3b4cca..65005d348877 100644
> --- a/arch/sh/drivers/dma/dma-api.c
> +++ b/arch/sh/drivers/dma/dma-api.c
> @@ -116,6 +116,7 @@ static int search_cap(const char **haystack, const char *needle)
>   * request_dma_bycap - Allocate a DMA channel based on its capabilities
>   * @dmac: List of DMA controllers to search
>   * @caps: List of capabilities
> + * @dev_id: Unique identifier for the device that is requesting a DMA channel
>   *
>   * Search all channels of all DMA controllers to find a channel which
>   * matches the requested capabilities. The result is the channel

Thanks. This should include "sh:" in the subject. I will add this manually
when picking up the patch. Also, the missing Fixes tag as Geert noted.

Reviewed-by: John Paul Adrian Glaubitz <glaubitz@...sik.fu-berlin.de>

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913