[PATCH] bcache: Fix compiler warnings

Marilene A Garcia posted 1 patch 1 year, 8 months ago
drivers/md/bcache/request.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] bcache: Fix compiler warnings
Posted by Marilene A Garcia 1 year, 8 months ago
It fixes the following compiler warnings by
using "/*" instead of "/**" in the comments

drivers/md/bcache/request.c:309: warning:
Function parameter or struct member 'bch_data_insert'
not described in 'CLOSURE_CALLBACK'

drivers/md/bcache/request.c:309: warning: expecting
prototype for bch_data_insert(). Prototype was for
CLOSURE_CALLBACK() instead

Signed-off-by: Marilene A Garcia <marilene.agarcia@gmail.com>
---
Hello,
These warnings happen using GCC compiler the command 'make W=1'.

Thank you.

 drivers/md/bcache/request.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/bcache/request.c b/drivers/md/bcache/request.c
index af345dc6fde1..457b9605aff9 100644
--- a/drivers/md/bcache/request.c
+++ b/drivers/md/bcache/request.c
@@ -285,7 +285,7 @@ static CLOSURE_CALLBACK(bch_data_insert_start)
 	}
 }
 
-/**
+/*
  * bch_data_insert - stick some data in the cache
  * @cl: closure pointer.
  *
-- 
2.34.1
Re: [PATCH] bcache: Fix compiler warnings
Posted by Marilene Andrade Garcia 1 year, 8 months ago
On 05/06/2024 00:40, Marilene A Garcia wrote:
> It fixes the following compiler warnings by
> using "/*" instead of "/**" in the comments
> 
> drivers/md/bcache/request.c:309: warning:
> Function parameter or struct member 'bch_data_insert'
> not described in 'CLOSURE_CALLBACK'
> 
> drivers/md/bcache/request.c:309: warning: expecting
> prototype for bch_data_insert(). Prototype was for
> CLOSURE_CALLBACK() instead
> 
> Signed-off-by: Marilene A Garcia <marilene.agarcia@gmail.com>
> ---
> Hello,
> These warnings happen using GCC compiler the command 'make W=1'.
> 
> Thank you.
> 
>   drivers/md/bcache/request.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/md/bcache/request.c b/drivers/md/bcache/request.c
> index af345dc6fde1..457b9605aff9 100644
> --- a/drivers/md/bcache/request.c
> +++ b/drivers/md/bcache/request.c
> @@ -285,7 +285,7 @@ static CLOSURE_CALLBACK(bch_data_insert_start)
>   	}
>   }
>   
> -/**
> +/*
>    * bch_data_insert - stick some data in the cache
>    * @cl: closure pointer.
>    *

Hello,
I have realized that even though this change could fix the warning, it 
is not the correct fix to it, because it breaks the format of kernel 
function comments.

So please discard the patch.
Thank you.