From 88a66a3c93fe21f8fb6904a2ef95d53e637b355b Mon Sep 17 00:00:00 2001 From: Pan Chuang Date: Thu, 9 Jul 2026 11:33:46 +0800 Subject: [PATCH] power: supply: act8945a_charger: Remove redundant dev_err()/dev_err_probe() The devm_request_threaded_irq() and devm_request_irq() now automatically log 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 Link: https://patch.msgid.link/20260709033428.362970-8-panchuang@vivo.com Signed-off-by: Sebastian Reichel --- drivers/power/supply/act8945a_charger.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/power/supply/act8945a_charger.c b/drivers/power/supply/act8945a_charger.c index 9dec4486b143..7e4b087f7679 100644 --- a/drivers/power/supply/act8945a_charger.c +++ b/drivers/power/supply/act8945a_charger.c @@ -620,10 +620,8 @@ static int act8945a_charger_probe(struct platform_device *pdev) ret = devm_request_irq(&pdev->dev, irq, act8945a_status_changed, IRQF_TRIGGER_FALLING, "act8945a_interrupt", charger); - if (ret) { - dev_err(&pdev->dev, "failed to request nIRQ pin IRQ\n"); + if (ret) return ret; - } platform_set_drvdata(pdev, charger);