From d77b6dd6903da71896aa20863ee51df7f54b9e7c Mon Sep 17 00:00:00 2001 From: Pan Chuang Date: Thu, 9 Jul 2026 11:33:53 +0800 Subject: [PATCH] power: supply: bq24735-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-15-panchuang@vivo.com Signed-off-by: Sebastian Reichel --- drivers/power/supply/bq24735-charger.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/power/supply/bq24735-charger.c b/drivers/power/supply/bq24735-charger.c index 99abbaf0470f..22c2b86c3b30 100644 --- a/drivers/power/supply/bq24735-charger.c +++ b/drivers/power/supply/bq24735-charger.c @@ -462,12 +462,8 @@ static int bq24735_charger_probe(struct i2c_client *client) IRQF_ONESHOT, supply_desc->name, charger->charger); - if (ret) { - dev_err(&client->dev, - "Unable to register IRQ %d err %d\n", - client->irq, ret); + if (ret) return ret; - } } else { ret = device_property_read_u32(&client->dev, "poll-interval", &charger->poll_interval);