From: Pan Chuang <panchuang@vivo.com>
To: Sebastian Reichel <sre@kernel.org>,
Linus Walleij <linusw@kernel.org>, Chen-Yu Tsai <wens@kernel.org>,
Hans de Goede <hansg@kernel.org>,
Support Opensource <support.opensource@diasemi.com>,
Amit Sunil Dhamne <amitsd@google.com>,
Saravanan Sekar <sravanhome@gmail.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
Samuel Kayode <samkay014@gmail.com>,
Casey Connolly <casey.connolly@linaro.org>,
Orson Zhai <orsonzhai@gmail.com>,
Baolin Wang <baolin.wang@linux.alibaba.com>,
Chunyan Zhang <zhang.lyra@gmail.com>,
linux-pm@vger.kernel.org (open list:SYSTEM RESET/SHUTDOWN
DRIVERS), linux-kernel@vger.kernel.org (open list),
linux-arm-kernel@lists.infradead.org (moderated
list:ARM/Mediatek SoC support),
linux-mediatek@lists.infradead.org (moderated list:ARM/Mediatek
SoC support),
imx@lists.linux.dev (open list:NXP PF1550 PMIC MFD DRIVER),
linux-arm-msm@vger.kernel.org (open list:ARM/QUALCOMM MAILING
LIST)
Cc: Pan Chuang <panchuang@vivo.com>
Subject: [PATCH 00/43] power: Remove redundant error messages on IRQ request failure
Date: Thu, 9 Jul 2026 11:33:39 +0800 [thread overview]
Message-ID: <20260709033428.362970-1-panchuang@vivo.com> (raw)
Commit 55b48e23f5c4b6f5ca9b7ab09599b17dcf501c10 ("genirq/devres: Add
error handling in devm_request_*_irq()") added automatic error logging
to devm_request_threaded_irq() and devm_request_any_context_irq() via
the new devm_request_result() helper. The helper prints device name,
IRQ number, handler functions, and error code on failure.
Since devm_request_irq() is a static inline wrapper around
devm_request_threaded_irq(), it also benefits from this automatic
logging.
This series removes the now-redundant dev_err() and dev_err_probe() calls
in power drivers that follow these devm_request_*_irq() functions, as the
core now provides more detailed diagnostic information on failure.
Pan Chuang (43):
power: reset: pwr-mlxbf: Remove redundant dev_err()/dev_err_probe()
power: supply: 88pm860x_battery: Remove redundant
dev_err()/dev_err_probe()
power: supply: 88pm860x_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: ab8500_btemp: Remove redundant
dev_err()/dev_err_probe()
power: supply: ab8500_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: ab8500_fg: Remove redundant dev_err()/dev_err_probe()
power: supply: act8945a_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: axp20x_ac_power: Remove redundant
dev_err()/dev_err_probe()
power: supply: axp20x_usb_power: Remove redundant
dev_err()/dev_err_probe()
power: supply: axp288_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: axp288_fuel_gauge: Remove redundant
dev_err()/dev_err_probe()
power: supply: bq24190_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: bq24257_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: bq24735-charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: bq256xx_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: bq257xx_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: cpcap-battery: Remove redundant
dev_err()/dev_err_probe()
power: supply: cpcap-charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: da9150-fg: Remove redundant dev_err()/dev_err_probe()
power: supply: generic-adc-battery: Remove redundant
dev_err()/dev_err_probe()
power: supply: max14656_charger_detector: Remove redundant
dev_err()/dev_err_probe()
power: supply: max77759_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: max8903_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: max8971_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: mp2629_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: mt6360_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: mt6370-charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: pf1550-charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: qcom_smbb: Remove redundant dev_err()/dev_err_probe()
power: supply: qcom_smbx: Remove redundant dev_err()/dev_err_probe()
power: supply: rk817_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: rn5t618_power: Remove redundant
dev_err()/dev_err_probe()
power: supply: rt9455_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: rt9467-charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: rt9471: Remove redundant dev_err()/dev_err_probe()
power: supply: sbs-charger: Remove redundant dev_err()/dev_err_probe()
power: supply: sc27xx_fuel_gauge: Remove redundant
dev_err()/dev_err_probe()
power: supply: surface-rt-ec: Remove redundant
dev_err()/dev_err_probe()
power: supply: tps65090-charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: tps65217_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: twl4030_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: twl6030_charger: Remove redundant
dev_err()/dev_err_probe()
power: supply: ucs1002_power: Remove redundant
dev_err()/dev_err_probe()
drivers/power/reset/pwr-mlxbf.c | 2 --
drivers/power/supply/88pm860x_battery.c | 10 ++--------
drivers/power/supply/88pm860x_charger.c | 5 +----
drivers/power/supply/ab8500_btemp.c | 5 +----
drivers/power/supply/ab8500_charger.c | 5 +----
drivers/power/supply/ab8500_fg.c | 2 --
drivers/power/supply/act8945a_charger.c | 4 +---
drivers/power/supply/axp20x_ac_power.c | 5 +----
drivers/power/supply/axp20x_usb_power.c | 5 +----
drivers/power/supply/axp288_charger.c | 3 +--
drivers/power/supply/axp288_fuel_gauge.c | 2 +-
drivers/power/supply/bq24190_charger.c | 4 +---
drivers/power/supply/bq24257_charger.c | 4 +---
drivers/power/supply/bq24735-charger.c | 6 +-----
drivers/power/supply/bq256xx_charger.c | 4 +---
drivers/power/supply/bq257xx_charger.c | 2 --
drivers/power/supply/cpcap-battery.c | 6 +-----
drivers/power/supply/cpcap-charger.c | 6 +-----
drivers/power/supply/da9150-fg.c | 4 +---
drivers/power/supply/generic-adc-battery.c | 2 +-
drivers/power/supply/max14656_charger_detector.c | 4 +---
drivers/power/supply/max77759_charger.c | 4 +---
drivers/power/supply/max8903_charger.c | 15 +++------------
drivers/power/supply/max8971_charger.c | 2 +-
drivers/power/supply/mp2629_charger.c | 4 +---
drivers/power/supply/mt6360_charger.c | 3 +--
drivers/power/supply/mt6370-charger.c | 4 +---
drivers/power/supply/pf1550-charger.c | 3 +--
drivers/power/supply/qcom_smbb.c | 5 +----
drivers/power/supply/qcom_smbx.c | 3 +--
drivers/power/supply/rk817_charger.c | 12 ++++--------
drivers/power/supply/rn5t618_power.c | 5 +----
drivers/power/supply/rt9455_charger.c | 4 +---
drivers/power/supply/rt9467-charger.c | 3 +--
drivers/power/supply/rt9471.c | 3 +--
drivers/power/supply/sbs-charger.c | 2 +-
drivers/power/supply/sc27xx_fuel_gauge.c | 8 ++------
drivers/power/supply/surface-rt-ec.c | 2 +-
drivers/power/supply/tps65090-charger.c | 6 +-----
drivers/power/supply/tps65217_charger.c | 6 +-----
drivers/power/supply/twl4030_charger.c | 10 ++--------
drivers/power/supply/twl6030_charger.c | 4 +---
drivers/power/supply/ucs1002_power.c | 10 ++--------
43 files changed, 49 insertions(+), 159 deletions(-)
--
2.34.1
next reply other threads:[~2026-07-09 3:35 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-09 3:33 Pan Chuang [this message]
2026-07-09 3:34 ` [PATCH 26/43] power: supply: mt6360_charger: Remove redundant dev_err()/dev_err_probe() Pan Chuang
2026-07-09 3:34 ` [PATCH 27/43] power: supply: mt6370-charger: " Pan Chuang
2026-07-18 1:15 ` [PATCH 00/43] power: Remove redundant error messages on IRQ request failure Sebastian Reichel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260709033428.362970-1-panchuang@vivo.com \
--to=panchuang@vivo.com \
--cc=amitsd@google.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=baolin.wang@linux.alibaba.com \
--cc=casey.connolly@linaro.org \
--cc=hansg@kernel.org \
--cc=imx@lists.linux.dev \
--cc=linusw@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=matthias.bgg@gmail.com \
--cc=orsonzhai@gmail.com \
--cc=samkay014@gmail.com \
--cc=sravanhome@gmail.com \
--cc=sre@kernel.org \
--cc=support.opensource@diasemi.com \
--cc=wens@kernel.org \
--cc=zhang.lyra@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox