From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============6182262012330042486==" MIME-Version: 1.0 From: kernel test robot Subject: [PATCH] coccinelle: irqf_oneshot: fix irqf_oneshot.cocci warnings Date: Tue, 15 Mar 2022 12:51:48 +0800 Message-ID: <20220315045148.GA24790@38be7d09bd03> In-Reply-To: <202203151217.y9tLkLxv-lkp@intel.com> List-Id: To: kbuild@lists.01.org --===============6182262012330042486== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable CC: kbuild-all(a)lists.01.org BCC: lkp(a)intel.com CC: linux-kernel(a)vger.kernel.org TO: Krzysztof Kozlowski CC: Julia Lawall CC: Russell King CC: David Airlie CC: Daniel Vetter CC: dri-devel(a)lists.freedesktop.org CC: linux-kernel(a)vger.kernel.org From: kernel test robot drivers/gpu/drm/i2c/tda998x_drv.c:1916:8-28: WARNING: Threaded IRQ with no = primary handler requested without IRQF_ONESHOT (unless it is nested IRQ) Since commit 1c6c69525b40 ("genirq: Reject bogus threaded irq requests") threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. Generated by: scripts/coccinelle/misc/irqf_oneshot.cocci CC: Krzysztof Kozlowski Reported-by: kernel test robot Signed-off-by: kernel test robot --- tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git = master head: 09688c0166e76ce2fb85e86b9d99be8b0084cdf9 commit: 5d2db9bb5f8a850d037983f0df72ad59cefa9e3d coccinelle: irqf_oneshot: = reduce the severity due to false positives :::::: branch date: 32 hours ago :::::: commit date: 11 months ago Please take the patch only if it's a positive warning. Thanks! drivers/gpu/drm/i2c/tda998x_drv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- a/drivers/gpu/drm/i2c/tda998x_drv.c +++ b/drivers/gpu/drm/i2c/tda998x_drv.c @@ -1914,7 +1914,8 @@ static int tda998x_create(struct device = irq_flags |=3D IRQF_SHARED | IRQF_ONESHOT; ret =3D request_threaded_irq(client->irq, NULL, - tda998x_irq_thread, irq_flags, + tda998x_irq_thread, + irq_flags | IRQF_ONESHOT, "tda998x", priv); if (ret) { dev_err(dev, "failed to request IRQ#%u: %d\n", --===============6182262012330042486==--