From 1fa348a2a1825a23ab7836671f067640c98174bd Mon Sep 17 00:00:00 2001 From: Pan Chuang Date: Thu, 9 Jul 2026 11:33:52 +0800 Subject: [PATCH] power: supply: bq24257_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-14-panchuang@vivo.com Signed-off-by: Sebastian Reichel --- drivers/power/supply/bq24257_charger.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/power/supply/bq24257_charger.c b/drivers/power/supply/bq24257_charger.c index 72f1bfea8d54..d6286be4af74 100644 --- a/drivers/power/supply/bq24257_charger.c +++ b/drivers/power/supply/bq24257_charger.c @@ -1052,10 +1052,8 @@ static int bq24257_probe(struct i2c_client *client) IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING | IRQF_ONESHOT, bq->info->name, bq); - if (ret) { - dev_err(dev, "Failed to request IRQ #%d\n", client->irq); + if (ret) return ret; - } return 0; }