From: Roel Kluin <roel.kluin@xxxxxxxxx> In the comment on top of lpfc_sli_hbq_setup() and of its caller which transmits this error, lpfc_sli_hba_setup(), it is stated that the function should return a negative error code. Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx> Acked-by: James Smart <james.smart@xxxxxxxxxx> Cc: James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/scsi/lpfc/lpfc_sli.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/scsi/lpfc/lpfc_sli.c~lpfc-positive-error-return-into-negative drivers/scsi/lpfc/lpfc_sli.c --- a/drivers/scsi/lpfc/lpfc_sli.c~lpfc-positive-error-return-into-negative +++ a/drivers/scsi/lpfc/lpfc_sli.c @@ -3791,7 +3791,7 @@ lpfc_sli_hbq_setup(struct lpfc_hba *phba phba->link_state = LPFC_HBA_ERROR; mempool_free(pmb, phba->mbox_mem_pool); - return ENXIO; + return -ENXIO; } } phba->hbq_count = hbq_count; _ -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html