infiniband: remove redundant condition check before debugfs_remove

xiongzhongjiang · jgunthorpe · commit 3e5d60bcc8a4 · 2018-09-11T09:37:05.000-06:00
debugfs_remove has taken the IS_ERR_OR_NULL into account. Just remove the
unnecessary condition.

Signed-off-by: zhong jiang &lt;zhongjiang@huawei.com&gt;
Signed-off-by: Jason Gunthorpe &lt;jgg@mellanox.com&gt;
diff --git a/drivers/infiniband/hw/usnic/usnic_debugfs.c b/drivers/infiniband/hw/usnic/usnic_debugfs.c
@@ -165,6 +165,5 @@ void usnic_debugfs_flow_add(struct usnic_ib_qp_grp_flow *qp_flow)
 
 void usnic_debugfs_flow_remove(struct usnic_ib_qp_grp_flow *qp_flow)
 {
-	if (!IS_ERR_OR_NULL(qp_flow->dbgfs_dentry))
-		debugfs_remove(qp_flow->dbgfs_dentry);
+	debugfs_remove(qp_flow->dbgfs_dentry);
 }

Original file line number	Diff line number	Diff line change
`@@ -165,6 +165,5 @@ void usnic_debugfs_flow_add(struct usnic_ib_qp_grp_flow *qp_flow)`
`165`	`165`
`166`	`166`	`void usnic_debugfs_flow_remove(struct usnic_ib_qp_grp_flow *qp_flow)`
`167`	`167`	`{`
`168`		`- if (!IS_ERR_OR_NULL(qp_flow->dbgfs_dentry))`
`169`		`- debugfs_remove(qp_flow->dbgfs_dentry);`
	`168`	`+ debugfs_remove(qp_flow->dbgfs_dentry);`
`170`	`169`	`}`