netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] drivers/net: Remove trailing semicolons from do {...} while 0 macros
@ 2014-05-14 19:15 Joe Perches
  2014-05-14 19:15 ` [PATCH 1/2] ti: Remove trailing semicolon from do {...} while (0) macro Joe Perches
  2014-05-14 19:15 ` [PATCH 2/2] wlcore: " Joe Perches
  0 siblings, 2 replies; 4+ messages in thread
From: Joe Perches @ 2014-05-14 19:15 UTC (permalink / raw)
  To: netdev; +Cc: linux-kernel, linux-wireless

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(-)

-- 
1.8.1.2.459.gbcd45b4.dirty

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] ti: Remove trailing semicolon from do {...} while (0) macro
  2014-05-14 19:15 [PATCH 0/2] drivers/net: Remove trailing semicolons from do {...} while 0 macros Joe Perches
@ 2014-05-14 19:15 ` Joe Perches
  2014-05-15 20:41   ` David Miller
  2014-05-14 19:15 ` [PATCH 2/2] wlcore: " Joe Perches
  1 sibling, 1 reply; 4+ messages in thread
From: Joe Perches @ 2014-05-14 19:15 UTC (permalink / raw)
  To: netdev; +Cc: linux-kernel

These should not have trailing semicolons so remove them.

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/net/ethernet/ti/cpsw.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c
index e3d8710..da57d33 100644
--- a/drivers/net/ethernet/ti/cpsw.c
+++ b/drivers/net/ethernet/ti/cpsw.c
@@ -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 :	\
-- 
1.8.1.2.459.gbcd45b4.dirty

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] wlcore: Remove trailing semicolon from do {...} while (0) macro
  2014-05-14 19:15 [PATCH 0/2] drivers/net: Remove trailing semicolons from do {...} while 0 macros Joe Perches
  2014-05-14 19:15 ` [PATCH 1/2] ti: Remove trailing semicolon from do {...} while (0) macro Joe Perches
@ 2014-05-14 19:15 ` Joe Perches
  1 sibling, 0 replies; 4+ messages in thread
From: Joe Perches @ 2014-05-14 19:15 UTC (permalink / raw)
  To: netdev; +Cc: John W. Linville, linux-wireless, linux-kernel

These should not have trailing semicolons so remove them.

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/net/wireless/ti/wlcore/debugfs.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ti/wlcore/debugfs.h b/drivers/net/wireless/ti/wlcore/debugfs.h
index f7381dd..0f2cfb0 100644
--- a/drivers/net/wireless/ti/wlcore/debugfs.h
+++ b/drivers/net/wireless/ti/wlcore/debugfs.h
@@ -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,		\
-- 
1.8.1.2.459.gbcd45b4.dirty

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] ti: Remove trailing semicolon from do {...} while (0) macro
  2014-05-14 19:15 ` [PATCH 1/2] ti: Remove trailing semicolon from do {...} while (0) macro Joe Perches
@ 2014-05-15 20:41   ` David Miller
  0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2014-05-15 20:41 UTC (permalink / raw)
  To: joe; +Cc: netdev, linux-kernel

From: Joe Perches <joe@perches.com>
Date: Wed, 14 May 2014 12:15:13 -0700

> These should not have trailing semicolons so remove them.
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Applied, thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-05-15 20:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-14 19:15 [PATCH 0/2] drivers/net: Remove trailing semicolons from do {...} while 0 macros Joe Perches
2014-05-14 19:15 ` [PATCH 1/2] ti: Remove trailing semicolon from do {...} while (0) macro Joe Perches
2014-05-15 20:41   ` David Miller
2014-05-14 19:15 ` [PATCH 2/2] wlcore: " Joe Perches

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).