diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-08-12 19:20:06 -0400 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2014-08-12 21:59:54 -0700 |
commit | e42fa2092c1049ac9c0e38aaac39ef3c40e91a36 (patch) | |
tree | c6252e4e64d99acef64cf3a276b61754c5059854 /drivers/infiniband | |
parent | dd57c9308afff61e4c157d0a7260695fe2f9a98c (diff) | |
download | op-kernel-dev-e42fa2092c1049ac9c0e38aaac39ef3c40e91a36.zip op-kernel-dev-e42fa2092c1049ac9c0e38aaac39ef3c40e91a36.tar.gz |
IPoIB: Remove unnecessary test for NULL before debugfs_remove()
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>
Diffstat (limited to 'drivers/infiniband')
-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 5006185..6bd5740 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) |