Re: [PATCH 25/26] dmaengine: uniphier-xdmac: Remove redundant dev_err()/dev_err_probe()

From: Frank Li

Date: Fri Sep 04 2026 - 13:09:23 EST


On Thu, Jul 09, 2026 at 09:58:29PM +0800, Pan Chuang wrote:
> The devm_request_irq() now automatically logs detailed error messages on
> failure. This eliminates the need for driver-specific dev_err() and
> dev_err_probe() calls that previously printed generic messages.
>
> Signed-off-by: Pan Chuang <panchuang@xxxxxxxx>
> ---

Reviewed-by: Frank Li <Frank.Li@xxxxxxx>

> drivers/dma/uniphier-xdmac.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/dma/uniphier-xdmac.c b/drivers/dma/uniphier-xdmac.c
> index 120c0d4f12dd..b23d61bf7fd5 100644
> --- a/drivers/dma/uniphier-xdmac.c
> +++ b/drivers/dma/uniphier-xdmac.c
> @@ -532,10 +532,8 @@ static int uniphier_xdmac_probe(struct platform_device *pdev)
>
> ret = devm_request_irq(dev, irq, uniphier_xdmac_irq_handler,
> IRQF_SHARED, "xdmac", xdev);
> - if (ret) {
> - dev_err(dev, "Failed to request IRQ\n");
> + if (ret)
> return ret;
> - }
>
> ret = dma_async_device_register(ddev);
> if (ret) {
> --
> 2.34.1
>