diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-03-22 08:58:23 -0600 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2013-03-22 08:58:23 -0600 |
commit | d137c8306c748d89260400176613b5a85574b255 (patch) | |
tree | b8f175724538e06ecec6680656626ac25ce2ffd4 /drivers/block | |
parent | 7fbaee72ff62843198980c258d09590536681b15 (diff) | |
download | op-kernel-dev-d137c8306c748d89260400176613b5a85574b255.zip op-kernel-dev-d137c8306c748d89260400176613b5a85574b255.tar.gz |
mtip32xx: fix error return code in mtip_pci_probe()
Fix to return a negative error code from the error handling
case instead of 0, as returned elsewhere in this function.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/mtip32xx/mtip32xx.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/block/mtip32xx/mtip32xx.c b/drivers/block/mtip32xx/mtip32xx.c index 11cc952..92250af 100644 --- a/drivers/block/mtip32xx/mtip32xx.c +++ b/drivers/block/mtip32xx/mtip32xx.c @@ -4224,6 +4224,7 @@ static int mtip_pci_probe(struct pci_dev *pdev, dd->isr_workq = create_workqueue(dd->workq_name); if (!dd->isr_workq) { dev_warn(&pdev->dev, "Can't create wq %d\n", dd->instance); + rv = -ENOMEM; goto block_initialize_err; } @@ -4282,7 +4283,8 @@ static int mtip_pci_probe(struct pci_dev *pdev, INIT_WORK(&dd->work[7].work, mtip_workq_sdbf7); pci_set_master(pdev); - if (pci_enable_msi(pdev)) { + rv = pci_enable_msi(pdev); + if (rv) { dev_warn(&pdev->dev, "Unable to enable MSI interrupt.\n"); goto block_initialize_err; |