Re: iwlwifi A-MSDU tx

2015-12-04 Thread Emmanuel Grumbach
Adding the right mailing list this time. On Fri, Dec 4, 2015 at 10:18 AM, Emmanuel Grumbach wrote: > On Fri, Dec 4, 2015 at 9:35 AM, Emmanuel Grumbach wrote: >> Hi, >> >> On Fri, Dec 4, 2015 at 12:05 AM, Stefan Sperling wrote: >>> Hi

Re: [patch] iwlegacy: cleanup end of il_send_add_sta()

2015-12-04 Thread Stanislaw Gruszka
On Fri, Dec 04, 2015 at 04:16:45PM +0300, Dan Carpenter wrote: > This code causes a static checker warning because we check for > "if (ret == 0)" but we have already had verified that was true. Clean > it up a little. > > Signed-off-by: Dan Carpenter Acked-by:

[patch] hostap: fix an error code in prism2_config()

2015-12-04 Thread Dan Carpenter
The current code returns success if prism2_init_local_data() fails, but we want to return an error code. Also we can remove the bogus ret initializer because it is wrong and never used. Signed-off-by: Dan Carpenter diff --git

[patch] mwifiex: remove an unneeded condition

2015-12-04 Thread Dan Carpenter
We already know that "wep_key->key_length" is set so there is no need to check again. Also the last curly brace was indented too far. Signed-off-by: Dan Carpenter diff --git a/drivers/net/wireless/marvell/mwifiex/sta_ioctl.c

[patch] prism54: off by one BUG_ON() test

2015-12-04 Thread Dan Carpenter
This code was supposed to trigger a BUG() if we truncate the output but it's off by one so it allows one character to be truncated. Really drivers shouldn't call BUG_ON() and especially for something minor like this so I've changed it to a WARN_ON(). Signed-off-by: Dan Carpenter

[PATCH 5/6] mwifiex: add iw vendor command support

2015-12-04 Thread Amitkumar Karwar
From: chunfan chen The patch allows user to 1. Enable turbo mode in firmware using following command. iw dev mlan0 vendor send 0x005043 0x00 0x01 2. Download configuration data to FW using following command iw dev mlan0 vendor send 0x005043 0x01 filename Signed-off-by:

[PATCH 6/6] mwifiex: correction in region code to country mapping

2015-12-04 Thread Amitkumar Karwar
EU is not a valid country in db.txt file. Hence regulatory_hint returns failure if EEPROM provides region code as 0x30. Let's use FR for 0x30. Signed-off-by: Amitkumar Karwar Signed-off-by: Cathy Luo --- drivers/net/wireless/marvell/mwifiex/cfp.c | 2 +-

[PATCH 4/6] mwifiex: don't follow AP if country code received from EEPROM

2015-12-04 Thread Amitkumar Karwar
If device has already received country information from EEPROM, we won't parse AP's country IE and download it to firmware. We will also set regulatory flags to disable beacon hints and ignore country IE. Signed-off-by: Amitkumar Karwar Signed-off-by: Cathy Luo

[PATCH 1/6] mwifiex: parse adhoc start/join result

2015-12-04 Thread Amitkumar Karwar
Even if ADHOC start or join attempt is failed, these commands are returned with success status by firmware. Actual connection result is provided inside command response. This patch parses the adhoc connection result and resets connection state variables if connection is not successful.

[PATCH 3/6] mwifiex: set regulatory info from EEPROM

2015-12-04 Thread Amitkumar Karwar
Driver gets country information from EEPROM during initialization. We will call regulatory_hint to update current regulatory domain. As by default world regulatory domain is selected by cfg80211, country '00' from EEPROM is ignored. Signed-off-by: Amitkumar Karwar

[patch] iwlegacy: cleanup end of il_send_add_sta()

2015-12-04 Thread Dan Carpenter
This code causes a static checker warning because we check for "if (ret == 0)" but we have already had verified that was true. Clean it up a little. Signed-off-by: Dan Carpenter --- I have preserved the original behavior but it's possible that the original code was