summaryrefslogtreecommitdiffstats
path: root/drivers/edac
diff options
context:
space:
mode:
authorWei Yongjun <weiyongjun1@huawei.com>2016-09-08 15:58:04 +0000
committerBorislav Petkov <bp@suse.de>2016-09-09 19:27:22 +0200
commit43fa9ba632bd90b5cf63793a461e28df6af4387a (patch)
tree2ea42284cd130fc08eeacdac65565bf304233596 /drivers/edac
parent7d136731e9ba695a6cf775c3a804456eeef4527e (diff)
downloadtalos-obmc-linux-43fa9ba632bd90b5cf63793a461e28df6af4387a.tar.gz
talos-obmc-linux-43fa9ba632bd90b5cf63793a461e28df6af4387a.zip
EDAC, fsl_ddr: Fix error return code in fsl_mc_err_probe()
Return negative error code from the edac_mc_add_mc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> Acked-by: York Sun <york.sun@nxp.com> Cc: linux-edac <linux-edac@vger.kernel.org> Link: http://lkml.kernel.org/r/1473350284-26482-1-git-send-email-weiyj.lk@gmail.com Signed-off-by: Borislav Petkov <bp@suse.de>
Diffstat (limited to 'drivers/edac')
-rw-r--r--drivers/edac/fsl_ddr_edac.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/edac/fsl_ddr_edac.c b/drivers/edac/fsl_ddr_edac.c
index 4ddf8382369f..9774f52f0c3e 100644
--- a/drivers/edac/fsl_ddr_edac.c
+++ b/drivers/edac/fsl_ddr_edac.c
@@ -564,7 +564,8 @@ int fsl_mc_err_probe(struct platform_device *op)
/* clear all error bits */
ddr_out32(pdata->mc_vbase + FSL_MC_ERR_DETECT, ~0);
- if (edac_mc_add_mc_with_groups(mci, fsl_ddr_dev_groups)) {
+ res = edac_mc_add_mc_with_groups(mci, fsl_ddr_dev_groups);
+ if (res) {
edac_dbg(3, "failed edac_mc_add_mc()\n");
goto err;
}
OpenPOWER on IntegriCloud