diff options
author | Joe Perches <joe@perches.com> | 2012-04-03 17:46:49 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2012-04-11 16:23:56 -0400 |
commit | da951c2417ec1020d0d00813da36f38e395994e9 (patch) | |
tree | 65404e5b7e22c2fbaa8ac0609213404c47b60cf5 /drivers/net/wireless/ath/carl9170 | |
parent | 150721894e3613a228e212615909ade14964235c (diff) |
wireless: Remove unnecessary ; from while (0) macros
Semicolons are not necessary after macros that end in while (0).
Remove them.
Simplify the macros with tests of
do { if (foo>size) memset1; else memset2;} while (0);
to a single line memset(,,min_t(size_t, foo, size))
Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Arend van Spriel <arend@broadcom.com>
Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
Acked-by: Bing Zhao <bzhao@marvell.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/ath/carl9170')
-rw-r--r-- | drivers/net/wireless/ath/carl9170/cmd.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/wireless/ath/carl9170/cmd.h b/drivers/net/wireless/ath/carl9170/cmd.h index 885c42778b8b..65919c902f55 100644 --- a/drivers/net/wireless/ath/carl9170/cmd.h +++ b/drivers/net/wireless/ath/carl9170/cmd.h | |||
@@ -114,7 +114,7 @@ __regwrite_out : \ | |||
114 | 114 | ||
115 | #define carl9170_regwrite_result() \ | 115 | #define carl9170_regwrite_result() \ |
116 | __err; \ | 116 | __err; \ |
117 | } while (0); | 117 | } while (0) |
118 | 118 | ||
119 | 119 | ||
120 | #define carl9170_async_regwrite_get_buf() \ | 120 | #define carl9170_async_regwrite_get_buf() \ |
@@ -126,7 +126,7 @@ do { \ | |||
126 | __err = -ENOMEM; \ | 126 | __err = -ENOMEM; \ |
127 | goto __async_regwrite_out; \ | 127 | goto __async_regwrite_out; \ |
128 | } \ | 128 | } \ |
129 | } while (0); | 129 | } while (0) |
130 | 130 | ||
131 | #define carl9170_async_regwrite_begin(carl) \ | 131 | #define carl9170_async_regwrite_begin(carl) \ |
132 | do { \ | 132 | do { \ |
@@ -169,6 +169,6 @@ __async_regwrite_out: \ | |||
169 | 169 | ||
170 | #define carl9170_async_regwrite_result() \ | 170 | #define carl9170_async_regwrite_result() \ |
171 | __err; \ | 171 | __err; \ |
172 | } while (0); | 172 | } while (0) |
173 | 173 | ||
174 | #endif /* __CMD_H */ | 174 | #endif /* __CMD_H */ |