[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20210322104343.948660-1-arnd@kernel.org>
Date: Mon, 22 Mar 2021 11:43:31 +0100
From: Arnd Bergmann <arnd@...nel.org>
To: netdev@...r.kernel.org, Karsten Keil <isdn@...ux-pingi.de>
Cc: Arnd Bergmann <arnd@...db.de>, linux-kernel@...r.kernel.org
Subject: [PATCH net-next 1/5] misdn: avoid -Wempty-body warning
From: Arnd Bergmann <arnd@...db.de>
gcc warns about a pointless condition:
drivers/isdn/hardware/mISDN/hfcmulti.c: In function 'hfcmulti_interrupt':
drivers/isdn/hardware/mISDN/hfcmulti.c:2752:17: error: suggest braces around empty body in an 'if' statement [-Werror=empty-body]
2752 | ; /* external IRQ */
Change this as suggested by gcc, which also fits the style of the
other conditions in this function.
Signed-off-by: Arnd Bergmann <arnd@...db.de>
---
drivers/isdn/hardware/mISDN/hfcmulti.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/isdn/hardware/mISDN/hfcmulti.c b/drivers/isdn/hardware/mISDN/hfcmulti.c
index 7013a3f08429..8ab0fde758d2 100644
--- a/drivers/isdn/hardware/mISDN/hfcmulti.c
+++ b/drivers/isdn/hardware/mISDN/hfcmulti.c
@@ -2748,8 +2748,9 @@ hfcmulti_interrupt(int intno, void *dev_id)
if (hc->ctype != HFC_TYPE_E1)
ph_state_irq(hc, r_irq_statech);
}
- if (status & V_EXT_IRQSTA)
- ; /* external IRQ */
+ if (status & V_EXT_IRQSTA) {
+ /* external IRQ */
+ }
if (status & V_LOST_STA) {
/* LOST IRQ */
HFC_outb(hc, R_INC_RES_FIFO, V_RES_LOST); /* clear irq! */
--
2.29.2
Powered by blists - more mailing lists