[wireless-next:master 196/214] drivers/net/wireless/brcm80211/brcmsmac/main.c:308:10: sparse: symbol 'wlc_prio2prec_map' was not declared. Should it be static?

[wireless-next:master 196/214] drivers/net/wireless/brcm80211/brcmsmac/main.c:308:10: sparse: symbol 'wlc_prio2prec_map' was not declared. Should it be static?

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 656 bytes --]

tree:   git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-next.git master
head:   e2ff0498409af6f2023f91aba07b281e5e87ee15
commit: e041f65d5f00011049c7d6af0e31ce69ce9e2655 [196/214] brcmsmac: Remove internal tx queue


sparse warnings:

+ drivers/net/wireless/brcm80211/brcmsmac/main.c:308:10: sparse: symbol 'wlc_prio2prec_map' was not declared. Should it be static?
drivers/net/wireless/brcm80211/brcmsmac/main.c:6500:43:   also defined here

Please consider folding the attached diff :-)

---
0-DAY kernel build testing backend         Open Source Technology Center
Fengguang Wu, Yuanhan Liu                              Intel Corporation

[-- Attachment #2: make-it-static-e041f65.diff --]
[-- Type: text/x-diff, Size: 556 bytes --]

diff --git a/drivers/net/wireless/brcm80211/brcmsmac/main.c b/drivers/net/wireless/brcm80211/brcmsmac/main.c
index 5710dc0..1984b96 100644
--- a/drivers/net/wireless/brcm80211/brcmsmac/main.c
+++ b/drivers/net/wireless/brcm80211/brcmsmac/main.c
@@ -305,7 +305,7 @@ static const u8 wme_ac2fifo[] = {
 };
 
 /* 802.1D Priority to precedence queue mapping */
-const u8 wlc_prio2prec_map[] = {
+static const u8 wlc_prio2prec_map[] = {
 	_BRCMS_PREC_BE,		/* 0 BE - Best-effort */
 	_BRCMS_PREC_BK,		/* 1 BK - Background */
 	_BRCMS_PREC_NONE,		/* 2 None = - */

Re: [wireless-next:master 196/214] drivers/net/wireless/brcm80211/brcmsmac/main.c:308:10: sparse: symbol 'wlc_prio2prec_map' was not declared. Should it be static?

[Seth Forshee]

On Wed, Nov 21, 2012 at 02:59:19PM +0800, kbuild test robot wrote:
> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-next.git master
> head:   e2ff0498409af6f2023f91aba07b281e5e87ee15
> commit: e041f65d5f00011049c7d6af0e31ce69ce9e2655 [196/214] brcmsmac: Remove internal tx queue
> 
> 
> sparse warnings:
> 
> + drivers/net/wireless/brcm80211/brcmsmac/main.c:308:10: sparse: symbol 'wlc_prio2prec_map' was not declared. Should it be static?
> drivers/net/wireless/brcm80211/brcmsmac/main.c:6500:43:   also defined here
> 
> Please consider folding the attached diff :-)

Actually this just isn't used anymore. I'll send a patch to remove it
next week when I'm back from holiday.

Seth