Search Linux Wireless

[PATCH 13/29] staging: brcm80211: revisited END_FOREACH_BSS macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The definition of END_FOREACH_BSS macro resulted in an invalid
checkpatch.pl warning. By adding braces to the macro this warning
goes away. Ideally, the checkpatch.pl script should be adapted
but lacking good perl knowledge to do so.

Cc: devel@xxxxxxxxxxxxxxxxxxxxxx
Cc: linux-wireless@xxxxxxxxxxxxxxx
Reported-by: Dan Carpenter <error27@xxxxxxxxx>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@xxxxxxxxxxxx>
Reviewed-by: Roland Vossen <rvossen@xxxxxxxxxxxx>
Signed-off-by: Arend van Spriel <arend@xxxxxxxxxxxx>
---
 drivers/staging/brcm80211/brcmsmac/main.c |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/staging/brcm80211/brcmsmac/main.c b/drivers/staging/brcm80211/brcmsmac/main.c
index e67bf64..f5f914f 100644
--- a/drivers/staging/brcm80211/brcmsmac/main.c
+++ b/drivers/staging/brcm80211/brcmsmac/main.c
@@ -270,7 +270,7 @@
 		if (!cfg) \
 			continue;
 /* close marker for iterator code block */
-#define END_FOREACH_BSS		}
+#define END_FOREACH_BSS()	}
 
 /* Shared memory location index for various AC params */
 #define wme_shmemacindex(ac)	wme_ac2fifo[ac]
@@ -3345,7 +3345,7 @@ void brcms_c_init(struct brcms_c_info *wlc)
 	FOREACH_BSS(wlc, i, bsscfg)
 		brcms_c_set_mac(bsscfg);
 		brcms_c_set_bssid(bsscfg);
-	END_FOREACH_BSS
+	END_FOREACH_BSS()
 
 	/* Update tsf_cfprep if associated and up */
 	if (wlc->pub->associated) {
@@ -3367,7 +3367,7 @@ void brcms_c_init(struct brcms_c_info *wlc)
 
 				break;
 			}
-		END_FOREACH_BSS
+		END_FOREACH_BSS()
 	}
 
 	brcms_c_bandinit_ordered(wlc, chanspec);
@@ -3577,7 +3577,7 @@ void brcms_c_switch_shortslot(struct brcms_c_info *wlc, bool shortslot)
 		if (wlc->shortslot)
 			cfg->current_bss->capability |=
 					WLAN_CAPABILITY_SHORT_SLOT_TIME;
-	END_FOREACH_BSS /* this comment fixes a checkpatch warning */
+	END_FOREACH_BSS()
 
 	brcms_b_set_shortslot(wlc->hw, shortslot);
 }
@@ -3626,7 +3626,7 @@ void brcms_c_set_home_chanspec(struct brcms_c_info *wlc, u16 chanspec)
 				continue;
 
 			cfg->current_bss->chanspec = chanspec;
-		END_FOREACH_BSS
+		END_FOREACH_BSS()
 
 
 	}
@@ -5384,7 +5384,7 @@ static void brcms_c_watchdog(void *arg)
 			cfg->tk_cm_dt--;
 		if (cfg->tk_cm_bt)
 			cfg->tk_cm_bt--;
-	END_FOREACH_BSS
+	END_FOREACH_BSS()
 
 	/* Call any registered watchdog handlers */
 	for (i = 0; i < BRCMS_MAXMODULES; i++) {
@@ -5557,7 +5557,7 @@ int brcms_c_up(struct brcms_c_info *wlc)
 						  ": rfdisable -> "
 						  "bsscfg_disable()\n",
 						   wlc->pub->unit, idx);
-				END_FOREACH_BSS
+				END_FOREACH_BSS()
 			}
 		}
 	}
@@ -9075,7 +9075,7 @@ void brcms_c_update_beacon(struct brcms_c_info *wlc)
 	FOREACH_BSS(wlc, idx, bsscfg)
 		if (bsscfg->up && (BSSCFG_AP(bsscfg) || !bsscfg->BSS))
 			brcms_c_bss_update_beacon(wlc, bsscfg);
-	END_FOREACH_BSS
+	END_FOREACH_BSS()
 }
 
 /* Write ssid into shared memory */
@@ -9104,7 +9104,7 @@ void brcms_c_update_probe_resp(struct brcms_c_info *wlc, bool suspend)
 	FOREACH_BSS(wlc, idx, bsscfg)
 		if (bsscfg->up && (BSSCFG_AP(bsscfg) || !bsscfg->BSS))
 			brcms_c_bss_update_probe_resp(wlc, bsscfg, suspend);
-	END_FOREACH_BSS
+	END_FOREACH_BSS()
 }
 
 void
@@ -9191,7 +9191,7 @@ void brcms_c_reprate_init(struct brcms_c_info *wlc)
 
 	FOREACH_BSS(wlc, i, bsscfg)
 		brcms_c_bsscfg_reprate_init(bsscfg);
-	END_FOREACH_BSS
+	END_FOREACH_BSS()
 }
 
 /* per bsscfg init tx reported rate mechanism */
-- 
1.7.4.1


--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux