diff mbox series

dmaengine: uniphier-xdmac: Remove redandant error log for platform_get_irq

Message ID 20200323171928.424223-1-vkoul@kernel.org (mailing list archive)
State Accepted
Headers show
Series dmaengine: uniphier-xdmac: Remove redandant error log for platform_get_irq | expand

Commit Message

Vinod Koul March 23, 2020, 5:19 p.m. UTC
platform_get_irq prints the error on failure, so there is no need to
have caller add a log.
Remove the log in uniphier_xdmac_probe() for platform_get_irq() failure

Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/dma/uniphier-xdmac.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Vinod Koul March 26, 2020, 6:54 a.m. UTC | #1
On 23-03-20, 22:49, Vinod Koul wrote:
> platform_get_irq prints the error on failure, so there is no need to
> have caller add a log.
> Remove the log in uniphier_xdmac_probe() for platform_get_irq() failure

Applied, thanks
diff mbox series

Patch

diff --git a/drivers/dma/uniphier-xdmac.c b/drivers/dma/uniphier-xdmac.c
index 2f6fd518d180..7b2f8a8c2d31 100644
--- a/drivers/dma/uniphier-xdmac.c
+++ b/drivers/dma/uniphier-xdmac.c
@@ -525,10 +525,8 @@  static int uniphier_xdmac_probe(struct platform_device *pdev)
 		uniphier_xdmac_chan_init(xdev, i);
 
 	irq = platform_get_irq(pdev, 0);
-	if (irq < 0) {
-		dev_err(dev, "Failed to get IRQ\n");
+	if (irq < 0)
 		return irq;
-	}
 
 	ret = devm_request_irq(dev, irq, uniphier_xdmac_irq_handler,
 			       IRQF_SHARED, "xdmac", xdev);