Support of rtl8723bs

2016-09-12 Thread Hanno Zulla
Hi Jes, hello linux-wireless, I'd like to know more about the current progress of rtl8723bs support for Linux. Googling and reading the mailing list archives found an older vendor driver https://github.com/hadess/rtl8723bs plus patches at https://github.com/lwfinger/rtl8723bs which both aren't

Re: [PATCH 1/3 v3] mac80211: RX BA support for sta max_rx_aggregation_subframes

2016-09-12 Thread Johannes Berg
On Mon, 2016-09-12 at 11:37 +0200, Johannes Berg wrote: >  > I fixed the indentation of the commit message, please note for the > future. > Err, what? No, the *debug* message of course. johannes

Re: [PATCH] mac80211: fix sequence number assignment for PS response frames

2016-09-12 Thread Felix Fietkau
On 2016-09-12 12:03, Johannes Berg wrote: > >> Hm, I guess I didn't think of that. I guess this potential issue will >> go away once we get Toke's tx handler reorder patch fixed, rebased >> and integrated. >> > > I don't really see how that helps? It replaces the changes that I made. - Felix

Re: [PATCH] mac80211: make mpath path fixing more robust

2016-09-12 Thread Johannes Berg
On Tue, 2016-09-06 at 11:59 -0700, Thomas Pedersen wrote: > A fixed mpath was not quite being treated as such: > Applied. johannes

Re: [PATCH 1/3 v3] mac80211: RX BA support for sta max_rx_aggregation_subframes

