2014-11-11 05:21:46

by James Cameron

Subject: [PATCH] mwifiex: simplify ad hoc join capability info

While preparing an ad-hoc start command, the capability info bitmap is
needlessly set from the command, and then the ESS bit cleared.

Change to set the bitmap directly without reference to the command.

Signed-off-by: James Cameron <[email protected]>
---
drivers/net/wireless/mwifiex/join.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/net/wireless/mwifiex/join.c b/drivers/net/wireless/mwifiex/join.c
index 8d6c259..411a6c2 100644
--- a/drivers/net/wireless/mwifiex/join.c
+++ b/drivers/net/wireless/mwifiex/join.c
@@ -880,9 +880,7 @@ mwifiex_cmd_802_11_ad_hoc_start(struct mwifiex_private *priv,

/* Set Capability info */
bss_desc->cap_info_bitmap |= WLAN_CAPABILITY_IBSS;
- tmp_cap = le16_to_cpu(adhoc_start->cap_info_bitmap);
- tmp_cap &= ~WLAN_CAPABILITY_ESS;
- tmp_cap |= WLAN_CAPABILITY_IBSS;
+ tmp_cap = WLAN_CAPABILITY_IBSS;

/* Set up privacy in bss_desc */
if (priv->sec_info.encryption_mode) {
--
1.9.1


--
James Cameron
http://quozl.linux.org.au/

2014-11-11 13:05:48

by Amitkumar Karwar

Subject: RE: [PATCH] mwifiex: simplify ad hoc join capability info

Hi James,

>While preparing an ad-hoc start command, the capability info bitmap is
>needlessly set from the command, and then the ESS bit cleared.
>
>Change to set the bitmap directly without reference to the command.
>
>Signed-off-by: James Cameron <[email protected]>

Acked-by: Amitkumar Karwar <[email protected]>

Thanks,
Amitkumar