diff mbox series

scsi: qla2xxx: Remove redundant variable initialization

Message ID 20200802111527.4928-1-tianjia.zhang@linux.alibaba.com (mailing list archive)
State Accepted
Headers show
Series scsi: qla2xxx: Remove redundant variable initialization | expand

Commit Message

tianjia.zhang Aug. 2, 2020, 11:15 a.m. UTC
The initialization value of `rc` is wrong. It is unnecessary to
initialize `rc` variables, so remove its initialization operation.

Fixes: 84905dfe78d28 ("scsi: qla2xxx: Fix TMF and Multi-Queue config")
Cc: Quinn Tran <quinn.tran@cavium.com>
Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
---
 drivers/scsi/qla2xxx/qla_target.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Martin K. Petersen Aug. 31, 2020, 5:41 p.m. UTC | #1
On Sun, 2 Aug 2020 19:15:27 +0800, Tianjia Zhang wrote:

> The initialization value of `rc` is wrong. It is unnecessary to
> initialize `rc` variables, so remove its initialization operation.

Applied to 5.10/scsi-queue, thanks!

[1/1] scsi: qla2xxx: Remove redundant variable initialization
      https://git.kernel.org/mkp/scsi/c/8905cbdae986
diff mbox series

Patch

diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c
index fbb80a043b4f..c2c0e6049da4 100644
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -2025,7 +2025,7 @@  static void qlt_do_tmr_work(struct work_struct *work)
 	struct qla_tgt_mgmt_cmd *mcmd =
 		container_of(work, struct qla_tgt_mgmt_cmd, work);
 	struct qla_hw_data *ha = mcmd->vha->hw;
-	int rc = EIO;
+	int rc;
 	uint32_t tag;
 	unsigned long flags;