2011-05-24 11:35:32

by Roland Vossen

Subject: [PATCH] staging: brcm80211: fix for 'multiple definition of wl_msg_level' build err

As reported by Linus.

Signed-off-by: Roland Vossen <[email protected]>
---
drivers/staging/brcm80211/brcmfmac/wl_iw.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/brcm80211/brcmfmac/wl_iw.c b/drivers/staging/brcm80211/brcmfmac/wl_iw.c
index 929ceaf..15e1b05 100644
--- a/drivers/staging/brcm80211/brcmfmac/wl_iw.c
+++ b/drivers/staging/brcm80211/brcmfmac/wl_iw.c
@@ -64,8 +64,6 @@ wl_iw_extra_params_t g_wl_iw_params;
extern bool wl_iw_conn_status_str(u32 event_type, u32 status,
u32 reason, char *stringBuf, uint buflen);

-uint wl_msg_level = WL_ERROR_VAL;
-
#define MAX_WLIW_IOCTL_LEN 1024

#ifdef CONFIG_WIRELESS_EXT
--
1.7.4.1

2011-05-24 12:57:26

by Greg KH

Subject: Re: [PATCH] staging: brcm80211: fix for 'multiple definition of wl_msg_level' build err

On Tue, May 24, 2011 at 01:35:21PM +0200, Roland Vossen wrote:
> As reported by Linus.
>
> Signed-off-by: Roland Vossen <[email protected]>
> ---

When someone else reports it, then you need to say so this way:
Reported-by: Linus Torvalds <[email protected]>

I'll add it to the patch this time, but please be more careful next
time.

thanks,

greg k-h