summaryrefslogtreecommitdiffstats
path: root/drivers/edac
diff options
context:
space:
mode:
authorTang Bin <tangbin@cmss.chinamobile.com>2021-08-11 13:26:26 +0200
committerBorislav Petkov <bp@suse.de>2021-10-07 19:16:01 +0200
commit0b6d4ab2165c46c7f236744a2d05264f40172ae9 (patch)
treef8e73da1b975b1c5499cb78a61974d97d2d0ef7a /drivers/edac
parentEDAC/amd64: Handle three rank interleaving mode (diff)
downloadlinux-0b6d4ab2165c46c7f236744a2d05264f40172ae9.tar.xz
linux-0b6d4ab2165c46c7f236744a2d05264f40172ae9.zip
EDAC/ti: Remove redundant error messages
In the function ti_edac_probe(), devm_ioremap_resource() and platform_get_irq() already issue error messages when they fail so remove the redundant error messages in the EDAC driver. Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com> Signed-off-by: Borislav Petkov <bp@suse.de> Link: https://lkml.kernel.org/r/20210811112626.27848-1-tangbin@cmss.chinamobile.com
Diffstat (limited to 'drivers/edac')
-rw-r--r--drivers/edac/ti_edac.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/edac/ti_edac.c b/drivers/edac/ti_edac.c
index 169f96e51c29..6971ded598de 100644
--- a/drivers/edac/ti_edac.c
+++ b/drivers/edac/ti_edac.c
@@ -245,11 +245,8 @@ static int ti_edac_probe(struct platform_device *pdev)
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
reg = devm_ioremap_resource(dev, res);
- if (IS_ERR(reg)) {
- edac_printk(KERN_ERR, EDAC_MOD_NAME,
- "EMIF controller regs not defined\n");
+ if (IS_ERR(reg))
return PTR_ERR(reg);
- }
layers[0].type = EDAC_MC_LAYER_ALL_MEM;
layers[0].size = 1;
@@ -281,8 +278,6 @@ static int ti_edac_probe(struct platform_device *pdev)
error_irq = platform_get_irq(pdev, 0);
if (error_irq < 0) {
ret = error_irq;
- edac_printk(KERN_ERR, EDAC_MOD_NAME,
- "EMIF irq number not defined.\n");
goto err;
}