diff mbox

mwifiex: simplify ad hoc join capability info

Message ID 20141111052127.GA18499@us.netrek.org (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

James Cameron Nov. 11, 2014, 5:21 a.m. UTC
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 <quozl@laptop.org>
---
 drivers/net/wireless/mwifiex/join.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Amitkumar Karwar Nov. 11, 2014, 1:05 p.m. UTC | #1
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 <quozl@laptop.org>

Acked-by: Amitkumar Karwar <akarwar@marvell.com>

Thanks,
Amitkumar
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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) {