summaryrefslogtreecommitdiffstats
path: root/drivers/net/xen-netback
diff options
context:
space:
mode:
authorzhong jiang <zhongjiang@huawei.com>2018-09-08 15:35:06 +0200
committerDavid S. Miller <davem@davemloft.net>2018-09-12 08:01:33 +0200
commit0527097ce81cad20529e4f2c37a092774d713d36 (patch)
tree67c0f3b03a35e02a914324307f253ec6f6166353 /drivers/net/xen-netback
parentnet: dsa: b53: Uninitialized variable in b53_adjust_link() (diff)
downloadlinux-0527097ce81cad20529e4f2c37a092774d713d36.tar.xz
linux-0527097ce81cad20529e4f2c37a092774d713d36.zip
net: xenbus: remove redundant condition check before debugfs_remove_recursive
debugfs_remove_recursive has taken the IS_ERR_OR_NULL into account. Just remove the unnecessary condition check. Signed-off-by: zhong jiang <zhongjiang@huawei.com> Reviewed-by: Paul Durrant <paul.durrant@citrix.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/xen-netback')
-rw-r--r--drivers/net/xen-netback/xenbus.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/xen-netback/xenbus.c b/drivers/net/xen-netback/xenbus.c
index cd51492ae6c2..fe1d52247bbe 100644
--- a/drivers/net/xen-netback/xenbus.c
+++ b/drivers/net/xen-netback/xenbus.c
@@ -254,8 +254,7 @@ static void xenvif_debugfs_delif(struct xenvif *vif)
if (IS_ERR_OR_NULL(xen_netback_dbg_root))
return;
- if (!IS_ERR_OR_NULL(vif->xenvif_dbg_root))
- debugfs_remove_recursive(vif->xenvif_dbg_root);
+ debugfs_remove_recursive(vif->xenvif_dbg_root);
vif->xenvif_dbg_root = NULL;
}
#endif /* CONFIG_DEBUG_FS */