2016-09-12 Thread Johannes Berg
On Mon, 2016-08-22 at 17:14 +0300, Maxim Altshul wrote: > The ability to change the max_rx_aggregation frames is useful > in cases of IOP. > > There exist some devices (latest mobile phones and some AP's) > that tend to not respect a BA sessions maximum size (in Kbps). > These devices won't

[PATCH 0/9] rsi: code clean-up

2016-09-12 Thread Prameela Rani Garnepudi
*** BLURB HERE *** Prameela Rani Garnepudi (9): rsi: cleanup and added debug prints in file rsi_91x_core.c rsi: Spelling and code format corrections in rsi_91x_debugfs.c rsi: Added freeing of allocated buffers (sbands and dfsentry) Cleanup (spelling mistakes, code format

Re: [PATCH] mac80211: fix sequence number assignment for PS response frames

2016-09-12 Thread Johannes Berg
On Sun, 2016-09-04 at 18:00 +0200, Felix Fietkau wrote: > When using intermediate queues, sequence number allocation is > deferred > until dequeue. This doesn't work for PS response frames, which bypass > those queues. > Applied. This worries me a bit though - there's nothing, afaict, that

Re: [PATCH] mac80211: fix tim recalculation after PS response

2016-09-12 Thread Johannes Berg
On Sun, 2016-08-28 at 13:10 +0200, Felix Fietkau wrote: > Handle the case where the mac80211 intermediate queues are empty and > the driver has buffered frames > Applied. johannes

Re: [PATCH v8] cfg80211: Provision to allow the support for different beacon intervals

2016-09-12 Thread Johannes Berg
Hmm. Apart from some misleading documentation, that doesn't mention "GCD" at all, the logic seems odd. In order to validate a new beacon interval, you're first looking up the min GCD value of all the combinations that allow the *current* scenario, but doing that matching without the right # of

Re: [PATCH v2] mac80211: Re-structure aqm debugfs output and include CoDel drops

2016-09-12 Thread Johannes Berg
> --- a/net/mac80211/debugfs_sta.c > +++ b/net/mac80211/debugfs_sta.c > @@ -109,6 +109,48 @@ static ssize_t sta_last_seq_ctrl_read(struct > file *file, char __user *userbuf, >  } >  STA_OPS(last_seq_ctrl); >   > +#define AQM_TXQ_ENTRY_LEN 130 > + > +static ssize_t sta_aqm_read(struct file *file,

[PATCH 4/9] rsi: code clean up in file rsi_91x_main.c

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_main.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_main.c b/drivers/net/wireless/rsi/rsi_91x_main.c index

[PATCH 3/9] rsi: Added freeing of allocated buffers (sbands and dfsentry) Cleanup (spelling mistakes, code format corrections, empty lines after declaration)

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_mac80211.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_mac80211.c

[PATCH 8/9] rsi: code clean up and replaced ctrl message timeouts with macros in file rsi_91x_usb.c

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_usb.c | 42 +++--- 1 file changed, 23 insertions(+), 19 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_usb.c b/drivers/net/wireless/rsi/rsi_91x_usb.c

[PATCH 5/9] rsi: code cleanup in file rsi_91x_mgmt.c

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_mgmt.c | 30 -- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_mgmt.c b/drivers/net/wireless/rsi/rsi_91x_mgmt.c

[PATCH 9/9] rsi: code cleanup in file rsi_91x_usb_ops.c

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_usb_ops.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_usb_ops.c b/drivers/net/wireless/rsi/rsi_91x_usb_ops.c index

[PATCH 6/9] rsi: code cleanup (spelling mistakes and new line after declaration) in file rsi_91x_sdio.c Added developer name

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_sdio.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_sdio.c b/drivers/net/wireless/rsi/rsi_91x_sdio.c index

[PATCH 7/9] rsi: zone changed for debug message and cleanup in file rsi_91x_sdio_ops.c

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_sdio_ops.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_sdio_ops.c b/drivers/net/wireless/rsi/rsi_91x_sdio_ops.c index

Re: [PATCH] cfg80211: Add HT and VHT information in start_ap

2016-09-12 Thread Johannes Berg
So I haven't seen a response from you guys: I have no major objections to this. However, a few things: 1) are you planning to add support for this into a kernel driver at    all, anyway? 2) are you planning to have a driver upstream that contains the now    necessary parsing? Depending on the

Re: [PATCH] rtlbt: Add RTL8822BE Bluetooth device

2016-09-12 Thread Marcel Holtmann
Hi Larry, > The RTL8822BE is a new Realtek wifi and BT device. Support for the BT > part is hereby added. > > As this device is similar to most of the other Realtek BT devices, the > changes are minimal. The main difference is that the 8822BE needs a > configuration file for enabling and

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Julia Lawall
On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: > > Constify local structures. > > > > The semantic patch that makes this change is as follows: > > (http://coccinelle.lip6.fr/) > > Just my two cents but: > > 1. You *can* use a static

Re: [PATCH v2 0/6] some mac80211/cfg80211 patches

2016-09-12 Thread Johannes Berg
Applied all now. It seems I got held up somewhere in the middle before. johannes

Re: [PATCH] mac80211: fix sequence number assignment for PS response frames

2016-09-12 Thread Johannes Berg
On Mon, 2016-09-12 at 12:05 +0200, Felix Fietkau wrote: > On 2016-09-12 12:03, Johannes Berg wrote: > > > > > > > > > > Hm, I guess I didn't think of that. I guess this potential issue > > > will > > > go away once we get Toke's tx handler reorder patch fixed, > > > rebased > > > and

Re: [PATCH] mac80211: fix sequence number assignment for PS response frames

2016-09-12 Thread Felix Fietkau
On 2016-09-12 12:07, Johannes Berg wrote: > On Mon, 2016-09-12 at 12:05 +0200, Felix Fietkau wrote: >> On 2016-09-12 12:03, Johannes Berg wrote: >> > >> > >> > > >> > > Hm, I guess I didn't think of that. I guess this potential issue >> > > will >> > > go away once we get Toke's tx handler

Re: [PATCH v3] cfg80211: Add support to configure a beacon data rate

2016-09-12 Thread Johannes Berg
> +static u32 rateset_to_mask(struct ieee80211_supported_band *sband, > +    u8 *rates, u8 rates_len) All of these just move around, right? > + if (info->attrs[NL80211_ATTR_TX_RATES]) { > + u32 rate, count_ht, count_vht, i; > + enum nl80211_band

[PATCH 1/9] rsi: cleanup and added debug prints in file rsi_91x_core.c

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_core.c | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_core.c b/drivers/net/wireless/rsi/rsi_91x_core.c index

[PATCH 2/9] rsi: Spelling and code format corrections in rsi_91x_debugfs.c

2016-09-12 Thread Prameela Rani Garnepudi
Signed-off-by: Prameela Rani Garnepudi --- drivers/net/wireless/rsi/rsi_91x_debugfs.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/net/wireless/rsi/rsi_91x_debugfs.c b/drivers/net/wireless/rsi/rsi_91x_debugfs.c index

Re: [PATCH] mac80211: fix sequence number assignment for PS response frames

2016-09-12 Thread Johannes Berg
> Hm, I guess I didn't think of that. I guess this potential issue will > go away once we get Toke's tx handler reorder patch fixed, rebased > and integrated. > I don't really see how that helps? johannes

Re: [PATCH v3 1/6] staging: rtl8723au: remove unimplemented functions declaration

2016-09-12 Thread Greg Kroah-Hartman
On Tue, Sep 06, 2016 at 06:36:52PM +0200, Luca Ceresoli wrote: > These functions have been declared without any implementation since > the first commit (364e30ebd2dbaccba430c603da03e68746eb932a) and there > has been no mention of them in following commits. > > Signed-off-by: Luca Ceresoli

Re: [PATCH v2 1/6] rtl8723au: remove declaration of unimplemented functions

2016-09-12 Thread Greg Kroah-Hartman
On Fri, Sep 02, 2016 at 02:57:44PM +0200, Luca Ceresoli wrote: > These functions have been declared without any implementation since > the first commit (364e30ebd2dbaccba430c603da03e68746eb932a) and there > has been no mention of them in following commits. > > Signed-off-by: Luca Ceresoli

Re: [PATCH 2/3] ath10k: Grab rcu_read_lock before the txqs spinlock.

2016-09-12 Thread Johannes Berg
> > > - rcu_read_unlock(); > > >   spin_unlock_bh(>txqs_lock); > > > + rcu_read_unlock(); > > > > I'm no RCU expert but this isn't making any sense. Maybe it changes > > timings on your kernel so that it hides the real problem? > > I'm not sure this fixed anything or not, it just seemed

Re: [PATCH] cfg80211: cap 20MHz VHT bitrate at MCS 8

2016-09-12 Thread Johannes Berg
On Wed, 2016-09-07 at 18:20 +, Pedersen, Thomas wrote: > On 09/06/2016 12:07 PM, Ben Greear wrote: > > > > On 09/06/2016 12:00 PM, Thomas Pedersen wrote: > > > > > > Some drivers (ath10k) report MCS 9 @ 20MHz, which > > > technically isn't allowed. To get more meaningful value > > > than 0

Re: support for QCA9377 SDIO (ath10k or ath6kl)

2016-09-12 Thread Valo, Kalle
Martin Blumenstingl writes: > is QCA planning to add SDIO support to ath10k or would this have to be > done by a community developer? I cannot really comment about company plans. -- Kalle Valo

Re: [PATCH] mac80211: fix sequence number assignment for PS response frames

2016-09-12 Thread Toke Høiland-Jørgensen
Felix Fietkau writes: > On 2016-09-12 12:07, Johannes Berg wrote: >> On Mon, 2016-09-12 at 12:05 +0200, Felix Fietkau wrote: >>> On 2016-09-12 12:03, Johannes Berg wrote: >>> > >>> > >>> > > >>> > > Hm, I guess I didn't think of that. I guess this potential

Re: [PATCH 08/26] iwlegacy: constify local structures

2016-09-12 Thread Stanislaw Gruszka
On Sun, Sep 11, 2016 at 03:05:50PM +0200, Julia Lawall wrote: > For structure types defined in the same file or local header files, find > top-level static structure declarations that have the following > properties: > 1. Never reassigned. > 2. Address never taken > 3. Not passed to a top-level

Re: Support of rtl8723bs

2016-09-12 Thread Bastien Nocera
On Mon, 2016-09-12 at 11:20 +0200, Hanno Zulla wrote: > Hi Jes, > hello linux-wireless, > > I'd like to know more about the current progress of rtl8723bs support > for Linux. > > Googling and reading the mailing list archives found an older vendor > driver > https://github.com/hadess/rtl8723bs

[PATCH] rsi: fix ifnullfree.cocci warnings

2016-09-12 Thread kbuild test robot
drivers/net/wireless/rsi/rsi_91x_mac80211.c:209:3-8: WARNING: NULL check before freeing functions like kfree, debugfs_remove, debugfs_remove_recursive or usb_free_urb is not needed. Maybe consider reorganizing relevant code to avoid passing NULL values. NULL check before some freeing

Re: [PATCH 3/9] rsi: Added freeing of allocated buffers (sbands and dfsentry) Cleanup (spelling mistakes, code format corrections, empty lines after declaration)

2016-09-12 Thread kbuild test robot
Hi Prameela, [auto build test WARNING on wireless-drivers-next/master] [also build test WARNING on v4.8-rc6 next-20160912] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=a

Re: [PATCH v8] mac80211: Move reorder-sensitive TX handlers to after TXQ dequeue.

2016-09-12 Thread Johannes Berg
> +static int invoke_tx_handlers_late(struct ieee80211_tx_data *tx); > +static bool ieee80211_xmit_fast_finish(struct ieee80211_sub_if_data *sdata, > +    struct sta_info *sta, u8 pn_offs, > +    struct ieee80211_key_conf *key_conf,

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Julia Lawall
On Mon, 12 Sep 2016, Jarkko Sakkinen wrote: > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > > > > > On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > > > > > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: > > > > Constify local structures. > > > > > > > > The

[PATCH v3] mac80211: Re-structure aqm debugfs output and keep CoDel stats per txq

2016-09-12 Thread Toke Høiland-Jørgensen
Currently the 'aqm' stats in mac80211 only keeps overlimit drop stats, not CoDel stats. This moves the CoDel stats into the txqi structure to keep them per txq in order to show them in debugfs. In addition, the aqm debugfs output is restructured by splitting it up into three files: One global per

Re: [PATCH 0/9] rsi: code clean-up

2016-09-12 Thread Kalle Valo
Prameela Rani Garnepudi writes: > *** BLURB HERE *** This has a meaning :) > Prameela Rani Garnepudi (9): > rsi: cleanup and added debug prints in file rsi_91x_core.c > rsi: Spelling and code format corrections in rsi_91x_debugfs.c > rsi: Added freeing of

Re: [PATCH v8] mac80211: Move reorder-sensitive TX handlers to after TXQ dequeue.

2016-09-12 Thread Toke Høiland-Jørgensen
Johannes Berg writes: >> +static int invoke_tx_handlers_late(struct ieee80211_tx_data *tx); >> +static bool ieee80211_xmit_fast_finish(struct ieee80211_sub_if_data *sdata, >> +   struct sta_info *sta, u8 pn_offs, >> +

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Felipe Balbi
Hi, Jarkko Sakkinen writes: > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: >> >> >> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: >> >> > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: >> > > Constify local structures. >> > > >>

pull-request mwifiex-firmware 2015-09-12

2016-09-12 Thread Amitkumar Karwar
The following changes since commit e92f8b3f65443764297b947b1843955d9a65dde7: linux-firmware: update Marvell USB8797-B0 firmware image (2015-11-02 06:25:05 -0500) are available in the git repository at: git://git.marvell.com/mwifiex-firmware.git master for you to fetch changes up to

Re: [PATCH] mac80211: allow driver to handle packet-loss mechanism

2016-09-12 Thread Johannes Berg
On Tue, 2016-09-06 at 12:26 +0530, Rajkumar Manoharan wrote: > mac80211 keeps track of missing acks and triggers CQM packet-loss > mechanism whenever consecutive msdu failure reaches threshold limit > (STA_LOST_PKT_THRESHOLD). Drivers like ath10k offlaoded rate countrol > and aggregation to

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Jarkko Sakkinen
On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > > On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > > > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: > > > Constify local structures. > > > > > > The semantic patch that makes this change is as follows: > > >

[PATCH 1/2] mwifiex: Command 7 handling for USB chipsets

2016-09-12 Thread Amitkumar Karwar
From: Ganapathi Bhat Firmware image for newer USB chipsets starts with a command 7 block (special command). It doesn't contain data length field. This patch adds necessary handling. Signed-off-by: Ganapathi Bhat Signed-off-by: Amitkumar Karwar

[PATCH 0/2] mwifiex: usb8997 chipset support

2016-09-12 Thread Amitkumar Karwar
This patch series includes changes needed to support Marvell usb8997 chipset. Ganapathi Bhat (2): mwifiex: Command 7 handling for USB chipsets mwifiex: firmware name correction for usb8997 chipset drivers/net/wireless/marvell/mwifiex/usb.c | 4

[PATCH 2/2] mwifiex: firmware name correction for usb8997 chipset

2016-09-12 Thread Amitkumar Karwar
From: Ganapathi Bhat Similar to pcie8997 chipset, first firmware submitted for usb8997 chipset will be usbusb8997_combo_v4.bin. This patch corrects the name used in driver. Signed-off-by: Ganapathi Bhat Signed-off-by: Amitkumar Karwar

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Geert Uytterhoeven
On Mon, Sep 12, 2016 at 3:43 PM, Felipe Balbi wrote: > Jarkko Sakkinen writes: >> On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: >>> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: >>> > On Sun, Sep 11, 2016 at 03:05:42PM

Re: [PATCH v8] mac80211: Move reorder-sensitive TX handlers to after TXQ dequeue.

2016-09-12 Thread Johannes Berg
> Well, the TXQ already adds a lot of other overhead (hashing on the > packet header, for one), so my guess would be that this would be > negligible compared to all that?  > > > > > I suppose I don't have to care all that much about the TXQs, but > > ... > > > > Then again, adding a field in

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Julia Lawall
On Mon, 12 Sep 2016, Felipe Balbi wrote: > > Hi, > > Jarkko Sakkinen writes: > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > >> > >> > >> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > >> > >> > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Jarkko Sakkinen
On Mon, Sep 12, 2016 at 03:52:08PM +0200, Julia Lawall wrote: > > > On Mon, 12 Sep 2016, Felipe Balbi wrote: > > > > > Hi, > > > > Jarkko Sakkinen writes: > > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > >> > > >> > > >> On Sun, 11 Sep

Re: [PATCH 0/9] rsi: code clean-up

2016-09-12 Thread Joe Perches
On Mon, 2016-09-12 at 15:53 +0300, Kalle Valo wrote: > Also try to get more familiar how development is done, here are few > links: > > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches > https://www.kernel.org/doc/Documentation/SubmittingPatches >

[PATCH] staging: squash lines for simple wrapper functions

2016-09-12 Thread Masahiro Yamada
Remove unneeded variables and assignments. While we are here, clean up the following as well: - refactor rtl8723a_get_bcn_valid() a bit - remove unneeded casts in sii164Get{Vendor,Device}ID() Signed-off-by: Masahiro Yamada ---

Re: [PATCH] staging: squash lines for simple wrapper functions

2016-09-12 Thread Jes Sorensen
Masahiro Yamada writes: > Remove unneeded variables and assignments. > > While we are here, clean up the following as well: > - refactor rtl8723a_get_bcn_valid() a bit > - remove unneeded casts in sii164Get{Vendor,Device}ID() > > Signed-off-by: Masahiro Yamada

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Jarkko Sakkinen
On Mon, Sep 12, 2016 at 04:43:58PM +0300, Felipe Balbi wrote: > > Hi, > > Jarkko Sakkinen writes: > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > >> > >> > >> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > >> > >> > On Sun, Sep 11, 2016 at

Re: [PATCH] mac80211: allow driver to handle packet-loss mechanism

2016-09-12 Thread Rajkumar Manoharan
On 2016-09-12 18:14, Johannes Berg wrote: On Tue, 2016-09-06 at 12:26 +0530, Rajkumar Manoharan wrote: mac80211 keeps track of missing acks and triggers CQM packet-loss mechanism whenever consecutive msdu failure reaches threshold limit (STA_LOST_PKT_THRESHOLD). Drivers like ath10k offlaoded

Re: Support of rtl8723bs

2016-09-12 Thread Jes Sorensen
Bastien Nocera writes: > On Mon, 2016-09-12 at 17:48 +0200, Hanno Zulla wrote: >> Hi, >> >> > Longer term I think it makes sense to add SDIO support to rtl8xxxu. >> > The >> > differences between the USB version and the SDIO version are rather >> > small. >> >> >> This is

Re: [PATCH] cfg80211: cap 20MHz VHT bitrate at MCS 8

2016-09-12 Thread Ben Greear
On 09/11/2016 11:43 PM, Johannes Berg wrote: On Wed, 2016-09-07 at 18:20 +, Pedersen, Thomas wrote: On 09/06/2016 12:07 PM, Ben Greear wrote: On 09/06/2016 12:00 PM, Thomas Pedersen wrote: Some drivers (ath10k) report MCS 9 @ 20MHz, which technically isn't allowed. To get more

Re: [PATCH 2/3] ath10k: Grab rcu_read_lock before the txqs spinlock.

2016-09-12 Thread Ben Greear
On 09/11/2016 11:41 PM, Johannes Berg wrote: - rcu_read_unlock(); spin_unlock_bh(>txqs_lock); + rcu_read_unlock(); I'm no RCU expert but this isn't making any sense. Maybe it changes timings on your kernel so that it hides the real problem? I'm not sure this fixed

Re: [PATCH 00/26] constify local structures

2016-09-12 Thread Julia Lawall
On Mon, 12 Sep 2016, Jarkko Sakkinen wrote: > On Mon, Sep 12, 2016 at 04:43:58PM +0300, Felipe Balbi wrote: > > > > Hi, > > > > Jarkko Sakkinen writes: > > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > >> > > >> > > >> On Sun, 11 Sep 2016,

Re: [PATCH 00/15] drivers: net: use IS_ENABLED() instead of checking for built-in or module

2016-09-12 Thread David Miller
From: Javier Martinez Canillas Date: Mon, 12 Sep 2016 10:03:31 -0400 > This trivial series is similar to [0] for net/ that you already merged, but > for drivers/net. The patches replaces the open coding to check for a Kconfig > symbol being built-in or module, with

Re: [PATCH] ath10k: Fix rfc1042 header retrieval in QCA4019 with eth decap mode

2016-09-12 Thread Thiagarajan, Vasanthakumar
On Monday 12 September 2016 09:01 PM, Valo, Kalle wrote: > Vasanthakumar Thiagarajan writes: > >> Chipset from QCA99X0 onwards (QCA99X0, QCA9984, QCA4019 & future) >> rx_hdr_status is not padded to align in 4-byte boundary. Define a >> new hw_params field to handle

[PATCH 15/15] iwlegacy: use IS_ENABLED() instead of checking for built-in or module

2016-09-12 Thread Javier Martinez Canillas
The IS_ENABLED() macro checks if a Kconfig symbol has been enabled either built-in or as a module, use that macro instead of open coding the same. Using the macro makes the code more readable by helping abstract away some of the Kconfig built-in and module enable details. Signed-off-by: Javier

Re: Support of rtl8723bs

2016-09-12 Thread Larry Finger
On 09/12/2016 09:47 AM, Jes Sorensen wrote: Bastien Nocera writes: On Mon, 2016-09-12 at 11:20 +0200, Hanno Zulla wrote: Hi Jes, hello linux-wireless, it as a guinea pig to test on my hardware. The newer vendor driver crashed on Ubuntu' 4.4 sources and doesn't compile yet

[PATCH 00/15] drivers: net: use IS_ENABLED() instead of checking for built-in or module

2016-09-12 Thread Javier Martinez Canillas
Hello David, This trivial series is similar to [0] for net/ that you already merged, but for drivers/net. The patches replaces the open coding to check for a Kconfig symbol being built-in or module, with IS_ENABLED() macro that does the same. Using the macro makes the code more readable by

[PATCH] ath10k: Add missing CONFIG_ATH10K_DEBUGFS check

2016-09-12 Thread Benjamin Berg
The patch "ath10k: allow setting coverage class" was missing a check for CONFIG_ATH10K_DEBUGFS so it would try to use non-existing struct elements in some configurations. Fix this by adding the appropriate ifdef. Signed-off-by: Benjamin Berg --- Sorry, so turns out

Re: Support of rtl8723bs

2016-09-12 Thread Jes Sorensen
Bastien Nocera writes: > On Mon, 2016-09-12 at 11:20 +0200, Hanno Zulla wrote: >> Hi Jes, >> hello linux-wireless, >> it as a guinea pig to test on my hardware. The newer vendor driver >> crashed on Ubuntu' 4.4 sources and doesn't compile yet on Ubuntu 4.8- >> rc >> sources. >>

Re: Support of rtl8723bs

2016-09-12 Thread Anthony Wong
On 12 September 2016 at 20:23, Bastien Nocera wrote: > > On Mon, 2016-09-12 at 11:20 +0200, Hanno Zulla wrote: > > Hi Jes, > > hello linux-wireless, > > > > I'd like to know more about the current progress of rtl8723bs support > > for Linux. > > > > Googling and reading the

Re: [PATCH] ath10k: Fix rfc1042 header retrieval in QCA4019 with eth decap mode

2016-09-12 Thread Valo, Kalle
Vasanthakumar Thiagarajan writes: > Chipset from QCA99X0 onwards (QCA99X0, QCA9984, QCA4019 & future) > rx_hdr_status is not padded to align in 4-byte boundary. Define a > new hw_params field to handle different alignment behaviour between > different hw. This patch

Re: [PATCH 2/2] ath10k: add platform regulatory domain support

2016-09-12 Thread Valo, Kalle
Bartosz Markowski writes: > This overrides whatever regulatory the device > EEPROM contains and uses what the platform says > instead - in this implementation the ACPI driver. > > In case the hint is not programmed or corrupted (0x) > the device falls back to the

Re: Support of rtl8723bs

2016-09-12 Thread Hanno Zulla
Hi, > Longer term I think it makes sense to add SDIO support to rtl8xxxu. The > differences between the USB version and the SDIO version are rather > small. This is beyond my expertise, sadly. Is there a good blueprint / example of a previous case where a USB driver had SDIO support added that

Re: Support of rtl8723bs

2016-09-12 Thread Bastien Nocera
On Mon, 2016-09-12 at 17:48 +0200, Hanno Zulla wrote: > Hi, > > > Longer term I think it makes sense to add SDIO support to rtl8xxxu. > > The > > differences between the USB version and the SDIO version are rather > > small. > > > This is beyond my expertise, sadly. > > Is there a good