From bfae8c8ad5adc1796d7982feb2dd4a0bd947ba84 Mon Sep 17 00:00:00 2001 From: Pan Chuang Date: Thu, 9 Jul 2026 11:34:15 +0800 Subject: [PATCH] power: supply: sbs-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-37-panchuang@vivo.com Signed-off-by: Sebastian Reichel --- drivers/power/supply/sbs-charger.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/power/supply/sbs-charger.c b/drivers/power/supply/sbs-charger.c index a00c710601e8..e7b4e10dcfd5 100644 --- a/drivers/power/supply/sbs-charger.c +++ b/drivers/power/supply/sbs-charger.c @@ -217,7 +217,7 @@ static int sbs_probe(struct i2c_client *client) IRQF_TRIGGER_FALLING | IRQF_ONESHOT, dev_name(&client->dev), chip); if (ret) - return dev_err_probe(&client->dev, ret, "Failed to request irq\n"); + return ret; } else { ret = devm_delayed_work_autocancel(&client->dev, &chip->work, sbs_delayed_work);