diff mbox series

[-next] dma: Add dev_id parameter description in request_dma_bycap

Message ID 20240419090259.39542-1-yang.lee@linux.alibaba.com (mailing list archive)
State New
Headers show
Series [-next] dma: Add dev_id parameter description in request_dma_bycap | expand

Commit Message

Yang Li April 19, 2024, 9:02 a.m. UTC
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@linux.alibaba.com>
---
 arch/sh/drivers/dma/dma-api.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Geert Uytterhoeven April 19, 2024, 9:13 a.m. UTC | #1
On Fri, Apr 19, 2024 at 11:03 AM Yang Li <yang.lee@linux.alibaba.com> 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@linux.alibaba.com>

Fixes: db9b99d461ddbbaa ("sh: dma-api channel capability extensions.")
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert
John Paul Adrian Glaubitz April 19, 2024, 9:21 a.m. UTC | #2
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@linux.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@physik.fu-berlin.de>

Adrian
John Paul Adrian Glaubitz May 2, 2024, 10:31 a.m. UTC | #3
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@linux.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

This patch is now obsolete since search_cap() was removed in for-next by Geert
Uytterhoven in 89256d73 ("sh: dma: Remove unused functionality").

Thanks,
Adrian
diff mbox series

Patch

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