From 4f22a6bd5130779014d19a9946bda07768325930 Mon Sep 17 00:00:00 2001 From: Pan Chuang Date: Thu, 9 Jul 2026 11:33:54 +0800 Subject: [PATCH] power: supply: bq256xx_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-16-panchuang@vivo.com Signed-off-by: Sebastian Reichel --- drivers/power/supply/bq256xx_charger.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/power/supply/bq256xx_charger.c b/drivers/power/supply/bq256xx_charger.c index 5a6634fde837..4ea89b7897c8 100644 --- a/drivers/power/supply/bq256xx_charger.c +++ b/drivers/power/supply/bq256xx_charger.c @@ -1752,10 +1752,8 @@ static int bq256xx_probe(struct i2c_client *client) IRQF_TRIGGER_FALLING | IRQF_ONESHOT, dev_name(&client->dev), bq); - if (ret < 0) { - dev_err(dev, "get irq fail: %d\n", ret); + if (ret < 0) return ret; - } } ret = bq256xx_hw_init(bq);