Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1442412

[PATCH -next] drm/hisilicon: Remove redundant dev_err call in ade_dts_parse()

From weiyj_lk@163.com
Newsgroups linux.kernel
Subject [PATCH -next] drm/hisilicon: Remove redundant dev_err call in ade_dts_parse()
Date 2016-07-13 14:50 +0200
Message-ID <rUrLY-38m-31@gated-at.bofh.it> (permalink)
Organization linux.* mail to news gateway

Show all headers | View raw


From: Wei Yongjun <yongjun_wei@trendmicro.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@trendmicro.com.cn>
---
 drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c b/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c
index 805f432..3aea3bb 100644
--- a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c
+++ b/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c
@@ -944,10 +944,8 @@ static int ade_dts_parse(struct platform_device *pdev, struct ade_hw_ctx *ctx)
 
 	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
 	ctx->base = devm_ioremap_resource(dev, res);
-	if (IS_ERR(ctx->base)) {
-		DRM_ERROR("failed to remap ade io base\n");
+	if (IS_ERR(ctx->base))
 		return  PTR_ERR(ctx->base);
-	}
 
 	ctx->reset = devm_reset_control_get(dev, NULL);
 	if (IS_ERR(ctx->reset))

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread


Thread

[PATCH -next] drm/hisilicon: Remove redundant dev_err call in ade_dts_parse() weiyj_lk@163.com - 2016-07-13 14:50 +0200
  Re: [PATCH -next] drm/hisilicon: Remove redundant dev_err call in ade_dts_parse() Xinliang Liu <xinliang.liu@linaro.org> - 2016-07-18 03:30 +0200

csiph-web