nvme-fabrics: remove the unneeded ret variable in nvmf_dev_show

Changcheng Deng · Christoph Hellwig · commit a5f3851b7f79 · 2022-01-27T08:17:17.000+01:00
Remove unneeded variable and directly return 0.

Reported-by: Zeal Robot &lt;zealci@zte.com.cn&gt;
Signed-off-by: Changcheng Deng &lt;deng.changcheng@zte.com.cn&gt;
Signed-off-by: Christoph Hellwig &lt;hch@lst.de&gt;
diff --git a/drivers/nvme/host/fabrics.c b/drivers/nvme/host/fabrics.c
@@ -1092,7 +1092,6 @@ static void __nvmf_concat_opt_tokens(struct seq_file *seq_file)
 static int nvmf_dev_show(struct seq_file *seq_file, void *private)
 {
 	struct nvme_ctrl *ctrl;
-	int ret = 0;
 
 	mutex_lock(&nvmf_dev_mutex);
 	ctrl = seq_file->private;
@@ -1106,7 +1105,7 @@ static int nvmf_dev_show(struct seq_file *seq_file, void *private)
 
 out_unlock:
 	mutex_unlock(&nvmf_dev_mutex);
-	return ret;
+	return 0;
 }
 
 static int nvmf_dev_open(struct inode *inode, struct file *file)

Original file line number	Diff line number	Diff line change
`@@ -1092,7 +1092,6 @@ static void __nvmf_concat_opt_tokens(struct seq_file *seq_file)`
`1092`	`1092`	`static int nvmf_dev_show(struct seq_file seq_file, void private)`
`1093`	`1093`	`{`
`1094`	`1094`	`struct nvme_ctrl *ctrl;`
`1095`		`- int ret = 0;`
`1096`	`1095`
`1097`	`1096`	`mutex_lock(&nvmf_dev_mutex);`
`1098`	`1097`	`ctrl = seq_file->private;`
`@@ -1106,7 +1105,7 @@ static int nvmf_dev_show(struct seq_file seq_file, void private)`
`1106`	`1105`
`1107`	`1106`	`out_unlock:`
`1108`	`1107`	`mutex_unlock(&nvmf_dev_mutex);`
`1109`		`- return ret;`
	`1108`	`+ return 0;`
`1110`	`1109`	`}`
`1111`	`1110`
`1112`	`1111`	`static int nvmf_dev_open(struct inode inode, struct file file)`