All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH -next] drm: Remove unnecessary print function dev_err()
@ 2022-07-07  1:47 ` Yang Li
  0 siblings, 0 replies; 12+ messages in thread
From: Yang Li @ 2022-07-07  1:47 UTC (permalink / raw)
  To: tomba; +Cc: airlied, Yang Li, dri-devel, linux-kernel

The print function dev_err() is redundant because platform_get_irq()
already prints an error.

Eliminate the follow coccicheck warning:
./drivers/gpu/drm/omapdrm/omap_dmm_tiler.c:817:2-9: line 817 is redundant because platform_get_irq() already prints an error

Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
index ac869acf80ea..61a27dd7392e 100644
--- a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
+++ b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
@@ -813,10 +813,8 @@ static int omap_dmm_probe(struct platform_device *dev)
 	}
 
 	omap_dmm->irq = platform_get_irq(dev, 0);
-	if (omap_dmm->irq < 0) {
-		dev_err(&dev->dev, "failed to get IRQ resource\n");
+	if (omap_dmm->irq < 0)
 		goto fail;
-	}
 
 	omap_dmm->dev = &dev->dev;
 
-- 
2.20.1.7.g153144c


^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH -next] drm: Remove unnecessary print function dev_err()
@ 2022-06-24  0:50 ` Yang Li
  0 siblings, 0 replies; 12+ messages in thread
From: Yang Li @ 2022-06-24  0:50 UTC (permalink / raw)
  To: airlied
  Cc: zhang.lyra, linux-kernel, dri-devel, Yang Li, baolin.wang7,
	orsonzhai

The print function dev_err() is redundant because platform_get_irq()
already prints an error.

Eliminate the follow coccicheck warning:
./drivers/gpu/drm/sprd/sprd_dpu.c:808:2-9: line 808 is redundant because platform_get_irq() already prints an error

Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 drivers/gpu/drm/sprd/sprd_dpu.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/sprd/sprd_dpu.c b/drivers/gpu/drm/sprd/sprd_dpu.c
index 3664089b6983..de4848dc0d22 100644
--- a/drivers/gpu/drm/sprd/sprd_dpu.c
+++ b/drivers/gpu/drm/sprd/sprd_dpu.c
@@ -804,10 +804,8 @@ static int sprd_dpu_context_init(struct sprd_dpu *dpu,
 	}
 
 	ctx->irq = platform_get_irq(pdev, 0);
-	if (ctx->irq < 0) {
-		dev_err(dev, "failed to get dpu irq\n");
+	if (ctx->irq < 0)
 		return ctx->irq;
-	}
 
 	/* disable and clear interrupts before register dpu IRQ. */
 	writel(0x00, ctx->base + REG_DPU_INT_EN);
-- 
2.20.1.7.g153144c


^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH -next] drm: Remove unnecessary print function dev_err()
@ 2022-06-13  1:26 ` Yang Li
  0 siblings, 0 replies; 12+ messages in thread
From: Yang Li @ 2022-06-13  1:26 UTC (permalink / raw)
  To: airlied; +Cc: Abaci Robot, linux-kernel, dri-devel, paul.kocialkowski, Yang Li

The print function dev_err() is redundant because platform_get_irq()
already prints an error.

Eliminate the follow coccicheck warning:
./drivers/gpu/drm/logicvc/logicvc_drm.c:352:2-9: line 352 is redundant
because platform_get_irq() already prints an error

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 drivers/gpu/drm/logicvc/logicvc_drm.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/logicvc/logicvc_drm.c b/drivers/gpu/drm/logicvc/logicvc_drm.c
index df1805cf0f95..437b3011ae1e 100644
--- a/drivers/gpu/drm/logicvc/logicvc_drm.c
+++ b/drivers/gpu/drm/logicvc/logicvc_drm.c
@@ -349,7 +349,6 @@ static int logicvc_drm_probe(struct platform_device *pdev)
 
 	irq = platform_get_irq(pdev, 0);
 	if (irq < 0) {
-		dev_err(dev, "Failed to get IRQ\n");
 		ret = -ENODEV;
 		goto error_reserved_mem;
 	}
-- 
2.20.1.7.g153144c


^ permalink raw reply related	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2022-07-07  1:47 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-07-07  1:47 [PATCH -next] drm: Remove unnecessary print function dev_err() Yang Li
2022-07-07  1:47 ` Yang Li
  -- strict thread matches above, loose matches on Subject: below --
2022-06-24  0:50 Yang Li
2022-06-24  0:50 ` Yang Li
2022-06-26  3:46 ` Orson Zhai
2022-06-26  3:46   ` Orson Zhai
2022-06-13  1:26 Yang Li
2022-06-13  1:26 ` Yang Li
2022-06-24 15:00 ` Paul Kocialkowski
2022-06-24 15:00   ` Paul Kocialkowski
2022-06-24 15:26   ` Paul Kocialkowski
2022-06-24 15:26     ` Paul Kocialkowski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.