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

2014-05-14 19:15 UTC #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

Joe Perches

author

2014-05-14 19:15 UTC #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

David Miller

maintainer Applied

2014-05-15 20:41 UTC #2 lore raw

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

From: Joe Perches <[email protected]>
Date: Wed, 14 May 2014 12:15:13 -0700

Quoted 3 lines

These should not have trailing semicolons so remove them.
Signed-off-by Joe Perches <[email protected]>

Applied, thanks.

Joe Perches

author

2014-05-14 19:15 UTC #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