Re: [PATCH] bcache: fix kernel-doc warning on CLOSURE_CALLBACK

From: Coly Li

Date: Thu Oct 08 2026 - 23:42:22 EST


Hi Rosen,

On Sun, Oct 04, 2026 at 04:35:23PM +0800, Rosen Penev wrote:
> bch_data_insert() is defined through the CLOSURE_CALLBACK() macro, which
> kernel-doc cannot see through, so the kernel-doc comment above it
> produces:
>
> request.c:308 function parameter 'bch_data_insert' not described in 'CLOSURE_CALLBACK'
> request.c:308 expecting prototype for bch_data_insert(). Prototype was for CLOSURE_CALLBACK() instead
>
> Turn it into a regular block comment, and drop the @cl line along
> with the kernel-doc markup.
>
> Assisted-by: LLM

The patch is fine to me. Can you provide the detailed LLM name here?

> Signed-off-by: Rosen Penev <rosenp@xxxxxxxxx>

Thanks.

Coly Li

> ---
> drivers/md/bcache/request.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/md/bcache/request.c b/drivers/md/bcache/request.c
> index 3fa3b13a410f..e7f097346d64 100644
> --- a/drivers/md/bcache/request.c
> +++ b/drivers/md/bcache/request.c
> @@ -285,9 +285,8 @@ static CLOSURE_CALLBACK(bch_data_insert_start)
> }
> }
>
> -/**
> +/*
> * bch_data_insert - stick some data in the cache
> - * @cl: closure pointer.
> *
> * This is the starting point for any data to end up in a cache device; it could
> * be from a normal write, or a writeback write, or a write to a flash only
> --
> 2.56.0