brcmfmac: move ALLFFMAC variable in flowring module
authorArend Van Spriel <arend.vanspriel@broadcom.com>
Wed, 16 May 2018 12:11:58 +0000 (14:11 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 23 May 2018 15:51:47 +0000 (18:51 +0300)
The only user of ALLFFMAC is the flowring module so no need to
expose it in a header file.

Reviewed-by: Hante Meuleman <hante.meuleman@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieter-paul.giesberts@broadcom.com>
Reviewed-by: Franky Lin <franky.lin@broadcom.com>
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.h
drivers/net/wireless/broadcom/brcm80211/brcmfmac/flowring.c

index 105b8774fca9cb97bcdd49a514672c9e16940d93..cd3651069d0c4e5bbe1810b0a91a9e433f27217d 100644 (file)
@@ -36,8 +36,6 @@ MODULE_AUTHOR("Broadcom Corporation");
 MODULE_DESCRIPTION("Broadcom 802.11 wireless LAN fullmac driver.");
 MODULE_LICENSE("Dual BSD/GPL");
 
-const u8 ALLFFMAC[ETH_ALEN] = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff };
-
 #define BRCMF_DEFAULT_SCAN_CHANNEL_TIME        40
 #define BRCMF_DEFAULT_SCAN_UNASSOC_TIME        40
 
index ef914619e8e152bebe1403a2d17ac94092d72bbb..a34642cb4d2fb2fdca00c7311cd85c5e5efe7b58 100644 (file)
@@ -19,8 +19,6 @@
 #include <linux/platform_data/brcmfmac.h>
 #include "fwil_types.h"
 
-extern const u8 ALLFFMAC[ETH_ALEN];
-
 #define BRCMF_FW_ALTPATH_LEN                   256
 
 /* Definitions for the module global and device specific settings are defined
index d0b738da245822cd3e50d2b1a65764ce9cf12561..d0d8b32af7d05081f5a2ffa6fbf7f04032e7077a 100644 (file)
@@ -46,6 +46,8 @@ static const u8 brcmf_flowring_prio2fifo[] = {
        3
 };
 
+static const u8 ALLFFMAC[ETH_ALEN] = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff };
+
 
 static bool
 brcmf_flowring_is_tdls_mac(struct brcmf_flowring *flow, u8 mac[ETH_ALEN])