[PATCH] ath10k: Replace bit shifts with the BIT() macro for rx desc bits

2018-05-08 Thread Govind Singh
Use the BIT() macro from 'linux/bitops.h' to define the rx desc bit flags to have consistency with new definitions. Signed-off-by: Govind Singh --- drivers/net/wireless/ath/ath10k/rx_desc.h | 134 +++--- 1 file changed, 67 insertions(+), 67 deletions(-)

Re: [PATCH v3 00/12] *** Add support for wifi QMI client driver ***

2018-05-08 Thread Govind Singh
On 2018-05-08 23:07, Bjorn Andersson wrote: On Sun 25 Mar 22:37 PDT 2018, Govind Singh wrote: Add QMI client driver for Q6 integrated WLAN connectivity subsystem. This module is responsible for communicating WLAN control messages to FW over QMI interface. “QUALCOMM Messaging Interface”(QMI)

Re: [PATCH 06/12] firmware: qcom: scm: Add WLAN VMID for Qualcomm SCM interface

2018-05-08 Thread Govind Singh
On 2018-05-08 02:12, Bjorn Andersson wrote: On Sun 25 Mar 22:40 PDT 2018, Govind Singh wrote: Add WLAN related VMID's to support wlan driver to set up the remote's permissions call via TrustZone. Signed-off-by: Govind Singh Please use ./scripts/get_maintainer.pl for

Re: [PATCH v6 00/13] firmware_loader changes for v4.18

2018-05-08 Thread Kees Cook
ware loader can use my linux-next > branch 20180508-firmware_loader_for-v4.18-try2 for now. > > 0-day sends its blessings. > > The patches from Mimi's series still require a bit more discussion and > review. The discussion over the EFI firmware fallback mechanism is still > ongo

Re: [PATCH v6 05/13] firmware_loader: enhance Kconfig documentation over FW_LOADER

2018-05-08 Thread Kees Cook
On Tue, May 8, 2018 at 11:12 AM, Luis R. Rodriguez wrote: > If you try to read FW_LOADER today it speaks of old riddles and > unless you have been following development closely you will loose Typo: lose > track of what is what. Even the documentation for

Re: [PATCH] nl80211: Reject disconnect commands except from conn_owner

2018-05-08 Thread Andrew Zaborowski
Hi, On 8 May 2018 at 14:19, Johannes Berg wrote: > On Tue, 2018-05-08 at 14:18 +0200, Arend van Spriel wrote: >> On 5/7/2018 9:19 PM, Johannes Berg wrote: >> > On Sun, 2018-04-29 at 20:30 +0200, Andrew Zaborowski wrote: >> > > On 28 April 2018 at 15:07, Kalle Valo

Re: [PATCH v2 1/2] wireless-regdb: Add wmm rule for EEA and EFTA countries

2018-05-08 Thread Seth Forshee
On Thu, May 03, 2018 at 06:58:22PM +0300, Haim Dreyfuss wrote: > The ETSI EN 301 893 v211 (2017-05) standard defines a new channel > access mechanism that all devices (WLAN and LAA) need to comply with. > In previous versions the device was allowed by ETSI to implement > 802.11 channel access

Re: [PATCH v6 13/13] Documentation: clarify firmware_class provenance and why we can't rename the module

2018-05-08 Thread Andres Rodriguez
On 2018-05-08 02:12 PM, Luis R. Rodriguez wrote: Clarify the provenance of the firmware loader firmware_class module name and why we cannot rename the module in the future. Signed-off-by: Luis R. Rodriguez --- .../driver-api/firmware/fallback-mechanisms.rst | 9

[PATCH v6 01/13] firmware: wrap FW_OPT_* into an enum

2018-05-08 Thread Luis R. Rodriguez
From: Andres Rodriguez This should let us associate enum kdoc to these values. While at it, kdocify the fw_opt. Signed-off-by: Andres Rodriguez Acked-by: Luis R. Rodriguez [mcgrof: coding style fixes, merge kdoc with enum move]

[PATCH v6 04/13] firmware_loader: document firmware_sysfs_fallback()

