Re: [PATCH 4.3] ath10k: fix DMA related firmware crashes on multiple devices

2015-09-15 Thread Chris
Hi, Felix Fietkau writes: > > Some platforms really don't like DMA bursts of 256 bytes, and this > causes the firmware to crash when sending beacons. > Also, changing this based on the firmware version does not seem to make > much sense, so use 128 bytes for all version i can confirm that now

Re: [PATCH 1/4] staging: wilc1000: remove unused variable g_num_total_switches

2015-09-15 Thread Sudip Mukherjee
On Tue, Sep 15, 2015 at 02:33:54PM +0900, Tony Cho wrote: > From: Glen Lee > > g_num_total_switches is never used so just delete it. > > Signed-off-by: Glen Lee > Signed-off-by: Tony Cho > --- you only deleted from the .h file but it still remains in coreconfigurator.c regards sudip -- To uns

Re: [PATCH 2/4] staging: wilc1000: delete function declaration SendRawPacket

2015-09-15 Thread Sudip Mukherjee
On Tue, Sep 15, 2015 at 02:33:55PM +0900, Tony Cho wrote: > From: Glen Lee > > SendRawpacket is declared but not implemented. It is not used also. > So just delete it. > > Signed-off-by: Glen Lee > Signed-off-by: Tony Cho > --- Still remains in coreconfigurator.c regards sudip -- To unsubscri

Re: [PATCH 3/3] ath10k: implement mesh support

2015-09-15 Thread Peter Oh
On 09/10/2015 12:22 AM, Johannes Berg wrote: On Wed, 2015-09-09 at 12:49 -0400, Bob Copeland wrote: On Wed, Sep 09, 2015 at 12:10:38PM +0300, Kalle Valo wrote: I did a quick smoke test and saw the splat below. It's this warning from cfg80211: /*

Re: [PATCH 1/4] staging: wilc1000: remove unused variable g_num_total_switches

2015-09-15 Thread Tony Cho
On 2015년 09월 15일 16:13, Sudip Mukherjee wrote: On Tue, Sep 15, 2015 at 02:33:54PM +0900, Tony Cho wrote: From: Glen Lee g_num_total_switches is never used so just delete it. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- you only deleted from the .h file but it still remains in core

[PATCH 2/2] Staging: wilc1000: fix NULL comparison style

2015-09-15 Thread Ronit Halder
Fixed NULL comparison style as suggested by checkpatch.pl with --strict option. Signed-off-by: Ronit halder --- drivers/staging/wilc1000/coreconfigurator.c | 2 +- drivers/staging/wilc1000/linux_wlan.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/stagi

[PATCH v2 1/2] Staging: wilc1000: Use NULL instead of zero

2015-09-15 Thread Ronit Halder
This patch fixes the warning generated by sparse "Using plain integer as NULL pointer" by using NULL instead of zero. Signed-off-by: Ronit halder --- v2: added a new patch in this patch series to fix the NULL comparison style drivers/staging/wilc1000/coreconfigurator.c | 14 +++--- dri

Re: [RFC v2] [resend] ath9k: allow up to 8 access point vifs on DFS channels

2015-09-15 Thread Matthias May
On 09/06/15 18:26, Jean-Pierre TOSONI wrote: Allow the combination of 8 (AP + mesh) interfaces on radar channels Tested with hostapd, 2 and 8 BSSIDs, 1 or 2 DFS channels, BW 20 MHz TODO: test AP+mesh combination TODO: test 40 MHz BW --- This patch was done with quilt on Openwrt against compat-wi

[PATCH v2] drivers: staging: wilc1000: Add check for SPI availability

2015-09-15 Thread Chandra S Gorentla
NULL pointer deference is observed in the wilc1000.ko module with bus type SPI and when SPI is not ready. Signed-off-by: Chandra S Gorentla --- v2 - Subject corrected 'stating:' -> 'staging:' Following are the steps to reproduce. $ sudo insmod drivers/staging/wilc1000/wilc1000.ko $ sudo ifconf

Re: [PATCH 2/2] staging: wilc1000: remove RX_BH_TYPE

2015-09-15 Thread Greg KH
On Tue, Sep 15, 2015 at 10:17:41AM +0900, Tony Cho wrote: > Hi Greg, > > I am sorry but please ignore this patch in this time. It's already in my tree, right? I can revert it if you want me to, but I can't ignore it anymore, sorry. greg k-h -- To unsubscribe from this list: send the line "unsub

