2019-11-01 12:43:46

by Saurav Girepunje

[permalink] [raw]
Subject: [PATCH] scsi: ibmvscsi_tgt: Remove unneeded variable rc

Variable rc is not modified in ibmvscsis_srp_i_logout function.
So remove unneeded variable rc.

Issue found using coccicheck tool.

Signed-off-by: Saurav Girepunje <[email protected]>
---
drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c b/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c
index 7f9535392a93..09a4dbd0a9e5 100644
--- a/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c
+++ b/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c
@@ -2353,7 +2353,6 @@ static long ibmvscsis_srp_i_logout(struct scsi_info *vscsi,
{
struct iu_entry *iue = cmd->iue;
struct srp_i_logout *log_out = &vio_iu(iue)->srp.i_logout;
- long rc = ADAPT_SUCCESS;

if ((vscsi->debit > 0) || !list_empty(&vscsi->schedule_q) ||
!list_empty(&vscsi->waiting_rsp)) {
@@ -2369,7 +2368,7 @@ static long ibmvscsis_srp_i_logout(struct scsi_info *vscsi,
ibmvscsis_post_disconnect(vscsi, WAIT_IDLE, 0);
}

- return rc;
+ return ADAPT_SUCCESS;
}

/* Called with intr lock held */
--
2.20.1


2019-11-21 01:12:40

by Tyrel Datwyler

[permalink] [raw]
Subject: Re: [PATCH] scsi: ibmvscsi_tgt: Remove unneeded variable rc

On 11/1/19 5:35 AM, Saurav Girepunje wrote:
> Variable rc is not modified in ibmvscsis_srp_i_logout function.
> So remove unneeded variable rc.
>
> Issue found using coccicheck tool.
>
> Signed-off-by: Saurav Girepunje <[email protected]>

Reviewed-by: Tyrel Datwyler <[email protected]>

2019-11-22 01:52:14

by Martin K. Petersen

[permalink] [raw]
Subject: Re: [PATCH] scsi: ibmvscsi_tgt: Remove unneeded variable rc


>> Variable rc is not modified in ibmvscsis_srp_i_logout function. So
>> remove unneeded variable rc.
>>
>> Issue found using coccicheck tool.
>>
>> Signed-off-by: Saurav Girepunje <[email protected]>
>
> Reviewed-by: Tyrel Datwyler <[email protected]>

Applied to 5.5/scsi-queue, thanks.

--
Martin K. Petersen Oracle Linux Engineering