ipr: Fix error return code in ipr_probe_ioa()

weiyj · martinkpetersen · commit c8e18acccb9b · 2016-08-02T01:19:18.000-04:00
Fix to return error code -ENOMEM from the workqueue alloc error handling
case instead of 0, as done elsewhere in this function.

Signed-off-by: Wei Yongjun &lt;weiyj.lk@gmail.com&gt;
Acked-by: Brian King &lt;brking@linux.vnet.ibm.com&gt;
Signed-off-by: Martin K. Petersen &lt;martin.petersen@oracle.com&gt;
diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c
@@ -10219,6 +10219,7 @@ static int ipr_probe_ioa(struct pci_dev *pdev,
 
 		if (!ioa_cfg->reset_work_q) {
 			dev_err(&pdev->dev, "Couldn't register reset workqueue\n");
+			rc = -ENOMEM;
 			goto out_free_irq;
 		}
 	} else

Original file line number	Diff line number	Diff line change
`@@ -10219,6 +10219,7 @@ static int ipr_probe_ioa(struct pci_dev *pdev,`
`10219`	`10219`
`10220`	`10220`	`if (!ioa_cfg->reset_work_q) {`
`10221`	`10221`	`dev_err(&pdev->dev, "Couldn't register reset workqueue\n");`
	`10222`	`+ rc = -ENOMEM;`
`10222`	`10223`	`goto out_free_irq;`
`10223`	`10224`	`}`
`10224`	`10225`	`} else`