Re: [PATCH] net: carl9170: remove trailing semicolon in macro definition

[Kalle Valo]

t...@redhat.com wrote:

> The macro use will already have a semicolon.
> 
> Signed-off-by: Tom Rix 
> Signed-off-by: Kalle Valo 

Patch applied to ath-next branch of ath.git, thanks.

e65e8b608f68 carl9170: remove trailing semicolon in macro definition

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20201127175531.2754461-1-t...@redhat.com/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

[PATCH] net: carl9170: remove trailing semicolon in macro definition

[trix]

From: Tom Rix 

The macro use will already have a semicolon.

Signed-off-by: Tom Rix 
---
 drivers/net/wireless/ath/carl9170/debug.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ath/carl9170/debug.c 
b/drivers/net/wireless/ath/carl9170/debug.c
index 19009aafc4e1..bb40889d7c72 100644
--- a/drivers/net/wireless/ath/carl9170/debug.c
+++ b/drivers/net/wireless/ath/carl9170/debug.c
@@ -45,7 +45,7 @@
 #include "cmd.h"
 
 #define ADD(buf, off, max, fmt, args...)   \
-   off += scnprintf(&buf[off], max - off, fmt, ##args);
+   off += scnprintf(&buf[off], max - off, fmt, ##args)
 
 
 struct carl9170_debugfs_fops {
@@ -818,7 +818,7 @@ void carl9170_debugfs_register(struct ar9170 *ar)
 #define DEBUGFS_ADD(name)  \
debugfs_create_file(#name, carl_debugfs_##name ##_ops.attr, \
ar->debug_dir, ar,  \
-   &carl_debugfs_##name ## _ops.fops);
+   &carl_debugfs_##name ## _ops.fops)
 
DEBUGFS_ADD(usb_tx_anch_urbs);
DEBUGFS_ADD(usb_rx_pool_urbs);
-- 
2.18.4