Re: [PATCH 2/7] staging: wilc1000: remove definition WILC_NULLCHECK

2015-09-15 Thread Greg KH
On Tue, Sep 15, 2015 at 11:53:36AM +0900, Tony Cho wrote: > From: Leo Kim > > This patch removes the definition WILC_NULLCHECK which is not used anymore. No, it is used, you just remove all usages of it in this patch as well :) Also, your fixes are not correct, you are not keeping the same code

Re: [PATCH 1/4] staging: wilc1000: remove unused variable g_num_total_switches

2015-09-15 Thread Greg KH
On Tue, Sep 15, 2015 at 05:26:35PM +0900, Tony Cho wrote: > > > On 2015년 09월 15일 16:13, Sudip Mukherjee wrote: > >On Tue, Sep 15, 2015 at 02:33:54PM +0900, Tony Cho wrote: > >>From: Glen Lee > >> > >>g_num_total_switches is never used so just delete it. > >> > >>Signed-off-by: Glen Lee > >>Sign

[PATCH] staging: wilc1000: Removed curly braces

2015-09-15 Thread Aparna Karuthodi
Removed the curly braces of a single statement if block to remove a coding style warning detected by checkpatch. The warning is given below: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Aparna Karuthodi --- drivers/staging/wilc1000/coreconfigurator.c |3 +--

Re: [PATCH v2 1/2] Staging: wilc1000: Use NULL instead of zero

2015-09-15 Thread Greg KH
On Tue, Sep 15, 2015 at 03:04:58PM +0530, Ronit Halder wrote: > This patch fixes the warning generated by sparse > "Using plain integer as NULL pointer" by using NULL > instead of zero. > > Signed-off-by: Ronit halder > --- > > v2: added a new patch in this patch series to fix the NULL compariso

Re: [PATCH 1/4] staging: wilc1000: remove unused variable g_num_total_switches

2015-09-15 Thread Sudip Mukherjee
On Tue, Sep 15, 2015 at 06:46:25AM -0700, Greg KH wrote: > On Tue, Sep 15, 2015 at 05:26:35PM +0900, Tony Cho wrote: > > > > > > On 2015년 09월 15일 16:13, Sudip Mukherjee wrote: > > >On Tue, Sep 15, 2015 at 02:33:54PM +0900, Tony Cho wrote: > > >>From: Glen Lee > > >> > > >>g_num_total_switches is

Default rates for broadcast, multicast, and management frames?

2015-09-15 Thread Ben Greear
ath10k firmware allows one to set fixed rates for broadcast, multicast, and management/beacon frames. I think upstream firmware may not have full support, but the API is there in 10.1, at least, and CT firmware has full support. I think the defaults are 6Mbps, but not certain of that. CT kern

[PATCH] ath10k: Fix CCK rate definitions.

2015-09-15 Thread greearb
From: Ben Greear The old values did NOT match what is found in 10.1.467 firmware. I figure it is likely it is wrong for the others as well. Signed-off-by: Ben Greear --- drivers/net/wireless/ath/ath10k/hw.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/dri

CT firmware beta-15 can now configure arbitrary rate-sets.

2015-09-15 Thread Ben Greear
Here is the changelog note. This requires some corresponding driver patchs to enable configuring the rateset (and fetching the tx-ratectrl memory from the host) I tested this on my 4.0.4+ kernel. Should be useful for anyone trying make a station transmit in pure /b mode. And, likely could help

Re: [PATCH 2/2] staging: wilc1000: remove RX_BH_TYPE

2015-09-15 Thread Tony Cho
Hi Greg, If it's already in your tree and can be applied alone, no problem to me. I will later send remaining series of it. Thanks, Tony. On 2015년 09월 15일 22:31, Greg KH wrote: On Tue, Sep 15, 2015 at 10:17:41AM +0900, Tony Cho wrote: Hi Greg, I am sorry but please ignore this patch in th

Re: [PATCH 1/4] staging: wilc1000: remove unused variable g_num_total_switches

2015-09-15 Thread Tony Cho
On 2015년 09월 15일 22:46, Greg KH wrote: On Tue, Sep 15, 2015 at 05:26:35PM +0900, Tony Cho wrote: On 2015년 09월 15일 16:13, Sudip Mukherjee wrote: On Tue, Sep 15, 2015 at 02:33:54PM +0900, Tony Cho wrote: From: Glen Lee g_num_total_switches is never used so just delete it. Signed-off-by: Gl

