diff options
| author | Wei Yongjun <weiyongjun1@huawei.com> | 2016-09-08 11:58:04 -0400 |
|---|---|---|
| committer | Borislav Petkov <bp@suse.de> | 2016-09-09 13:27:22 -0400 |
| commit | 43fa9ba632bd90b5cf63793a461e28df6af4387a (patch) | |
| tree | 2ea42284cd130fc08eeacdac65565bf304233596 /drivers/edac/fsl_ddr_edac.c | |
| parent | 7d136731e9ba695a6cf775c3a804456eeef4527e (diff) | |
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/fsl_ddr_edac.c')
| -rw-r--r-- | drivers/edac/fsl_ddr_edac.c | 3 |
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) | |||
| 564 | /* clear all error bits */ | 564 | /* clear all error bits */ |
| 565 | ddr_out32(pdata->mc_vbase + FSL_MC_ERR_DETECT, ~0); | 565 | ddr_out32(pdata->mc_vbase + FSL_MC_ERR_DETECT, ~0); |
| 566 | 566 | ||
| 567 | if (edac_mc_add_mc_with_groups(mci, fsl_ddr_dev_groups)) { | 567 | res = edac_mc_add_mc_with_groups(mci, fsl_ddr_dev_groups); |
| 568 | if (res) { | ||
| 568 | edac_dbg(3, "failed edac_mc_add_mc()\n"); | 569 | edac_dbg(3, "failed edac_mc_add_mc()\n"); |
| 569 | goto err; | 570 | goto err; |
| 570 | } | 571 | } |