2018-05-08 Thread Luis R. Rodriguez
This also sets the expecations for future fallback interfaces, even if they are not exported. Signed-off-by: Luis R. Rodriguez --- drivers/base/firmware_loader/fallback.c | 20 1 file changed, 20 insertions(+) diff --git

[PATCH v6 02/13] firmware: use () to terminate kernel-doc function names

2018-05-08 Thread Luis R. Rodriguez
From: Andres Rodriguez The kernel-doc spec dictates a function name ends in (). Signed-off-by: Andres Rodriguez Acked-by: Randy Dunlap Acked-by: Luis R. Rodriguez [mcgrof: adjust since the wide API rename is

[PATCH v6 05/13] firmware_loader: enhance Kconfig documentation over FW_LOADER

2018-05-08 Thread Luis R. Rodriguez
If you try to read FW_LOADER today it speaks of old riddles and unless you have been following development closely you will loose track of what is what. Even the documentation for PREVENT_FIRMWARE_BUILD is a bit fuzzy and how it fits into this big picture. Give the FW_LOADER kconfig documentation

[PATCH v6 03/13] firmware: rename fw_sysfs_fallback to firmware_fallback_sysfs()

2018-05-08 Thread Luis R. Rodriguez
From: Andres Rodriguez This is done since this call is now exposed through kernel-doc, and since this also paves the way for different future types of fallback mechanims. Signed-off-by: Andres Rodriguez Acked-by: Luis R. Rodriguez

[PATCH v6 06/13] firmware_loader: move kconfig FW_LOADER entries to its own file

2018-05-08 Thread Luis R. Rodriguez
This will make it easier to track and easier to understand what components and features are part of the FW_LOADER. There are some components related to firmware which have *nothing* to do with the FW_LOADER, souch as PREVENT_FIRMWARE_BUILD. Signed-off-by: Luis R. Rodriguez ---

[PATCH v6 09/13] ath10k: use firmware_request_nowarn() to load firmware

2018-05-08 Thread Luis R. Rodriguez
From: Andres Rodriguez This reduces the unnecessary spew when trying to load optional firmware: "Direct firmware load for ... failed with error -2" Signed-off-by: Andres Rodriguez Acked-by: Kalle Valo Signed-off-by: Luis R.

[PATCH v6 07/13] firmware_loader: make firmware_fallback_sysfs() print more useful

2018-05-08 Thread Luis R. Rodriguez
If we resort to using the sysfs fallback mechanism we don't print the filename. This can be deceiving given we could have a series of callers intertwined and it'd be unclear exactly for what firmware this was meant for. Additionally, although we don't currently use FW_OPT_NO_WARN when dealing

[PATCH v6 10/13] ath10k: re-enable the firmware fallback mechanism for testmode

2018-05-08 Thread Luis R. Rodriguez
From: Andres Rodriguez The ath10k testmode uses request_firmware_direct() in order to avoid producing firmware load warnings. Disabling the fallback mechanism was a side effect of disabling warnings. We now have a new API that allows us to avoid warnings while keeping the

[PATCH v6 08/13] firmware: add firmware_request_nowarn() - load firmware without warnings

2018-05-08 Thread Luis R. Rodriguez
From: Andres Rodriguez Currently the firmware loader only exposes one silent path for querying optional firmware, and that is firmware_request_direct(). This function also disables the sysfs fallback mechanism, which might not always be the desired behaviour [0]. This patch

[PATCH v6 11/13] Documentation: fix few typos and clarifications for the firmware loader

2018-05-08 Thread Luis R. Rodriguez
Fix a few typos, and clarify a few sentences. Signed-off-by: Luis R. Rodriguez --- Documentation/driver-api/firmware/fallback-mechanisms.rst | 5 +++-- Documentation/driver-api/firmware/firmware_cache.rst | 4 ++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff

[PATCH v6 12/13] Documentation: remove stale firmware API reference

2018-05-08 Thread Luis R. Rodriguez
It refers to a pending patch, but this was merged eons ago. Signed-off-by: Luis R. Rodriguez --- Documentation/dell_rbu.txt | 3 --- 1 file changed, 3 deletions(-) diff --git a/Documentation/dell_rbu.txt b/Documentation/dell_rbu.txt index 0fdb6aa2704c..077fdc29a0d0 100644

[PATCH v6 13/13] Documentation: clarify firmware_class provenance and why we can't rename the module

2018-05-08 Thread Luis R. Rodriguez
Clarify the provenance of the firmware loader firmware_class module name and why we cannot rename the module in the future. Signed-off-by: Luis R. Rodriguez --- .../driver-api/firmware/fallback-mechanisms.rst | 9 ++--- 1 file changed, 6 insertions(+), 3

[PATCH v6 00/13] firmware_loader changes for v4.18

2018-05-08 Thread Luis R. Rodriguez
which have come up recently. These changes are available on my git tree both based on linux-next [0] and Linus' latest tree [1]. Folks working on new developments for the firmware loader can use my linux-next branch 20180508-firmware_loader_for-v4.18-try2 for now. 0-day sends its blessings

[mac80211-next:master 12/14] drivers/net//wireless/ath/wil6210/debugfs.c:1245:1: warning: the frame size of 1600 bytes is larger than 1024 bytes

2018-05-08 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211-next.git master head: 57c6cb81717f957fb741f2e4c79bd0e2f4f55910 commit: 52539ca89f365d3db530535fbffa88a3cca4d2ec [12/14] cfg80211: Expose TXQ stats and parameters to userspace config: i386-randconfig-i1-201818 (attached as

Re: [PATCH 00/18] Fix some build warnings/errors with Sphinx

2018-05-08 Thread Luis R. Rodriguez
On Tue, May 08, 2018 at 10:13:42AM -0600, Jonathan Corbet wrote: > On Mon, 7 May 2018 06:35:36 -0300 > Mauro Carvalho Chehab wrote: > > > I decided to give a try with Sphinx last stable version > > (1.17.4), and noticed several issues. The worse one was > > with the

Re: [PATCH v3 00/12] *** Add support for wifi QMI client driver ***

2018-05-08 Thread Bjorn Andersson
On Sun 25 Mar 22:37 PDT 2018, Govind Singh wrote: > Add QMI client driver for Q6 integrated WLAN connectivity subsystem. > This module is responsible for communicating WLAN control messages to FW > over QMI interface. > > “QUALCOMM Messaging Interface”(QMI) provides the control interface between

Re: [PATCH 3/6] firmware: differentiate between signed regulatory.db and other firmware

2018-05-08 Thread Luis R. Rodriguez
On Thu, May 03, 2018 at 08:24:26PM -0400, Mimi Zohar wrote: > On Fri, 2018-05-04 at 00:07 +, Luis R. Rodriguez wrote: > > On Tue, May 01, 2018 at 09:48:20AM -0400, Mimi Zohar wrote: > > > Allow LSMs and IMA to differentiate between signed regulatory.db and > > > other firmware. > > > > > >

[PATCH] iw: Add duration parameter to scan command

2018-05-08 Thread Pradeep Kumar Chitrapu
This patch lets user to specify duration(TUs) and set duration-mandatory flag in scan command. Signed-off-by: Pradeep Kumar Chitrapu --- scan.c | 31 --- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git a/scan.c b/scan.c index

[mac80211-next:master 12/14] htmldocs: include/net/cfg80211.h:3997: warning: Function parameter or member 'txq_limit' not described in 'wiphy'

2018-05-08 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211-next.git master head: 57c6cb81717f957fb741f2e4c79bd0e2f4f55910 commit: 52539ca89f365d3db530535fbffa88a3cca4d2ec [12/14] cfg80211: Expose TXQ stats and parameters to userspace reproduce: make htmldocs All warnings (new ones

Re: [PATCH 4.17] bcma: fix buffer size caused crash in bcma_core_mips_print_irq()

2018-05-08 Thread Joe Perches
On Tue, 2018-05-08 at 11:31 +0200, Rafał Miłecki wrote: > From: Rafał Miłecki > > Used buffer wasn't big enough to hold whole strings. Example output of > this function is: > [0.180892] bcma: bus0: core 0x0800, irq: 2(S)* 3 4 5 6 D I > [0.180948] bcma: bus0: core

Re: [PATCH 00/18] Fix some build warnings/errors with Sphinx

2018-05-08 Thread Jonathan Corbet
On Mon, 7 May 2018 06:35:36 -0300 Mauro Carvalho Chehab wrote: > I decided to give a try with Sphinx last stable version > (1.17.4), and noticed several issues. The worse one was > with the networking book: a non-standard footnote there > with [*] instead of a number

Re: [v2, 06/10] wil6210: use country specific board file upon reg domain change

2018-05-08 Thread Kalle Valo
Maya Erez wrote: > From: Dedy Lansky > > wil6210 device needs to use country specific board file while in China > regulatory domain. > Register cfg80211 reg_notifier and switch board file if needed > according to new regulatory domain. > This

Re: [PATCH v5 0/6] firmware_loader: cleanups for v4.18

2018-05-08 Thread Luis R. Rodriguez
On Fri, May 04, 2018 at 10:43:49AM -0700, Luis R. Rodriguez wrote: > Greg, > > I've reviewed the pending patches for the firmware_laoder and as for > v4.18, the following 3 patches from Andres have been iterated enough > that they're ready after I made some final minor changes, mostly just >

Re: [02/10] wil6210: set/get EDMG channel through debugfs

2018-05-08 Thread Kalle Valo
me...@codeaurora.org writes: > On 2018-04-24 12:47, Kalle Valo wrote: >> me...@codeaurora.org writes: >> >>> On 2018-04-19 19:15, Kalle Valo wrote: Maya Erez wrote: > Setting EDMG channel through debugfs for connect and PCP start > commands. Why? And

Re: [PATCHv2,3/3] ath10k: average ack rssi support for data frames

2018-05-08 Thread Kalle Valo
Balaji Pothunoori wrote: > Average ack rssi value is weighted average of ack rssi for > no of msdu's has been sent. > This feature is enabled by the host driver if firmware is capable. > After receiving event from host, firmware allocates the necessary > memory to store

Re: [PATCH] net: wireless: ath: ath9k: Fix a possible data race in ath_chanctx_set_next

2018-05-08 Thread Toke Høiland-Jørgensen
Kalle Valo writes: > Jia-Ju Bai writes: > >> The write operation to "sc->next_chan" is protected by >> the lock on line 1287, but the read operation to >> this data on line 1262 is not protected by the lock. >> Thus, there may exist a data race for

Re: [PATCH v2] ath10k: remove variables which set but not used

2018-05-08 Thread Jeff Johnson
On 2018-05-07 02:39, Kenneth Lu wrote: Variable buf_len and num_vdev_stats are bging assigned but never read. typo: bging => being

Re: [PATCH] net: wireless: ath: ath9k: Fix a possible data race in ath_chanctx_set_next

2018-05-08 Thread Kalle Valo
Jia-Ju Bai writes: > The write operation to "sc->next_chan" is protected by > the lock on line 1287, but the read operation to > this data on line 1262 is not protected by the lock. > Thus, there may exist a data race for "sc->next_chan". > > To fix this data race, the

Re: [PATCH] mac80211: ethtool: avoid 32 bit multiplication overflow

2018-05-08 Thread Johannes Berg
On Tue, 2018-05-08 at 13:57 +0100, Colin King wrote: > From: Colin Ian King > > The multiplication of 10 * cfg80211_calculate_bitrate() is a 32 bit > operation and can overflow if cfg80211_calculate_bitrate is greater > than 42949. Although I don't believe this is

[PATCH] mac80211: ethtool: avoid 32 bit multiplication overflow

2018-05-08 Thread Colin King
From: Colin Ian King The multiplication of 10 * cfg80211_calculate_bitrate() is a 32 bit operation and can overflow if cfg80211_calculate_bitrate is greater than 42949. Although I don't believe this is occurring at present, it would be safer to avoid the potential

Re: [PATCH] brcmfmac: Add support for bcm43364 wireless chipset

2018-05-08 Thread Kalle Valo
Arend van Spriel writes: > + Kalle (explicitly) > > On 5/8/2018 9:58 AM, Ulf Hansson wrote: >> On 4 May 2018 at 08:48, Sean Lanigan wrote: >>> Add support for the BCM43364 chipset via an SDIO interface, as used in >>> e.g. the Murata 1FX module.

Re: [PATCH] nl80211: Reject disconnect commands except from conn_owner

2018-05-08 Thread Arend van Spriel
On 5/8/2018 2:19 PM, Johannes Berg wrote: On Tue, 2018-05-08 at 14:18 +0200, Arend van Spriel wrote: On 5/7/2018 9:19 PM, Johannes Berg wrote: On Sun, 2018-04-29 at 20:30 +0200, Andrew Zaborowski wrote: On 28 April 2018 at 15:07, Kalle Valo wrote: Andrew Zaborowski

Re: [PATCH] nl80211: Reject disconnect commands except from conn_owner

2018-05-08 Thread Johannes Berg
On Tue, 2018-05-08 at 14:18 +0200, Arend van Spriel wrote: > On 5/7/2018 9:19 PM, Johannes Berg wrote: > > On Sun, 2018-04-29 at 20:30 +0200, Andrew Zaborowski wrote: > > > On 28 April 2018 at 15:07, Kalle Valo wrote: > > > > Andrew Zaborowski

Re: [PATCH] nl80211: Reject disconnect commands except from conn_owner

2018-05-08 Thread Arend van Spriel
On 5/7/2018 9:19 PM, Johannes Berg wrote: On Sun, 2018-04-29 at 20:30 +0200, Andrew Zaborowski wrote: On 28 April 2018 at 15:07, Kalle Valo wrote: Andrew Zaborowski writes: Reject NL80211_CMD_DISCONNECT, NL80211_CMD_DISASSOCIATE,

Re: [PATCH] brcmfmac: Add support for bcm43364 wireless chipset

2018-05-08 Thread Arend van Spriel
+ Kalle (explicitly) On 5/8/2018 9:58 AM, Ulf Hansson wrote: On 4 May 2018 at 08:48, Sean Lanigan wrote: Add support for the BCM43364 chipset via an SDIO interface, as used in e.g. the Murata 1FX module. The BCM43364 uses the same firmware as the BCM43430 (which is already

[PATCH] ath10k: add fw crash dump support for QCA9888 and QCA99X0

2018-05-08 Thread Anilkumar Kolli
This patch adds crash dump support for QCA9888 and QCA99X0 Tested On: QCA9888, Firmware Version: 10.4-3.5.3-00053 Tested on QCA99X0, Firmware Version: 10.4.1.00030-1 Signed-off-by: Anilkumar Kolli --- drivers/net/wireless/ath/ath10k/coredump.c | 98

[PATCH v2 2/2] iw: Add getting and setting of TXQ params for phy

2018-05-08 Thread Toke Høiland-Jørgensen
This adds commands to get and set the per-phy TXQ parameters for drivers that use the intermediate TXQs. These are the settings and statistics that are also available through /sys/kernel/debug/ieee80211/phyX/aqm. Sample output: $ iw phy phy0 get txq Packet limit: 8192 pkts Memory

[PATCH v2 1/2] iw: Print TXQ statistics for stations and interfaces

2018-05-08 Thread Toke Høiland-Jørgensen
This adds printing of TXQ statistics for stations and interfaces when supplied by the kernel. For stations, another section is added when verbose mode is enabled; for interfaces, the multicast queue information is always printed when available. This is the information also available through

[PATCH v2 2/2] mac80211: Support the new cfg80211 TXQ stats API

2018-05-08 Thread Toke Høiland-Jørgensen
This adds support to mac80211 to export TXQ stats via the newly added cfg80211 API. Signed-off-by: Toke Høiland-Jørgensen --- net/mac80211/cfg.c | 99 net/mac80211/ieee80211_i.h |3 + net/mac80211/main.c|3 +

[PATCH v2 1/2] cfg80211: Expose TXQ stats and parameters to userspace

2018-05-08 Thread Toke Høiland-Jørgensen
This adds support for exporting the mac80211 TXQ stats via nl80211 by way of a nested TXQ stats attribute, as well as for configuring the quantum and limits that were previously only changeable through debugfs. This commit adds just the nl80211 API, a subsequent commit adds support to mac80211

Re: [PATCH 1/3] cfg80211: Expose TXQ stats and parameters to userspace

2018-05-08 Thread Toke Høiland-Jørgensen
Johannes Berg writes: > On Tue, 2018-05-08 at 12:18 +0200, Toke Høiland-Jørgensen wrote: >> Johannes Berg writes: >> >> > net/mac80211/cfg.c:3762:12: warning: context imbalance in >> > 'ieee80211_get_txq_stats' - different lock contexts for

Re: [PATCH 1/3] cfg80211: Expose TXQ stats and parameters to userspace

2018-05-08 Thread Johannes Berg
On Mon, 2018-02-19 at 18:02 +0100, Toke Høiland-Jørgensen wrote: > +static int ieee80211_get_txq_stats(struct wiphy *wiphy, > + struct wireless_dev *wdev, > + struct cfg80211_txq_stats *txqstats) > +{ > + struct

Re: [PATCH 1/3] cfg80211: Expose TXQ stats and parameters to userspace

2018-05-08 Thread Johannes Berg
On Tue, 2018-05-08 at 12:18 +0200, Toke Høiland-Jørgensen wrote: > Johannes Berg writes: > > > net/mac80211/cfg.c:3762:12: warning: context imbalance in > > 'ieee80211_get_txq_stats' - different lock contexts for basic block > > Found and fixed all the other warnings,

Re: [PATCH 1/3] cfg80211: Expose TXQ stats and parameters to userspace

2018-05-08 Thread Johannes Berg
On Tue, 2018-05-08 at 12:18 +0200, Toke Høiland-Jørgensen wrote: > Johannes Berg writes: > > > net/mac80211/cfg.c:3762:12: warning: context imbalance in > > 'ieee80211_get_txq_stats' - different lock contexts for basic block > > Found and fixed all the other warnings,

Re: [PATCH 1/3] cfg80211: Expose TXQ stats and parameters to userspace

2018-05-08 Thread Toke Høiland-Jørgensen
Johannes Berg writes: > net/mac80211/cfg.c:3762:12: warning: context imbalance in > 'ieee80211_get_txq_stats' - different lock contexts for basic block Found and fixed all the other warnings, but I'm not seeing this one. And I don't really see what is wrong with the

Re: [PATCH 4.17] bcma: fix buffer size caused crash in bcma_core_mips_print_irq()

2018-05-08 Thread Kalle Valo
Rafał Miłecki writes: > From: Rafał Miłecki > > Used buffer wasn't big enough to hold whole strings. Example output of > this function is: > [0.180892] bcma: bus0: core 0x0800, irq: 2(S)* 3 4 5 6 D I > [0.180948] bcma: bus0: core 0x0812, irq:

[PATCH 4.17] bcma: fix buffer size caused crash in bcma_core_mips_print_irq()

2018-05-08 Thread Rafał Miłecki
From: Rafał Miłecki Used buffer wasn't big enough to hold whole strings. Example output of this function is: [0.180892] bcma: bus0: core 0x0800, irq: 2(S)* 3 4 5 6 D I [0.180948] bcma: bus0: core 0x0812, irq: 2(S) 3* 4 5 6 D I [0.180998] bcma: bus0: core

Re: RTL8723BE performance regression

2018-05-08 Thread Pkshih
On Mon, 2018-05-07 at 14:49 -0700, João Paulo Rechi Vita wrote: > On Tue, May 1, 2018 at 10:58 PM, Pkshih wrote: > > On Wed, 2018-05-02 at 05:44 +, Pkshih wrote: > >> > >> > -Original Message- > >> > From: João Paulo Rechi Vita [mailto:jprv...@gmail.com] > >> >

[PATCH] net: wireless: ath: ath9k: Fix a possible data race in ath_chanctx_set_next

2018-05-08 Thread Jia-Ju Bai
The write operation to "sc->next_chan" is protected by the lock on line 1287, but the read operation to this data on line 1262 is not protected by the lock. Thus, there may exist a data race for "sc->next_chan". To fix this data race, the read operation to "sc->next_chan" should be also

Re: [PATCH] brcmfmac: Add support for bcm43364 wireless chipset

2018-05-08 Thread Ulf Hansson
On 4 May 2018 at 08:48, Sean Lanigan wrote: > Add support for the BCM43364 chipset via an SDIO interface, as used in > e.g. the Murata 1FX module. > > The BCM43364 uses the same firmware as the BCM43430 (which is already > included), the only difference is the omission of