From b5c60b5fdbb3e27fa70f182bba98cdbca02f5c3f Mon Sep 17 00:00:00 2001 From: Arend van Spriel Date: Tue, 3 May 2011 11:35:11 +0200 Subject: [PATCH] staging: brcm80211: remove unnecessary if statements from bss_info_changed 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 Cc: Henry Ptasinski Cc: Roland Vossen Signed-off-by: Arend van Spriel Signed-off-by: Greg Kroah-Hartman --- drivers/staging/brcm80211/brcmsmac/wl_mac80211.c | 11 ----------- 1 file changed, 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) -- 2.7.4