[PATCH 0/7] soc: Remove redundant error messages on IRQ request failure

[PATCH 0/7] soc: Remove redundant error messages on IRQ request failure

[Pan Chuang]

Commit 55b48e23f5c4 ("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 soc drivers that follow these devm_request_*_irq()
functions, as the core now provides more detailed diagnostic
information on failure.

Pan Chuang (7):
  soc: fsl: dpio: Remove redundant dev_err()
  soc: ixp4xx: Remove redundant dev_err()
  soc: mediatek: mtk-svs: Remove redundant dev_err_probe()
  soc: qcom: Remove redundant dev_err()
  soc/tegra: cbb: Remove redundant dev_err()
  soc: ti: wkup_m3_ipc: Remove redundant dev_err()
  soc: xilinx: Remove redundant dev_err()

 drivers/soc/fsl/dpio/dpio-driver.c   |  6 +-----
 drivers/soc/ixp4xx/ixp4xx-qmgr.c     | 10 ++--------
 drivers/soc/mediatek/mtk-svs.c       |  4 +---
 drivers/soc/qcom/qcom_aoss.c         |  4 +---
 drivers/soc/qcom/smp2p.c             |  4 +---
 drivers/soc/qcom/smsm.c              |  4 +---
 drivers/soc/tegra/cbb/tegra194-cbb.c |  9 ++-------
 drivers/soc/ti/wkup_m3_ipc.c         |  4 +---
 drivers/soc/xilinx/zynqmp_power.c    |  5 +----
 9 files changed, 11 insertions(+), 39 deletions(-)

-- 
2.34.1

[PATCH 1/7] soc: fsl: dpio: Remove redundant dev_err()

[Pan Chuang]

Since commit 55b48e23f5c4 ("genirq/devres: Add error handling in
devm_request_*_irq()"), devm_request_irq() automatically logs
detailed error messages on failure. Remove the now-redundant
driver-specific dev_err() calls.

Signed-off-by: Pan Chuang <panchuang@vivo.com>
---
 drivers/soc/fsl/dpio/dpio-driver.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/soc/fsl/dpio/dpio-driver.c b/drivers/soc/fsl/dpio/dpio-driver.c
index 9e3fddd8f5a9..d5198ffc4455 100644
--- a/drivers/soc/fsl/dpio/dpio-driver.c
+++ b/drivers/soc/fsl/dpio/dpio-driver.c
@@ -102,12 +102,8 @@ static int register_dpio_irq_handlers(struct fsl_mc_device *dpio_dev, int cpu)
 				 0,
 				 dev_name(&dpio_dev->dev),
 				 &dpio_dev->dev);
-	if (error < 0) {
-		dev_err(&dpio_dev->dev,
-			"devm_request_irq() failed: %d\n",
-			error);
+	if (error < 0)
 		return error;
-	}
 
 	/* set the affinity hint */
 	if (irq_set_affinity_hint(irq->virq, cpumask_of(cpu)))
-- 
2.34.1

Re: (subset) [PATCH 0/7] soc: Remove redundant error messages on IRQ request failure

[Nishanth Menon]

Hi Pan Chuang,

On Mon, 20 Jul 2026 17:59:06 +0800, Pan Chuang wrote:
> Commit 55b48e23f5c4 ("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.
> 
> [...]

I have applied the following to branch ti-drivers-soc-next on [1].
Thank you!

[6/7] soc: ti: wkup_m3_ipc: Remove redundant dev_err()
      commit: dce7afe3efe485a99200a59dc78738539cbd133f

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
https://ti.com/opensource