Re: [PATCH 4/5] staging: wilc1000: SendConfigPkt(): replace integer with void pointer

2015-07-10 Thread Julian Calaby
Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: This patch replaces a integer argument of SendConfigPkt function with void type pointer and fix code that the function is called. Because tstrWILC_WFIDrv structure is defined after SendConfigPkt function is

Re: [PATCH 3/3] staging: wilc1000: wilc_wlan_cfg_set(): replace integer with void pointer

2015-07-10 Thread Julian Calaby
Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: Last argument of wilc_wlan_cfg_set function is actually structure's address. This should be changed to be compatible with 64bit machine. Because wilc_wlan_cfg_set function is mapped by function pointer later,

Re: [PATCH 1/3] staging: wilc1000: wilc_wlan_cfg_commit(): replace integer with pointer

2015-07-10 Thread Julian Calaby
Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: A argument of wilc_wlan_cfg_commit() is address of structure. But because the size is restricted to 32bit, it is not correct for 64bit machine. So, this changes the interger value to obvious structure pointer.

Re: [PATCH 2/3] staging: wilc1000: wilc_wlan_cfg_get(): replace integer with void pointer

2015-07-10 Thread Julian Calaby
Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: Last argument of wilc_wlan_cfg_get function is actually structure's address. This should be changed to be compatible with 64bit machine. Because wilc_wlan_cfg_get function is mapped by function pointer later,

[PATCH] nfc: Drop owner assignment from i2c_driver

2015-07-10 Thread Krzysztof Kozlowski
i2c_driver does not need to set an owner because i2c_register_driver() will set it. Signed-off-by: Krzysztof Kozlowski k.kozlow...@samsung.com --- The coccinelle script which generated the patch was sent here: http://www.spinics.net/lists/kernel/msg2029903.html --- drivers/nfc/nxp-nci/i2c.c |

Re: [PATCH] ath10k: Delay device access after cold reset

2015-07-10 Thread Kalle Valo
Vasanthakumar Thiagarajan vthia...@qti.qualcomm.com writes: It is observed that during cold reset pcie access right after a write operation to SOC_GLOBAL_RESET_ADDRESS causes Data Bus Error and system hard lockup. The reason for bus error is that pcie needs some time to get back to stable

[PATCH V2] ath10k: Delay device access after cold reset

2015-07-10 Thread Vasanthakumar Thiagarajan
It is observed that during cold reset pcie access right after a write operation to SOC_GLOBAL_RESET_ADDRESS causes Data Bus Error and system hard lockup. The reason for bus error is that pcie needs some time to get back to stable state for any transaction during cold reset. Add delay of 20 msecs

Re: [PATCH 2/3] brcmfmac: dhd_sdio.c: use existing atomic_or primitive

2015-07-10 Thread Arend van Spriel
On 07/10/2015 06:49 AM, Vineet Gupta wrote: On Thursday 09 July 2015 11:55 PM, Arend van Spriel wrote: On 07/09/2015 10:13 AM, Vineet Gupta wrote: There's already a generic implementation so use that instead. There is or there was? If there is now I am fine with this patch, but if it already

Re: WARNING: brcmfmac/core.c:1144 brcmf_netdev_wait_pend8021x

2015-07-10 Thread Arend van Spriel
On 07/09/2015 09:12 PM, Rafał Miłecki wrote: On 9 July 2015 at 20:48, Arend van Spriel ar...@broadcom.com wrote: On 07/09/2015 08:01 PM, Rafał Miłecki wrote: After fixing user space - brcmfmac communication for using valid MACs: [PATCH] brcmfmac: set wiphy's addresses to provide valid MACs i

Re: [PATCH 1/3] staging: wilc1000: wilc_wlan_cfg_commit(): replace integer with pointer

2015-07-10 Thread Johnny Kim
On 2015년 07월 10일 15:20, Julian Calaby wrote: Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: A argument of wilc_wlan_cfg_commit() is address of structure. But because the size is restricted to 32bit, it is not correct for 64bit machine. So, this changes the

Re: [PATCH 2/3] staging: wilc1000: wilc_wlan_cfg_get(): replace integer with void pointer

2015-07-10 Thread Johnny Kim
On 2015년 07월 10일 15:25, Julian Calaby wrote: Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: Last argument of wilc_wlan_cfg_get function is actually structure's address. This should be changed to be compatible with 64bit machine. Because wilc_wlan_cfg_get

Re: [PATCH 1/3] staging: wilc1000: wilc_wlan_cfg_commit(): replace integer with pointer

2015-07-10 Thread Julian Calaby
Hi Johnny, On Fri, Jul 10, 2015 at 5:58 PM, Johnny Kim johnny@atmel.com wrote: On 2015년 07월 10일 15:20, Julian Calaby wrote: Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: A argument of wilc_wlan_cfg_commit() is address of structure. But because the

