diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-08-13 03:20:06 +0400 |
---|---|---|
committer | Ben Hutchings <ben@decadent.org.uk> | 2017-08-26 04:14:28 +0300 |
commit | 01168394ba812e29b1b6caf31bd3d97678573484 (patch) | |
tree | 911a67f80022df069d54c141127e5b445aaf4b9a /drivers/infiniband/ulp/ipoib/ipoib_fs.c | |
parent | f11af14aee3069ef0d3566366abc77275bb27290 (diff) | |
download | linux-01168394ba812e29b1b6caf31bd3d97678573484.tar.xz |
IPoIB: Remove unnecessary test for NULL before debugfs_remove()
commit e42fa2092c1049ac9c0e38aaac39ef3c40e91a36 upstream.
Fix checkpatch warning:
WARNING: debugfs_remove(NULL) is safe this check is probably not required
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'drivers/infiniband/ulp/ipoib/ipoib_fs.c')
-rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_fs.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_fs.c b/drivers/infiniband/ulp/ipoib/ipoib_fs.c index 50061854616e..6bd5740e2691 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_fs.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_fs.c @@ -281,10 +281,8 @@ void ipoib_delete_debug_files(struct net_device *dev) { struct ipoib_dev_priv *priv = netdev_priv(dev); - if (priv->mcg_dentry) - debugfs_remove(priv->mcg_dentry); - if (priv->path_dentry) - debugfs_remove(priv->path_dentry); + debugfs_remove(priv->mcg_dentry); + debugfs_remove(priv->path_dentry); } int ipoib_register_debugfs(void) |