From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============1076523015907849130==" MIME-Version: 1.0 From: kernel test robot Subject: [PATCH] fix irqf_oneshot.cocci warnings Date: Tue, 11 May 2021 01:21:59 +0800 Message-ID: <20210510172159.GA31315@99b3544d71b6> In-Reply-To: <202105110130.DI1mYXzR-lkp@intel.com> List-Id: To: kbuild@lists.01.org --===============1076523015907849130== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable CC: kbuild-all(a)lists.01.org CC: linux-arm-kernel(a)lists.infradead.org TO: Linus Walleij CC: Nicolas Pitre CC: Jakub Kicinski CC: Andrew Lunn CC: Allen Pais CC: Colin Ian King CC: Luo Jiaxing CC: Wang Hai CC: Lee Jones CC: netdev(a)vger.kernel.org From: kernel test robot drivers/net/ethernet/smsc/smc91x.c:2001:10-30: ERROR: Threaded IRQ with no = primary handler requested without IRQF_ONESHOT 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 Fixes: 8272c2e41330 ("SMSC slask") Reported-by: kernel test robot Signed-off-by: kernel test robot --- tree: https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-nomadi= k.git nomadik-nhk15-smsc head: 8272c2e41330833ecf16669b97a251b502f21b94 commit: 8272c2e41330833ecf16669b97a251b502f21b94 [3/3] SMSC slask :::::: branch date: 18 hours ago :::::: commit date: 18 hours ago Please take the patch only if it's a positive warning. Thanks! smc91x.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) --- a/drivers/net/ethernet/smsc/smc91x.c +++ b/drivers/net/ethernet/smsc/smc91x.c @@ -1998,7 +1998,9 @@ static int smc_probe(struct net_device * = /* Grab the IRQ */ irq_flags |=3D IRQF_ONESHOT; - retval =3D request_threaded_irq(dev->irq, NULL, smc_interrupt, irq_flags,= dev->name, dev); + retval =3D request_threaded_irq(dev->irq, NULL, smc_interrupt, + irq_flags | IRQF_ONESHOT, dev->name, + dev); if (retval) goto err_out; =20 --===============1076523015907849130==--