staging: brcm80211: remove unnecessary if statements from bss_info_changed
authorArend van Spriel <arend@broadcom.com>
Tue, 3 May 2011 09:35:11 +0000 (11:35 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 3 May 2011 19:42:31 +0000 (12:42 -0700)
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@linuxdriverproject.org
Cc: linux-wireless@vger.kernel.org
Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/brcm80211/brcmsmac/wl_mac80211.c

index 7e9ed523b48b383beaa1cbe94fe5129904917ffa..8144587c7664ff5b18402b8ae54122b6db08d78a 100644 (file)
@@ -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)