lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 13 Jul 2016 12:46:57 +0000 From: weiyj_lk@....com To: "David S . Miller" <davem@...emloft.net> Cc: Wei Yongjun <yongjun_wei@...ndmicro.com.cn>, netdev@...r.kernel.org Subject: [PATCH -next] net: ethernet: bgmac: Remove redundant dev_err call in bgmac_probe() From: Wei Yongjun <yongjun_wei@...ndmicro.com.cn> There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun <yongjun_wei@...ndmicro.com.cn> --- drivers/net/ethernet/broadcom/bgmac-platform.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/net/ethernet/broadcom/bgmac-platform.c b/drivers/net/ethernet/broadcom/bgmac-platform.c index 1a2d841..be52f27 100644 --- a/drivers/net/ethernet/broadcom/bgmac-platform.c +++ b/drivers/net/ethernet/broadcom/bgmac-platform.c @@ -129,10 +129,8 @@ static int bgmac_probe(struct platform_device *pdev) } bgmac->plat.base = devm_ioremap_resource(&pdev->dev, regs); - if (IS_ERR(bgmac->plat.base)) { - dev_err(&pdev->dev, "Unable to map base resource\n"); + if (IS_ERR(bgmac->plat.base)) return PTR_ERR(bgmac->plat.base); - } regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "idm_base"); if (!regs) { @@ -141,10 +139,8 @@ static int bgmac_probe(struct platform_device *pdev) } bgmac->plat.idm_base = devm_ioremap_resource(&pdev->dev, regs); - if (IS_ERR(bgmac->plat.idm_base)) { - dev_err(&pdev->dev, "Unable to map idm resource\n"); + if (IS_ERR(bgmac->plat.idm_base)) return PTR_ERR(bgmac->plat.idm_base); - } bgmac->read = platform_bgmac_read; bgmac->write = platform_bgmac_write;
Powered by blists - more mailing lists