* [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure
@ 2026-07-09 2:30 Pan Chuang
2026-07-09 2:30 ` [PATCH 15/18] thermal/drivers/exynos: Remove redundant dev_err() Pan Chuang
2026-07-09 7:09 ` [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure Krzysztof Kozlowski
0 siblings, 2 replies; 5+ messages in thread
From: Pan Chuang @ 2026-07-09 2:30 UTC (permalink / raw)
To: Rafael J. Wysocki, Daniel Lezcano, Zhang Rui, Lukasz Luba,
Miquel Raynal, Markus Mayer, Broadcom internal kernel review list,
Florian Fainelli, Frank Li, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, zhanghongchen, Yinbo Zhu, Amit Kucheria,
Thara Gopinath, Niklas Söderlund, Geert Uytterhoeven,
Magnus Damm, John Madieu, Heiko Stuebner,
Bartlomiej Zolnierkiewicz, Krzysztof Kozlowski, Peter Griffin,
Alim Akhtar, Maxime Coquelin, Alexandre Torgue, Thierry Reding,
Jonathan Hunter, Matthias Brugger, AngeloGioacchino Del Regno,
Srinivas Pandruvada, Pan Chuang, Laura Nao, Mason Chang, Fei Shao,
Frank Wunderlich, Jiapeng Chong, Andy Shevchenko,
Jiri Slaby (SUSE), Svyatoslav Ryhel, open list:THERMAL, open list,
moderated list:BROADCOM BCM7XXX ARM ARCHITECTURE,
open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE,
open list:QUALCOMM TSENS THERMAL DRIVER,
open list:RENESAS R-CAR THERMAL DRIVERS,
open list:ARM/Rockchip SoC support,
open list:SAMSUNG THERMAL DRIVER,
moderated list:ARM/STM32 ARCHITECTURE,
open list:TEGRA ARCHITECTURE SUPPORT,
moderated list:ARM/Mediatek SoC support:Keyword:mediatek
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, which 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.
Remove the now-redundant dev_err() and dev_err_probe() calls in
thermal drivers that follow these devm_request_*_irq() functions, as
the core now provides more detailed diagnostic information on failure.
Pan Chuang (18):
thermal/drivers/airoha: Remove redundant dev_err()
thermal/drivers/armada: Remove redundant dev_err()
thermal/drivers/db8500: Remove redundant dev_err()
thermal/drivers/hisi: Remove redundant dev_err()
thermal/drivers/imx: Remove redundant dev_err()
thermal/drivers/loongson2: Remove redundant dev_err_probe()
thermal/drivers/max77620: Remove redundant dev_err()
thermal/drivers/rockchip: Remove redundant dev_err_probe()
thermal/drivers/brcmstb_thermal: Remove redundant dev_err_probe()
thermal: intel: int340x: Remove redundant dev_err()
thermal/drivers/intel/bxt_pmic: Remove redundant dev_err()
thermal/drivers/mediatek/lvts_thermal: Remove redundant dev_err()
thermal/drivers/qcom: Remove redundant dev_err()
thermal/drivers/renesas: Remove redundant dev_err()
thermal/drivers/exynos: Remove redundant dev_err()
thermal/drivers/st: Remove redundant dev_err()
thermal/drivers/tegra: Remove redundant dev_err() and dev_err_probe()
thermal/drivers/imx91: Remove redundant dev_err_probe()
drivers/thermal/airoha_thermal.c | 4 +---
drivers/thermal/armada_thermal.c | 5 +----
drivers/thermal/broadcom/brcmstb_thermal.c | 3 +--
drivers/thermal/db8500_thermal.c | 8 ++------
drivers/thermal/hisi_thermal.c | 4 +---
drivers/thermal/imx91_thermal.c | 2 +-
drivers/thermal/imx_thermal.c | 4 +---
.../intel/int340x_thermal/processor_thermal_device_pci.c | 8 ++------
drivers/thermal/intel/intel_bxt_pmic_thermal.c | 4 +---
drivers/thermal/loongson2_thermal.c | 2 +-
drivers/thermal/max77620_thermal.c | 8 ++------
drivers/thermal/mediatek/lvts_thermal.c | 2 +-
drivers/thermal/qcom/lmh.c | 1 -
drivers/thermal/qcom/tsens.c | 5 +----
drivers/thermal/renesas/rcar_thermal.c | 4 +---
drivers/thermal/renesas/rzg3e_thermal.c | 4 +---
drivers/thermal/rockchip_thermal.c | 3 +--
drivers/thermal/samsung/exynos_tmu.c | 4 +---
drivers/thermal/st/st_thermal_memmap.c | 4 +---
drivers/thermal/st/stm_thermal.c | 5 +----
drivers/thermal/tegra/soctherm.c | 8 ++------
drivers/thermal/tegra/tegra30-tsensor.c | 3 +--
22 files changed, 25 insertions(+), 70 deletions(-)
--
2.34.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 15/18] thermal/drivers/exynos: Remove redundant dev_err()
2026-07-09 2:30 [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure Pan Chuang
@ 2026-07-09 2:30 ` Pan Chuang
2026-07-09 7:09 ` [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure Krzysztof Kozlowski
1 sibling, 0 replies; 5+ messages in thread
From: Pan Chuang @ 2026-07-09 2:30 UTC (permalink / raw)
To: Bartlomiej Zolnierkiewicz, Krzysztof Kozlowski, Rafael J. Wysocki,
Daniel Lezcano, Zhang Rui, Lukasz Luba, Peter Griffin,
Alim Akhtar, open list:SAMSUNG THERMAL DRIVER,
open list:SAMSUNG THERMAL DRIVER,
moderated list:ARM/SAMSUNG S3C, S5P AND EXYNOS ARM ARCHITECTURES,
open list
Cc: Pan Chuang
The devm_request_threaded_irq() now automatically logs detailed error
messages on failure. This eliminates the need for driver-specific
dev_err() calls that previously printed generic messages.
Signed-off-by: Pan Chuang <panchuang@vivo.com>
---
drivers/thermal/samsung/exynos_tmu.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c
index 47a99b3c5395..56717bb50d60 100644
--- a/drivers/thermal/samsung/exynos_tmu.c
+++ b/drivers/thermal/samsung/exynos_tmu.c
@@ -1102,10 +1102,8 @@ static int exynos_tmu_probe(struct platform_device *pdev)
IRQF_TRIGGER_RISING
| IRQF_SHARED | IRQF_ONESHOT,
dev_name(dev), data);
- if (ret) {
- dev_err(dev, "Failed to request irq: %d\n", data->irq);
+ if (ret)
goto err_sclk;
- }
exynos_tmu_control(pdev, true);
return 0;
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure
2026-07-09 2:30 [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure Pan Chuang
2026-07-09 2:30 ` [PATCH 15/18] thermal/drivers/exynos: Remove redundant dev_err() Pan Chuang
@ 2026-07-09 7:09 ` Krzysztof Kozlowski
2026-07-09 7:24 ` Wolfram Sang
1 sibling, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2026-07-09 7:09 UTC (permalink / raw)
To: Pan Chuang, Rafael J. Wysocki, Daniel Lezcano, Zhang Rui,
Lukasz Luba, Miquel Raynal, Markus Mayer,
Broadcom internal kernel review list, Florian Fainelli, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
zhanghongchen, Yinbo Zhu, Amit Kucheria, Thara Gopinath,
Niklas Söderlund, Geert Uytterhoeven, Magnus Damm,
John Madieu, Heiko Stuebner, Bartlomiej Zolnierkiewicz,
Peter Griffin, Alim Akhtar, Maxime Coquelin, Alexandre Torgue,
Thierry Reding, Jonathan Hunter, Matthias Brugger,
AngeloGioacchino Del Regno, Srinivas Pandruvada, Laura Nao,
Mason Chang, Fei Shao, Frank Wunderlich, Jiapeng Chong,
Andy Shevchenko, Jiri Slaby (SUSE), Svyatoslav Ryhel,
open list:THERMAL, open list,
moderated list:BROADCOM BCM7XXX ARM ARCHITECTURE,
open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE,
open list:QUALCOMM TSENS THERMAL DRIVER,
open list:RENESAS R-CAR THERMAL DRIVERS,
open list:ARM/Rockchip SoC support,
open list:SAMSUNG THERMAL DRIVER,
moderated list:ARM/STM32 ARCHITECTURE,
open list:TEGRA ARCHITECTURE SUPPORT,
moderated list:ARM/Mediatek SoC support:Keyword:mediatek
On 09/07/2026 04:30, Pan Chuang wrote:
> 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, which 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.
>
> Remove the now-redundant dev_err() and dev_err_probe() calls in
> thermal drivers that follow these devm_request_*_irq() functions, as
> the core now provides more detailed diagnostic information on failure.
>
> Pan Chuang (18):
> thermal/drivers/airoha: Remove redundant dev_err()
> thermal/drivers/armada: Remove redundant dev_err()
> thermal/drivers/db8500: Remove redundant dev_err()
all of these 18 are just one commit, no?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure
2026-07-09 7:09 ` [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure Krzysztof Kozlowski
@ 2026-07-09 7:24 ` Wolfram Sang
2026-07-15 6:52 ` Pan Chuang
0 siblings, 1 reply; 5+ messages in thread
From: Wolfram Sang @ 2026-07-09 7:24 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Pan Chuang, Rafael J. Wysocki, Daniel Lezcano, Zhang Rui,
Lukasz Luba, Miquel Raynal, Markus Mayer,
Broadcom internal kernel review list, Florian Fainelli, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
zhanghongchen, Yinbo Zhu, Amit Kucheria, Thara Gopinath,
Niklas Söderlund, Geert Uytterhoeven, Magnus Damm,
John Madieu, Heiko Stuebner, Bartlomiej Zolnierkiewicz,
Peter Griffin, Alim Akhtar, Maxime Coquelin, Alexandre Torgue,
Thierry Reding, Jonathan Hunter, Matthias Brugger,
AngeloGioacchino Del Regno, Srinivas Pandruvada, Laura Nao,
Mason Chang, Fei Shao, Frank Wunderlich, Jiapeng Chong,
Andy Shevchenko, Jiri Slaby (SUSE), Svyatoslav Ryhel,
open list:THERMAL, open list,
moderated list:BROADCOM BCM7XXX ARM ARCHITECTURE,
open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE,
open list:QUALCOMM TSENS THERMAL DRIVER,
open list:RENESAS R-CAR THERMAL DRIVERS,
open list:ARM/Rockchip SoC support,
open list:SAMSUNG THERMAL DRIVER,
moderated list:ARM/STM32 ARCHITECTURE,
open list:TEGRA ARCHITECTURE SUPPORT,
moderated list:ARM/Mediatek SoC support:Keyword:mediatek
> all of these 18 are just one commit, no?
I think so, too.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure
2026-07-09 7:24 ` Wolfram Sang
@ 2026-07-15 6:52 ` Pan Chuang
0 siblings, 0 replies; 5+ messages in thread
From: Pan Chuang @ 2026-07-15 6:52 UTC (permalink / raw)
To: wsa+renesas
Cc: Frank.Li, alexandre.torgue, alim.akhtar, amitk, andriy.shevchenko,
angelogioacchino.delregno, bcm-kernel-feedback-list, bzolnier,
clamor95, daniel.lezcano, festevam, florian.fainelli, frank-w,
fshao, geert+renesas, heiko, imx, jiapeng.chong, jirislaby,
john.madieu.xa, jonathanh, kernel, krzk, laura.nao,
linux-arm-kernel, linux-arm-msm, linux-kernel, linux-mediatek,
linux-pm, linux-renesas-soc, linux-rockchip, linux-samsung-soc,
linux-stm32, linux-tegra, lukasz.luba, magnus.damm,
mason-cw.chang, matthias.bgg, mcoquelin.stm32, miquel.raynal,
mmayer, niklas.soderlund, panchuang, peter.griffin, rafael,
rui.zhang, s.hauer, srinivas.pandruvada, thara.gopinath,
thierry.reding, zhanghongchen, zhuyinbo
On 2026/7/9 15:24, Wolfram Sang wrote:
>
>> all of these 18 are just one commit, no?
>
> I think so, too.
>
Thanks, I'll merge them in v2.
Best Regards,
PanChuang
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2026-07-15 6:53 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-09 2:30 [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure Pan Chuang
2026-07-09 2:30 ` [PATCH 15/18] thermal/drivers/exynos: Remove redundant dev_err() Pan Chuang
2026-07-09 7:09 ` [PATCH 00/18] thermal/drivers: Remove redundant error messages on IRQ request failure Krzysztof Kozlowski
2026-07-09 7:24 ` Wolfram Sang
2026-07-15 6:52 ` Pan Chuang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox