diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-06-18 22:42:35 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-06-24 19:23:25 -0400 |
commit | 0686ab7a7d05cda3373e311a49b2478bd4f03c5d (patch) | |
tree | b1eef9b7ae019f3cc2a7e740e254ba3950cd4dc6 /drivers/vme/bridges | |
parent | 4640a3f2bff64b808bdedadcddf882aa4606f374 (diff) |
vme: vme_tsi148.c: fix error return code in tsi148_probe()
Fix to return a negative error code in the tsi148_crcsr_init() error
handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/vme/bridges')
-rw-r--r-- | drivers/vme/bridges/vme_tsi148.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/vme/bridges/vme_tsi148.c b/drivers/vme/bridges/vme_tsi148.c index 0bb512cfad77..94c892f27be4 100644 --- a/drivers/vme/bridges/vme_tsi148.c +++ b/drivers/vme/bridges/vme_tsi148.c | |||
@@ -2582,7 +2582,8 @@ static int tsi148_probe(struct pci_dev *pdev, const struct pci_device_id *id) | |||
2582 | dev_info(&pdev->dev, "VME Write and flush and error check is %s\n", | 2582 | dev_info(&pdev->dev, "VME Write and flush and error check is %s\n", |
2583 | err_chk ? "enabled" : "disabled"); | 2583 | err_chk ? "enabled" : "disabled"); |
2584 | 2584 | ||
2585 | if (tsi148_crcsr_init(tsi148_bridge, pdev)) { | 2585 | retval = tsi148_crcsr_init(tsi148_bridge, pdev); |
2586 | if (retval) { | ||
2586 | dev_err(&pdev->dev, "CR/CSR configuration failed.\n"); | 2587 | dev_err(&pdev->dev, "CR/CSR configuration failed.\n"); |
2587 | goto err_crcsr; | 2588 | goto err_crcsr; |
2588 | } | 2589 | } |