[PATCH 11/61] staging: brcm80211: remove unnecessary if statements from bss_info_changed
Arend van Spriel
arend at broadcom.com
Tue May 3 09:35:11 UTC 2011
The mac80211 api callback bss_info_changed contained two placeholders
that were expected to need to be implemented. However, reading the mac80211
notes (http://wireless.kernel.org/en/developers/Documentation/mac80211/API)
shows that it is not needed as the driver honours the flags provided with
the transmit sk_buff.
Cc: devel at linuxdriverproject.org
Cc: linux-wireless at vger.kernel.org
Cc: Brett Rudley <brudley at broadcom.com>
Cc: Henry Ptasinski <henryp at broadcom.com>
Cc: Roland Vossen <rvossen at broadcom.com>
Signed-off-by: Arend van Spriel <arend at broadcom.com>
---
drivers/staging/brcm80211/brcmsmac/wl_mac80211.c | 11 -----------
1 files changed, 0 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/brcm80211/brcmsmac/wl_mac80211.c b/drivers/staging/brcm80211/brcmsmac/wl_mac80211.c
index 7e9ed52..8144587 100644
--- a/drivers/staging/brcm80211/brcmsmac/wl_mac80211.c
+++ b/drivers/staging/brcm80211/brcmsmac/wl_mac80211.c
@@ -346,17 +346,6 @@ wl_ops_bss_info_changed(struct ieee80211_hw *hw,
__func__, info->assoc ? "" : "dis");
wlc_associate_upd(wl->wlc, info->assoc);
}
- if (changed & BSS_CHANGED_ERP_CTS_PROT) {
- /* CTS protection changed */
- wiphy_err(wiphy, "%s: use_cts_prot: %s (implement)\n", __func__,
- info->use_cts_prot ? "true" : "false");
- }
- if (changed & BSS_CHANGED_ERP_PREAMBLE) {
- /* preamble changed */
- wiphy_err(wiphy, "%s: short preamble: %s (implement)\n",
- __func__, info->use_short_preamble ? "true" :
- "false");
- }
if (changed & BSS_CHANGED_ERP_SLOT) {
/* slot timing changed */
if (info->use_short_slot)
--
1.7.4.1
More information about the devel
mailing list