diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-08-12 19:20:06 -0400 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2014-08-13 00:59:54 -0400 |
commit | e42fa2092c1049ac9c0e38aaac39ef3c40e91a36 (patch) | |
tree | c6252e4e64d99acef64cf3a276b61754c5059854 /drivers/infiniband | |
parent | dd57c9308afff61e4c157d0a7260695fe2f9a98c (diff) |
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 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) | |||
281 | { | 281 | { |
282 | struct ipoib_dev_priv *priv = netdev_priv(dev); | 282 | struct ipoib_dev_priv *priv = netdev_priv(dev); |
283 | 283 | ||
284 | if (priv->mcg_dentry) | 284 | debugfs_remove(priv->mcg_dentry); |
285 | debugfs_remove(priv->mcg_dentry); | 285 | debugfs_remove(priv->path_dentry); |
286 | if (priv->path_dentry) | ||
287 | debugfs_remove(priv->path_dentry); | ||
288 | } | 286 | } |
289 | 287 | ||
290 | int ipoib_register_debugfs(void) | 288 | int ipoib_register_debugfs(void) |