From 16447b9ef408768d94b9b15b589f287eb8ff2127 Mon Sep 17 00:00:00 2001 From: Pan Chuang Date: Thu, 9 Jul 2026 11:33:57 +0800 Subject: [PATCH] power: supply: cpcap-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-19-panchuang@vivo.com Signed-off-by: Sebastian Reichel --- drivers/power/supply/cpcap-charger.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/power/supply/cpcap-charger.c b/drivers/power/supply/cpcap-charger.c index 24221244b45b..b29b77698de7 100644 --- a/drivers/power/supply/cpcap-charger.c +++ b/drivers/power/supply/cpcap-charger.c @@ -762,12 +762,8 @@ static int cpcap_usb_init_irq(struct platform_device *pdev, cpcap_charger_irq_thread, IRQF_SHARED | IRQF_ONESHOT, name, ddata); - if (error) { - dev_err(ddata->dev, "could not get irq %s: %i\n", - name, error); - + if (error) return error; - } d = devm_kzalloc(ddata->dev, sizeof(*d), GFP_KERNEL); if (!d)