Skip to contentSkip to conversation

drivers/net: Remove trailing semicolons from do {...} while 0 macros

PATCH Applied Joe Perches 2 participants 4 messages May 14, 2014 – May 15, 2014

Subsystems (4)
Networking Network drivers TI Ethernet switch driver (cpsw) TI wilink wireless drivers
lore
Share & tools
lore

Joe Perches

author
#0 lore raw

cover drivers/net: Remove trailing semicolons from do {...} while 0 macros

Joe Perches (2):
ti: Remove trailing semicolon from do {...} while (0) macro
wlcore: Remove trailing semicolon from do {...} while (0) macro

 drivers/net/ethernet/ti/cpsw.c           | 4 ++--
 drivers/net/wireless/ti/wlcore/debugfs.h | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)
Signature · 1 line
1.8.1.2.459.gbcd45b4.dirty
author
#1 lore raw

PATCH 1/2 ti: Remove trailing semicolon from do {...} while (0) macro

These should not have trailing semicolons so remove them.

Signed-off-by Joe Perches <[email protected]>
---
 drivers/net/ethernet/ti/cpsw.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/ethernet/ti/cpsw.c +2 -2
index e3d8710..da57d33 100644
@@ -143,13 +143,13 @@ do {								\
 		u32 i;		\
 		for (i = 0; i < priv->num_irqs; i++) \
 			enable_irq(priv->irqs_table[i]); \
-	} while (0);
+	} while (0)
 #define cpsw_disable_irq(priv)	\
 	do {			\
 		u32 i;		\
 		for (i = 0; i < priv->num_irqs; i++) \
 			disable_irq_nosync(priv->irqs_table[i]); \
-	} while (0);
+	} while (0)
 
 #define cpsw_slave_index(priv)				\
 		((priv->data.dual_emac) ? priv->emac_port :	\
Signature · 1 line
1.8.1.2.459.gbcd45b4.dirty
author
#3 lore raw

PATCH 2/2 wlcore: Remove trailing semicolon from do {...} while (0) macro

In reply to Joe Perches

These should not have trailing semicolons so remove them.

Signed-off-by Joe Perches <[email protected]>
---
 drivers/net/wireless/ti/wlcore/debugfs.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/wireless/ti/wlcore/debugfs.h +2 -2
index f7381dd..0f2cfb0 100644
@@ -57,7 +57,7 @@ static const struct file_operations name## _ops = {			\
 					    wl, &name## _ops);		\
 		if (!entry || IS_ERR(entry))				\
 			goto err;					\
-	} while (0);
+	} while (0)
 
 
 #define DEBUGFS_ADD_PREFIX(prefix, name, parent)			\

@@ -66,7 +66,7 @@ static const struct file_operations name## _ops = {			\
 				    wl, &prefix## _## name## _ops);	\
 		if (!entry || IS_ERR(entry))				\
 			goto err;					\
-	} while (0);
+	} while (0)
 
 #define DEBUGFS_FWSTATS_FILE(sub, name, fmt, struct_type)		\
 static ssize_t sub## _ ##name## _read(struct file *file,		\
Signature · 1 line
1.8.1.2.459.gbcd45b4.dirty