scsi: fnic: Propagate SCSI error code from fnic_scsi_drv_init()

Karan Kumar · johnmeneghini · commit 65dc387dd673 · 2025-04-08T15:42:13.000-04:00
JIRA: https://issues.redhat.com/browse/RHEL-72902 Propagate scsi_add_host() error instead of returning -1. Suggested-by: Dan Carpenter <dan.carpenter@linaro.org> Reviewed-by: Sesidhar Baddela <sebaddel@cisco.com> Reviewed-by: Arulprabhu Ponnusamy <arulponn@cisco.com> Reviewed-by: Gian Carlo Boffa <gcboffa@cisco.com> Signed-off-by: Arun Easi <aeasi@cisco.com> Signed-off-by: Karan Tilak Kumar <kartilak@cisco.com> Link: https://lore.kernel.org/r/20250110091956.17749-1-kartilak@cisco.com Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> (cherry picked from commit 8697934) Signed-off-by: Karan Kumar <karkumar@redhat.com>
diff --git a/drivers/scsi/fnic/fnic_main.c b/drivers/scsi/fnic/fnic_main.c
@@ -636,7 +636,7 @@ static int fnic_scsi_drv_init(struct fnic *fnic)
 	err = scsi_add_host(fnic->host, &pdev->dev);
 	if (err) {
 		dev_err(&fnic->pdev->dev, "fnic: scsi add host failed: aborting\n");
-		return -1;
+		return err;
 	}
 	fc_host_maxframe_size(fnic->host) = iport->max_payload_size;
 	fc_host_dev_loss_tmo(fnic->host) =

Original file line number	Diff line number	Diff line change
`@@ -636,7 +636,7 @@ static int fnic_scsi_drv_init(struct fnic *fnic)`
`636`	`636`	`err = scsi_add_host(fnic->host, &pdev->dev);`
`637`	`637`	`if (err) {`
`638`	`638`	`dev_err(&fnic->pdev->dev, "fnic: scsi add host failed: aborting\n");`
`639`		`- return -1;`
	`639`	`+ return err;`
`640`	`640`	`}`
`641`	`641`	`fc_host_maxframe_size(fnic->host) = iport->max_payload_size;`
`642`	`642`	`fc_host_dev_loss_tmo(fnic->host) =`