net/mlx5e: Remove incorrect debugfs_create_dir NULL check in TLS
authorGal Pressman <gal@nvidia.com>
Thu, 19 Jan 2023 12:09:00 +0000 (14:09 +0200)
committerSaeed Mahameed <saeedm@nvidia.com>
Wed, 8 Feb 2023 00:29:55 +0000 (16:29 -0800)
Remove the NULL check on debugfs_create_dir() return value as the
function returns an ERR pointer on failure, not NULL.
The check is not replaced with a IS_ERR_OR_NULL() as
debugfs_create_file(), and debugfs functions in general don't need error
checking.

Fixes: 0fedee1ae9ef ("net/mlx5e: kTLS, Add debugfs")
Reported-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Gal Pressman <gal@nvidia.com>
Reviewed-by: Tariq Toukan <tariqt@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
drivers/net/ethernet/mellanox/mlx5/core/en_accel/ktls_tx.c

index e80b43b..60b3e08 100644 (file)
@@ -899,8 +899,6 @@ static void mlx5e_tls_tx_debugfs_init(struct mlx5e_tls *tls,
                return;
 
        tls->debugfs.dfs_tx = debugfs_create_dir("tx", dfs_root);
-       if (!tls->debugfs.dfs_tx)
-               return;
 
        debugfs_create_size_t("pool_size", 0400, tls->debugfs.dfs_tx,
                              &tls->tx_pool->size);