LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Pan Chuang <panchuang@vivo.com>
To: "Roy Pledge" <Roy.Pledge@nxp.com>,
	"Christophe Leroy (CS GROUP)" <chleroy@kernel.org>,
	"Linus Walleij" <linusw@kernel.org>,
	"Imre Kaloz" <kaloz@openwrt.org>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Bjorn Andersson" <andersson@kernel.org>,
	"Konrad Dybcio" <konradybcio@kernel.org>,
	"Thierry Reding" <thierry.reding@kernel.org>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Nishanth Menon" <nm@ti.com>,
	"Santosh Shilimkar" <ssantosh@kernel.org>,
	"Michal Simek" <michal.simek@amd.com>,
	"Shashank Balaji" <shashank.mahadasyam@sony.com>,
	"Gary Guo" <gary@garyguo.net>, "Sumit Gupta" <sumitg@nvidia.com>,
	"Rahul Bukte" <rahul.bukte@sony.com>,
	"Bruno Sobreira França" <brunofrancadevsec@gmail.com>,
	"Pan Chuang" <panchuang@vivo.com>,
	"Prasanna Kumar T S M" <ptsm@linux.microsoft.com>,
	"Marco Crivellari" <marco.crivellari@suse.com>,
	"Jay Buddhabhatti" <jay.buddhabhatti@amd.com>,
	linux-kernel@vger.kernel.org (open list:DPAA2 DATAPATH I/O
	(DPIO) DRIVER),
	linuxppc-dev@lists.ozlabs.org (open list:FREESCALE SOC DRIVERS),
	linux-arm-kernel@lists.infradead.org (moderated list:FREESCALE
	SOC DRIVERS),
	linux-mediatek@lists.infradead.org (moderated list:ARM/Mediatek
	SoC support),
	linux-arm-msm@vger.kernel.org (open list:ARM/QUALCOMM MAILING
	LIST),
	linux-tegra@vger.kernel.org (open list:TEGRA ARCHITECTURE
	SUPPORT)
Subject: [PATCH 0/7] soc: Remove redundant error messages on IRQ request failure
Date: Mon, 20 Jul 2026 17:59:06 +0800	[thread overview]
Message-ID: <20260720095920.542801-1-panchuang@vivo.com> (raw)

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



             reply	other threads:[~2026-07-20 10:00 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-20  9:59 Pan Chuang [this message]
2026-07-20  9:59 ` [PATCH 1/7] soc: fsl: dpio: Remove redundant dev_err() Pan Chuang
2026-07-24 16:38 ` (subset) [PATCH 0/7] soc: Remove redundant error messages on IRQ request failure Nishanth Menon

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=20260720095920.542801-1-panchuang@vivo.com \
    --to=panchuang@vivo.com \
    --cc=Roy.Pledge@nxp.com \
    --cc=andersson@kernel.org \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=brunofrancadevsec@gmail.com \
    --cc=chleroy@kernel.org \
    --cc=gary@garyguo.net \
    --cc=jay.buddhabhatti@amd.com \
    --cc=jonathanh@nvidia.com \
    --cc=kaloz@openwrt.org \
    --cc=konradybcio@kernel.org \
    --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-tegra@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=marco.crivellari@suse.com \
    --cc=matthias.bgg@gmail.com \
    --cc=michal.simek@amd.com \
    --cc=nm@ti.com \
    --cc=ptsm@linux.microsoft.com \
    --cc=rahul.bukte@sony.com \
    --cc=shashank.mahadasyam@sony.com \
    --cc=ssantosh@kernel.org \
    --cc=sumitg@nvidia.com \
    --cc=thierry.reding@kernel.org \
    /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