diff mbox series

crypto: acix: Remove unused struct 'dbgfs_u32'

Message ID 20240511145017.226166-1-linux@treblig.org (mailing list archive)
State Accepted
Delegated to: Herbert Xu
Headers show
Series crypto: acix: Remove unused struct 'dbgfs_u32' | expand

Commit Message

Dr. David Alan Gilbert May 11, 2024, 2:50 p.m. UTC
From: "Dr. David Alan Gilbert" <linux@treblig.org>

'dbgfs_u32' appears unused.
Remove it.
(pdma_stat_descr is also unused, but I'm assuming it's
some useful layout description of firmware/hardware
so best left in)

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/crypto/axis/artpec6_crypto.c | 7 -------
 1 file changed, 7 deletions(-)

Comments

Dr. David Alan Gilbert May 25, 2024, 12:06 a.m. UTC | #1
* linux@treblig.org (linux@treblig.org) wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> 'dbgfs_u32' appears unused.
> Remove it.
> (pdma_stat_descr is also unused, but I'm assuming it's
> some useful layout description of firmware/hardware
> so best left in)
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>

Ping.

Dave

> ---
>  drivers/crypto/axis/artpec6_crypto.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/drivers/crypto/axis/artpec6_crypto.c b/drivers/crypto/axis/artpec6_crypto.c
> index dbc1d483f2afa..75440ea6206e2 100644
> --- a/drivers/crypto/axis/artpec6_crypto.c
> +++ b/drivers/crypto/axis/artpec6_crypto.c
> @@ -2811,13 +2811,6 @@ static struct aead_alg aead_algos[] = {
>  
>  #ifdef CONFIG_DEBUG_FS
>  
> -struct dbgfs_u32 {
> -	char *name;
> -	mode_t mode;
> -	u32 *flag;
> -	char *desc;
> -};
> -
>  static struct dentry *dbgfs_root;
>  
>  static void artpec6_crypto_init_debugfs(void)
> -- 
> 2.45.0
>
Jesper Nilsson May 31, 2024, 7:48 a.m. UTC | #2
On Sat, May 11, 2024 at 03:50:17PM +0100, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> 'dbgfs_u32' appears unused.
> Remove it.
> (pdma_stat_descr is also unused, but I'm assuming it's
> some useful layout description of firmware/hardware
> so best left in)

IIRC, the stat data is of this format, but it is only
checked for "non-zero" as indicator a finished job.

> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>

Acked-by: Jesper Nilsson <jesper.nilsson@axis.com>

/^JN - Jesper Nilsson
Herbert Xu May 31, 2024, 9:41 a.m. UTC | #3
On Sat, May 11, 2024 at 03:50:17PM +0100, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> 'dbgfs_u32' appears unused.
> Remove it.
> (pdma_stat_descr is also unused, but I'm assuming it's
> some useful layout description of firmware/hardware
> so best left in)
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> ---
>  drivers/crypto/axis/artpec6_crypto.c | 7 -------
>  1 file changed, 7 deletions(-)

Patch applied.  Thanks.
diff mbox series

Patch

diff --git a/drivers/crypto/axis/artpec6_crypto.c b/drivers/crypto/axis/artpec6_crypto.c
index dbc1d483f2afa..75440ea6206e2 100644
--- a/drivers/crypto/axis/artpec6_crypto.c
+++ b/drivers/crypto/axis/artpec6_crypto.c
@@ -2811,13 +2811,6 @@  static struct aead_alg aead_algos[] = {
 
 #ifdef CONFIG_DEBUG_FS
 
-struct dbgfs_u32 {
-	char *name;
-	mode_t mode;
-	u32 *flag;
-	char *desc;
-};
-
 static struct dentry *dbgfs_root;
 
 static void artpec6_crypto_init_debugfs(void)