[PATCH v2 00/10] resend all of wilc1000 patches

2015-09-15 Thread Tony Cho
This series of patches are being resent becasue it seems that we are out of sync with Greg. In this series, I squashed two commits seperately issued for removing variable, g_num_total_switches which is not used anymore to avoid more confusions. Glen Lee (9): staging: wilc1000: remove useless def

[PATCH v2 01/10] staging: wilc1000: remove PANDA_BOARD

2015-09-15 Thread Tony Cho
This patch removes one of the platform dependencies, PANDA_BOARD from the driver codes. The device tree will provide the platform dependencies in the future commits. Signed-off-by: Tony Cho Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 4 drivers/staging/wilc1

[PATCH v2 04/10] staging: wilc1000: remove unused defines

2015-09-15 Thread Tony Cho
From: Glen Lee This patch removes follwing unused defines. MAX_CFG_PKTLEN MSG_HEADER_LEN QUERY_MSG_TYPE WRITE_MSG_TYPE RESP_MSG_TYPE WRITE_RESP_SUCCESS INVALID MAC_ADDR_LEN Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 8 1 file cha

[PATCH v2 02/10] staging: wilc1000: remove useless define PHY_802_11n

2015-09-15 Thread Tony Cho
From: Glen Lee The define PHY_802_11n is useless because wilc1000 support 802.11n as always. Remove PHY_802_11n and it related code. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/

[PATCH v2 03/10] staging: wilc1000: remove unnecessary define FIVE_GHZ_BAND

2015-09-15 Thread Tony Cho
From: Glen Lee wilc1000 driver does not support five Ghz band. Remove FIVE_GHZ_BAND and it's related code. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/wilc1000/coreconf

[PATCH v2 08/10] staging: wilc1000: delete function declaration SendRawPacket

2015-09-15 Thread Tony Cho
From: Glen Lee SendRawpacket is declared but not implemented. It is not used also. So just delete it. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/wilc1000/coreconfigurator.h

[PATCH v2 07/10] staging: wilc1000: remove unused variables

2015-09-15 Thread Tony Cho
From: Glen Lee Variable g_num_total_switches is never used so just remove it. gastrWIDs is also not used because g_num_total_switches is deleted. Remove gastrWIDs also. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 127 --

[PATCH v2 09/10] staging: wilc1000: coreconfigurator.h: remove unused defines

2015-09-15 Thread Tony Cho
From: Glen Lee The define MAX_PACKET_BUFF_SIZE and STATUS_MSG_LEN is not used in the driver. Delete two defines. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/wilc1000/corec

[PATCH v2 05/10] staging: wilc1000: remove unnecessary extern function declaration

2015-09-15 Thread Tony Cho
From: Glen Lee Function NetworkInfoReceived, GnrlAsyncInfoReceived and host_int_ScanCompleteReceived are declared in the header file already. Just removes the declaration and comments. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 8

[PATCH v2 10/10] staging: wilc1000: remove unused variable g_flushing_in_progress

2015-09-15 Thread Tony Cho
From: Glen Lee g_flushing_in_progress is never used so just delete it. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c b/drivers/staging/

[PATCH v2 06/10] staging: wilc1000: delete unused global variables.

2015-09-15 Thread Tony Cho
From: Glen Lee gstrConfigPktInfo, SemHandleSendPkt and SemHandlePktResp are never used in the drivers. So just delete the variables and it's related code. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 11 --- 1 file changed, 11 delet

Re: [PATCH] staging: wilc1000: Removed curly braces

2015-09-15 Thread Sudip Mukherjee
On Tue, Sep 15, 2015 at 07:24:00PM +0530, Aparna Karuthodi wrote: > Removed the curly braces of a single statement if block to remove a > coding style warning detected by checkpatch. > The warning is given below: > WARNING: braces {} are not necessary for single statement blocks > > Signed-off-by:

RE: kernel log flooded with GET_CMD_NODE: cmd node not available

2015-09-15 Thread Amitkumar Karwar
> From: Marty Faltesek [mailto:mfalte...@google.com] > Sent: Friday, September 04, 2015 3:21 AM > To: Belisko Marek > Cc: Amitkumar Karwar; linux-wireless@vger.kernel.org > Subject: Re: kernel log flooded with GET_CMD_NODE: cmd node not > available > > On Mon, Aug 31, 2015 at 7:54 AM, Belisko Mare