diff options
author | Tang Bin <tangbin@cmss.chinamobile.com> | 2021-08-11 13:26:26 +0200 |
---|---|---|
committer | Borislav Petkov <bp@suse.de> | 2021-10-07 19:16:01 +0200 |
commit | 0b6d4ab2165c46c7f236744a2d05264f40172ae9 (patch) | |
tree | f8e73da1b975b1c5499cb78a61974d97d2d0ef7a /drivers/edac | |
parent | EDAC/amd64: Handle three rank interleaving mode (diff) | |
download | linux-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.c | 7 |
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; } |