Commit de05e484 authored by ye xingchen's avatar ye xingchen Committed by Martin K. Petersen
Browse files

scsi: lpfc: Remove unneeded result variable 

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: default avatarZeal Robot <zealci@zte.com.cn>
Reviewed-by: default avatarJames Smart <jsmart2021@gmail.com>
Signed-off-by: default avatarye xingchen <ye.xingchen@zte.com.cn>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 7fd080e1

drivers/scsi/lpfc/lpfc_bsg.c

+1 −4
Original line number Diff line number Diff line
@@ -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);

}



/**