staging: brcm80211: remove for_each_bss() macro from brcmfmac driver

The macro was only used once in the code and has been removed for
clarity reading the code.

Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
Arend van Spriel 2011-09-16 12:28:05 -07:00 committed by Greg Kroah-Hartman
parent 66b9aa0245
commit 0569c8e2d6
2 changed files with 2 additions and 6 deletions

View File

@ -1993,8 +1993,8 @@ static s32 brcmf_inform_bss(struct brcmf_cfg80211_priv *cfg_priv)
return -EOPNOTSUPP; return -EOPNOTSUPP;
} }
WL_SCAN("scanned AP count (%d)\n", bss_list->count); WL_SCAN("scanned AP count (%d)\n", bss_list->count);
bi = next_bss(bss_list, bi); for (i = 0; i < bss_list->count && i < WL_AP_MAX; i++) {
for_each_bss(bss_list, bi, i) { bi = next_bss(bss_list, bi);
err = brcmf_inform_single_bss(cfg_priv, bi); err = brcmf_inform_single_bss(cfg_priv, bi);
if (unlikely(err)) if (unlikely(err))
break; break;

View File

@ -362,10 +362,6 @@ static inline struct brcmf_bss_info *next_bss(struct brcmf_scan_results *list,
list->bss_info; list->bss_info;
} }
#define for_each_bss(list, bss, __i) \
for (__i = 0; __i < list->count && __i < WL_AP_MAX; __i++, \
bss = next_bss(list, bss))
extern struct brcmf_cfg80211_dev *brcmf_cfg80211_attach(struct net_device *ndev, extern struct brcmf_cfg80211_dev *brcmf_cfg80211_attach(struct net_device *ndev,
struct device *busdev, struct device *busdev,
void *data); void *data);