Re: [PATCH 2/3] staging: wilc1000: wilc_wlan_cfg_get(): replace integer with void pointer

2015-07-10 Thread Julian Calaby
Hi Johnny, On Fri, Jul 10, 2015 at 6:11 PM, Johnny Kim johnny@atmel.com wrote: On 2015년 07월 10일 15:25, Julian Calaby wrote: Hi Johnny, On Fri, Jul 10, 2015 at 3:55 PM, Johnny Kim johnny@atmel.com wrote: Last argument of wilc_wlan_cfg_get function is actually structure's address.

Re: ath9k_htc: virtual interfaces, AP connection drop kernel warning

2015-07-10 Thread Oleksij Rempel
Hi, Am 08.07.2015 um 17:28 schrieb Rolf Anderegg: I am having trouble with the following setup using ath9k_htc (AR9271): I would like to setup two virtual interfaces on a TP-Link Wireless USB adapter (TL-WN722N). ath0: AP (access point/master mode) with hostapd [1], on bridge br0 ath1: STA

Re: How is AP supposed to handle power-save packets from peer?

2015-07-10 Thread Ben Greear
On 07/09/2015 10:20 PM, Michal Kazior wrote: On 10 July 2015 at 02:03, Ben Greear gree...@candelatech.com wrote: Suppose one is doing heavy download (AP - peer traffic), and there are lots of frames in the NIC's tx buffers (ath10k firmware, in this case). Then, peer sends a power-save pkt

[PATCH 7/7] brcmfmac: block the correct flowring when backup queue overflow

2015-07-10 Thread Arend van Spriel
From: Franky Lin fran...@broadcom.com brcmf_flowring_block blocks the last active flowring under the same interface instead of the one provided by caller. This could lead to a dead lock of netif stop if there are more than one flowring under the interface and the traffic is high enough so

[PATCH 6/7] brcmfmac: consolidate ifp lookup in driver core

2015-07-10 Thread Arend van Spriel
In rx path the firmware provide an interface index which is used to map to a struct brcmf_if instance. However, this involves some trick that is done in two places. This is changed by having driver core providing brcmf_get_ifp() function. Reviewed-by: Hante Meuleman meule...@broadcom.com

[PATCH 5/7] brcmfmac: make use of cfg80211_check_combinations()

2015-07-10 Thread Arend van Spriel
Use cfg80211_check_combinations() so we can bail out early when an interface add or change results in an invalid combination. Reviewed-by: Hante Meuleman meule...@broadcom.com Reviewed-by: Franky (Zhenhui) Lin fran...@broadcom.com Reviewed-by: Pieter-Paul Giesberts piete...@broadcom.com

[PATCH 1/7] brcmfmac: Add support for host platform NVRAM loading.

2015-07-10 Thread Arend van Spriel
From: Hante Meuleman meule...@broadcom.com Host platforms such as routers supported by OpenWRT can support NVRAM reading directly from internal NVRAM store. With this patch the nvram load routines will fall back to this method when there is no nvram file and support is available in the kernel.

[PATCH 2/7] brcmfmac: correct interface combination info

2015-07-10 Thread Arend van Spriel
The interface combination provided by brcmfmac did not truly reflect the combinations supported by driver and/or firmware. Reviewed-by: Hante Meuleman meule...@broadcom.com Reviewed-by: Franky (Zhenhui) Lin fran...@broadcom.com Reviewed-by: Pieter-Paul Giesberts piete...@broadcom.com Reviewed-by:

[PATCH 4/7] brcmfmac: add debugfs entry for msgbuf statistics

2015-07-10 Thread Arend van Spriel
From: Franky Lin fran...@broadcom.com Expose ring buffer read/write pointers and other useful statistics through debugfs. Reviewed-by: Arend Van Spriel ar...@broadcom.com Reviewed-by: Hante Meuleman meule...@broadcom.com Reviewed-by: Pieter-Paul Giesberts piete...@broadcom.com Signed-off-by:

[PATCH 3/7] brcmfmac: Increase nr of supported flowrings.

2015-07-10 Thread Arend van Spriel
From: Hante Meuleman meule...@broadcom.com Next generation devices will have firmware which will have more than 256 flowrings. This patch increases the maximum number of supported flowrings to 512. Reviewed-by: Arend Van Spriel ar...@broadcom.com Reviewed-by: Franky (Zhenhui) Lin

[PATCH 0/7] brcmfmac: nvram loading and code rework

2015-07-10 Thread Arend van Spriel
This series comprises of following changes: - support NVRAM loading for bcm47xx platform. - revise announced interface combinations and validate against it. - new debugfs entry for msgbuf protocol layer used with PCIe devices. - couple of PCIe fixes. - rework dealing with interface instances. The