scsi: lpfc: Remove unneeded result variable
authorye xingchen <ye.xingchen@zte.com.cn>
Wed, 24 Aug 2022 07:51:23 +0000 (07:51 +0000)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 1 Sep 2022 04:39:05 +0000 (00:39 -0400)
Return the value from lpfc_issue_reg_vfi() directly instead of storing it
in another redundant variable.

Link: https://lore.kernel.org/r/20220824075123.221316-1-ye.xingchen@zte.com.cn
Reported-by: Zeal Robot <zealci@zte.com.cn>
Reviewed-by: James Smart <jsmart2021@gmail.com>
Signed-off-by: ye xingchen <ye.xingchen@zte.com.cn>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/lpfc/lpfc_bsg.c

index 9be3bb01a8ec7630ac2b435ed9a25aae810ccdda..ac0c7ccf2eaee59cfc335d672543b3abfd485bd0 100644 (file)
@@ -1977,8 +1977,6 @@ lpfc_sli4_bsg_set_loopback_mode(struct lpfc_hba *phba, int mode,
 static int
 lpfc_sli4_diag_fcport_reg_setup(struct lpfc_hba *phba)
 {
-       int rc;
-
        if (phba->pport->fc_flag & FC_VFI_REGISTERED) {
                lpfc_printf_log(phba, KERN_WARNING, LOG_LIBDFC,
                                "3136 Port still had vfi registered: "
@@ -1988,8 +1986,7 @@ lpfc_sli4_diag_fcport_reg_setup(struct lpfc_hba *phba)
                                phba->vpi_ids[phba->pport->vpi]);
                return -EINVAL;
        }
-       rc = lpfc_issue_reg_vfi(phba->pport);
-       return rc;
+       return lpfc_issue_reg_vfi(phba->pport);
 }
 
 /**