Commit 5860d9fb authored by Colin Ian King's avatar Colin Ian King Committed by Martin K. Petersen
Browse files

scsi: lpfc: Return NULL rather than a plain 0 integer 

Function lpfc_sli4_perform_vport_cvl() returns a pointer to struct
lpfc_nodelist so returning a plain 0 integer isn't good practice.  Fix this
by returning a NULL instead.

Link: https://lore.kernel.org/r/20210925224113.183040-1-colin.king@canonical.com


Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 9f80eca4

drivers/scsi/lpfc/lpfc_init.c

+1 −1
Original line number Diff line number Diff line
@@ -6564,7 +6564,7 @@ lpfc_sli4_perform_vport_cvl(struct lpfc_vport *vport) 
		/* Cannot find existing Fabric ndlp, so allocate a new one */

		ndlp = lpfc_nlp_init(vport, Fabric_DID);

		if (!ndlp)

			return 0;

			return NULL;

		/* Set the node type */

		ndlp->nlp_type |= NLP_FABRIC;

		/* Put ndlp onto node list */