[RFC 01/16] rtl8xxxu: Treat REG_9346CR as an 8-bit wide register

rtl8723au driver treats REG_9346CR register as either 8- or 16-bit wide, while rtl8192cu driver treats it as 8-bit wide, and so does rtl8188eu driver. Use the lowest common denominator strategy. Signed-off-by: Jakub Sitnicki ---

[RFC 00/16] rtl8xxxu: eFUSE parsing for RTL8188EU chips

Hi Jes, I'm posting this patch set to show in what direction the work is going. Hopefully any changes (e.g. renames) which you wouldn't see fit for this driver can be identified early this way. The work is nowhere near the end. These changes only get me past the parse_efuse() step in the

[RFC 09/16] rtl8xxxu: Rename struct struct rtl8723au_tx_power fields

Remove the redundant infix ("tx_power") from field names. Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 92 drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 18 ++--- 2 files changed, 55 insertions(+), 55

[RFC 14/16] rtl8xxxu: Introduce a device agnostic constant for efuse map length

Preparatory step for adding support for RTL8188EU devices. Signed-off-by: Jakub Sitnicki --- Notes: An alternative approach to keeping a buffer that is big enough to hold the efuse map of any of the supported chips would be to allocate the efuse map on-the-go to

[PATCH v2] ath10k: add FW API support to test mode

From: Alan Liu Add WMI-TLV and FW API support in ath10k testmode. Ath10k can get right wmi command format from UTF image to communicate UTF firmware. Signed-off-by: Alan Liu Signed-off-by: Kalle Valo ---

Re: QCA6174 hw2.1 irq_mode=0 broken on 4.2-rc3, working on 4.1.2

Hi there, it's been a while, but I have good news: tried Kernel 4.2.3 from Fedora and it just works! No need to use irq_mode=1 (and I also removed skip_opt), tested with firmware API 5 from ath10k-firmware since a couple of days now. No issue so far. Thank you for the help and the work, and

[PATCH v3] ath10k: add fw_stats support to 10.4 firmware

From: Manikanta Pubbisetty This patch adds support for getting firmware debug stats in 10.4 fw. Signed-off-by: Manikanta Pubbisetty Signed-off-by: Tamizh chelvam --- v3: >Fixed the parsing error for

[PATCH 2/2] rtl8xxxu: Add debugfs for RCR register

Signed-off-by: Bruno Randolf --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 32 1 file changed, 32 insertions(+) diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c index

[PATCH 1/2] rtl8xxxu: Enable monitor mode by handling filters

Monitor mode is enabled by handling the filter flags we get from mac80211 in rtl8xxxu_configure_filter() and writing them to the RCR register. By handling these filters correctly we can also stop setting the BSSID filters in the association event. Also remove initial setting of the RCR register

Re: [PATCH 1/5] ath10k: QCA988X hw2.0: add board.bin

Ben Hutchings writes: > On Sun, 2015-10-18 at 19:23 +0100, Ben Hutchings wrote: >> On Thu, 2015-10-15 at 13:40 +0300, Kalle Valo wrote: >> > In commit 40d14143b787 ("ath10k: add firmware 10.2.4.45 as >> > firmware-4.bin") I >> > forgot to commit board.bin file and that's

[PATCH] ath9k: fix phyerror codes

Some of the ath9k_phyerr enums were wrong from the beginning (and even before). Most of the time the codes were used for counters to be displayed over debugfs, which made this a non-functional issue. Some (e.g. ATH9K_PHYERR_FALSE_RADAR_EXT) are used for radar detection and require the correct

Re: [PATCH 1/5] ath10k: QCA988X hw2.0: add board.bin

Ben Hutchings writes: > On Thu, 2015-10-15 at 13:40 +0300, Kalle Valo wrote: >> In commit 40d14143b787 ("ath10k: add firmware 10.2.4.45 as firmware-4.bin") I >> forgot to commit board.bin file and that's why it was not included in the >> commit even I had added an entry to

[RFC 16/16] rtl8xxxu: rtl8188eu: Implement parse_efuse()

As opposed to the vendor driver (rtl8188eu), extract from efuse only these TX power values that are going to be of use later on. Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 35 +++-

[RFC 11/16] rtl8xxxu: Make efuse content length a parameter

This is a preparatory step for adding support for RTL8188EU chips which have different efuse length. Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 5 - drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 1 + 2 files changed, 5

[RFC 10/16] rtl8xxxu: Rename struct rtl8723au_idx

Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 16 +- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 38 2 files changed, 27 insertions(+), 27 deletions(-) diff --git

[RFC 13/16] rtl8xxxu: Make set_tx_power a chip-type-dependent operation

Preparatory step for adding support for RTL8188EU chips. Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 6 -- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 1 + 2 files changed, 5 insertions(+), 2 deletions(-) diff --git

[RFC 08/16] rtl8xxxu: Extract TX power fields from struct rtl8xxu_priv

Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 95 drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 30 2 files changed, 66 insertions(+), 59 deletions(-) diff --git

[RFC 06/16] rtl8xxxu: rtl8723au: Introduce a pointer to efuse

Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 26 +--- 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c

[RFC 05/16] rtl8xxxu: Unbreak a user-visible string

Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c index

[RFC 12/16] rtl8xxxu: Prepare to have more than one kind of struct *_tx_power

Preparatory step for adding support for RTL8188EU chips. We follow the same approach as with efuse here. Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 6 +++--- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 4 +++- 2 files

[RFC 07/16] rtl8xxxu: rtl8192cu: Introduce a pointer to efuse

Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 29 1 file changed, 15 insertions(+), 14 deletions(-) diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c

[RFC 15/16] rtl8xxxu: rtl8188eu: Add stubbed fileops and chip-specific constants

Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 32 drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 6 - 2 files changed, 37 insertions(+), 1 deletion(-) diff --git

[RFC 04/16] rtl8xxxu: Skip disabled efuse words early

Avoid a negative conditional and an extra level of indentation in the bigger part of the loop body. Signed-off-by: Jakub Sitnicki --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 28 +++- 1 file changed, 13 insertions(+), 15 deletions(-) diff

[PATCH] staging: wilc1000: fix the bug on copying bssid

This patch reverts the commit, d79fd35b8c5d927695b48fa35aa586919818cce9. The WID_JOIN_REQ_EXTENDED among WIDs needs two parameters for the request to be sent to the firmware, which are the SA and the BSSID. For this case, both is the same bssid in the handle_connect function. So, it's required to

[PATCH 02/13] staging: wilc1000: wilc1000_wlan_init: use netdev private wilc

From: Glen Lee This patch use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 25 + 1 file

[PATCH 05/13] staging: wilc1000: mac_close: use netdev private wilc instead of g_linux_wlan

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 16 +--- 1 file changed, 9

[PATCH 03/13] staging: wilc1000: wlan_init_locks: change parameter and use wilc

From: Glen Lee This patch changes function parameter type wilc with net_device and use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho ---

[PATCH 06/13] staging: wilc1000: chage_virtual_intf: use netdev private wilc

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 85 --- 1 file

[PATCH 04/13] staging: wilc1000: mac_open: use netdev private wilc

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 20 1 file changed, 12

[PATCH 07/13] staging: wilc1000: wilc1000_wlan_deinit: change argument and use wilc

From: Glen Lee This patch changes function argument wilc with net_device and use nic->wilc instead of global variable g_linux_wlan. The null check codes should be placed before it is used so move it. Signed-off-by: Glen Lee Signed-off-by: Tony Cho

[PATCH 01/13] staging: wilc1000: add wilc to netdev private data structure

From: Glen Lee This patch add wilc to struct perInterface_wlan_t which is netdev private data to access wilc via netdev_priv function. Assign wilc to nic->wilc. The global variable g_linux_wlan will be replaced with netdev private data member wilc step by step.

[PATCH 3/3] mac80211: move station statistics into sub-structs

From: Johannes Berg Group station statistics by where they're (mostly) updated (TX, RX and TX-status) and group them into sub-structs of the struct sta_info. Also rename the variables since the grouping now makes it obvious where they belong. This makes it easier to

[PATCH 09/13] staging: wilc1000: add_key: use netdev private wilc instead of g_linux_wlan

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8 ++-- 1 file changed, 6

[PATCH 1/3] mac80211: remove sta->last_ack_signal

From: Johannes Berg This file only feeds a debugfs file that isn't very useful, so remove it. If necessary, we can add other ways to get this information, for example in the NL80211_CMD_PROBE_CLIENT response. Signed-off-by: Johannes Berg ---

[PATCH 08/13] staging: wilc1000: CfgConnectResult: use netdev private wilc

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8 ++-- 1 file changed, 6

[PATCH 13/13] staging: wilc1000: init_irq: change argument and use netdev private wilc

From: Glen Lee This patch changes function argument type wilc with net_device and use netdev private data member wilc instead of p_nic. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 18

[PATCH 12/13] staging: wilc1000: start_ap: use netdev private data wilc

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 6 +- 1 file changed, 5

[PATCH 10/13] staging: wilc1000: del_key: use netdev private wilc instead of g_linux_wlan

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 6 +- 1 file changed, 5

[PATCH 2/3] mac80211: move beacon_loss_count into ifmgd

From: Johannes Berg There's little point in keeping (and even sending to userspace) the beacon_loss_count value per station, since it can only apply to the AP on a managed-mode connection. Move the value to ifmgd, advertise it only in managed mode, and remove it from

[PATCH 11/13] staging: wilc1000: wilc_mgmt_frame_register: use netdev private wilc

From: Glen Lee Use netdev private data member wilc instead of global variable g_linux_wlan. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 7 +++ 1 file changed, 3

[PATCH] ath10k: consolidate if statements in ath10k_wmi_event_mgmt_rx

From: Manikanta Pubbisetty This patch combines several if statements into a single if condition in WMI management rx handler. Found during code review. Signed-off-by: Manikanta Pubbisetty --- drivers/net/wireless/ath/ath10k/wmi.c

[PATCHv2] ath10k: consolidate if statements in ath10k_wmi_event_mgmt_rx

From: Manikanta Pubbisetty This patch replaces mutliple if conditional checks with a single if condition in WMI management rx handler. Found during code review. Signed-off-by: Manikanta Pubbisetty --- v2: 1) Changed commit

Re: [PATCH v2] ath10k: add FW API support to test mode

On 10/20/2015 04:55 PM, Kalle Valo wrote: From: Alan Liu Add WMI-TLV and FW API support in ath10k testmode. Ath10k can get right wmi command format from UTF image to communicate UTF firmware. Signed-off-by: Alan Liu Signed-off-by: Kalle

[PATCH-v2 1/2] mac80211: Take bitrates into account when building IEs.

From: Ben Greear If a user restricts the rateset for some reason, then the probe requests should not advertise rates that are not selected by the user. To implement this, we save the requested bitrates at the mac80211 level and take it into account when building the

[PATCH-v2 2/2] mac80211: ensure association req uses configured ratemask.

From: Ben Greear When sending the association request, pay attention to the legacy rates configured by the user, and do not advertise support for any that are not configured. This makes the assoc request packet look more correct when making modern hardware act like a /b

[PATCH v2] ath10k: add notice.txt references to WHENCE

This is to make sure that the notice files get automatically included. Signed-off-by: Kalle Valo --- WHENCE |5 + 1 file changed, 5 insertions(+) diff --git a/WHENCE b/WHENCE index 9b7e8e8526aa..ded2e7ad161b 100755 --- a/WHENCE +++ b/WHENCE @@ -2393,17 +2393,22

Re: [PATCH] mwifiex: toggle carrier state in start_ap/stop_ap.

Hello Marty, On Mon, Oct 19, 2015 at 11:49 AM, Marty Faltesek wrote: > > In uap mode the carrier is not enabled until after the first STA joins. > The carrier triggers the bridge to start its state machine, and if STP > is enabled, it takes 4 seconds as it transitions from

[PATCH 17/19] staging/wilc1000: pass hif operations through initialization

The wilc1000_hif_spi and wilc1000_hif_sdio structures are part of the bus specific code, and the generic code should have no knowledge of their addresses. This changes the code to reference them only from the bus specific initialization code, which we can then use to split up the driver into

[PATCH 11/19] staging/wilc1000: move wilc1000_ops to drivers

The two instances of struct wilc1000_ops reference functions that are defined in just one file each. By moving the operations there, we can mark all those functions as 'static' and avoid a number of #ifdefs. Signed-off-by: Arnd Bergmann --- drivers/staging/wilc1000/linux_wlan.c

[PATCH 15/19] staging/wilc1000: turn enable_irq/disable_irq into callbacks

As a preparation for turning the SDIO side of wilc1000 into a separate module, this removes the last direct caller from the core module into the sdio specific portion. All calls to wilc1000_sdio_enable_interrupt() and wilc1000_sdio_disable_interrupt() now go through a function pointer in

[PATCH 04/19] staging/wilc1000: move extern declarations to headers

'extern' declarations belong into a header file rather than a .c file, to ensure that the definition matches the declaration. This moves all declarations into a header file that seems most appropriate for it. Signed-off-by: Arnd Bergmann ---

[PATCH 09/19] staging/wilc1000: move init/exit functions to driver files

The driver interfaces are in linux_wlan_sdio.c and linux_wlan_spi.c, so this is where the init and exit functions should be. Splitting this up enables further cleanups, including eventually allowing both modules to be built together. Signed-off-by: Arnd Bergmann ---

[PATCH 19/19] [RFC] staging/wilc1000: use more regular probing

So far, my patches tried to do equivalent conversions of the existing code. This one goes beyond that by restructuring how the devices get probed. In particular, the spi driver no longer creates the netdev until the device is probed, and I've removed the global wilc1000_sdio_func and wilc_spi_dev

[PATCH 02/19] staging/wilc1000: make symbols static if possible

All symbols that are only referenced in the file that defines them can be declared 'static' to avoid namespace pollution, to produce better object code, and to make the source more readable. Signed-off-by: Arnd Bergmann --- drivers/staging/wilc1000/Makefile | 3

[PATCH 08/19] staging/wilc1000: move wilc_wlan_inp_t into struct wilc

wilc_wlan_inp_t is an unnecessary indirection and requires linux_wlan.c to have knowledge of the specific sdio and spi front-ends. This removes the structure and places the members with a constant 'struct wilc1000_ops' pointer inside of 'struct wilc'. Signed-off-by: Arnd Bergmann

[PATCH 05/19] staging/wilc1000: use NO_SECURITY instead of NO_ENCRYPT

The linux_wlan.c file uses a set of enums from wilc_wlan_if.h, with the exception of the NO_ENCRYPT that comes from wilc_wfi_cfgoperations.h. The two sets of enums clearly have the same intention but are defined a bit different. To prepare to clean up the ones in wilc_wfi_cfgoperations.h, this

[PATCH 12/19] staging/wilc1000: use device pointer for phy creation

wilc_create_wiphy tries to get a pointer to a device from the global wilc1000_sdio_func variable. This is a layering violation and we can use the wilc1000_dev->dev pointer instead. Signed-off-by: Arnd Bergmann --- drivers/staging/wilc1000/linux_wlan.c | 2 +-

[PATCH 13/19] staging/wilc1000: move COMPLEMENT_BOOT code to linux_wlan_sdio.c

The COMPLEMENT_BOOT code is used only for SDIO and is aware of details of the SDIO implementation that don't belong into the common linux_wlan.c file. This moves the functions to linux_wlan_sdio.c to get them out of the way. Signed-off-by: Arnd Bergmann ---

[PATCH 16/19] staging/wilc1000: remove WILC_SDIO/WILC_SPI macros

The last remaining user of WILC_SDIO macro checks for the correct time to wait in an interrupt for the PLL to settle. We can replace this with a runtime check and remove both WILC_SDIO and WILC_SPI, as we no longer need conditional compilation based on the hardware type. Signed-off-by: Arnd

[PATCH 06/19] staging/wilc1000: avoid static definitions in header

The wilc_wfi_cfgoperations.h header defines the ieee80211_txrx_stypes and cipher_suites variables that are only used in wilc_wfi_cfgoperations.c and should not be shared in a header file. This moves over all that data into the .c file, and also moves all the macro definitions from the file that

Re: [PATCH] staging/wilc: fix Kconfig dependencies, second try

On Monday 19 October 2015 11:53:40 Arnd Bergmann wrote: > On Monday 19 October 2015 11:39:31 Arnd Bergmann wrote: > > On Monday 19 October 2015 16:54:23 Tony Cho wrote: > > > Hi Arnd Bergmann, > > > > > > When I apply this patch, I cannot make WILC1000 module (wilc1000.ko) > > > because

[PATCH 10/19] staging/wilc1000: unify device pointer

struct wilc has two pointers to store the device, one for sdio_func and one for spi_device. By changing the pointer to a 'struct device', we can simplify the logic and avoid a few #ifdefs. Signed-off-by: Arnd Bergmann --- drivers/staging/wilc1000/linux_wlan.c | 29

[PATCH 14/19] staging/wilc1000: get rid of WILC_SDIO_IRQ_GPIO

Whether the SDIO function uses an internal or external interrupt should not be a compiletime decision but be determined at runtime. This changes the code to pass a GPIO number from the init code as early as possible, and leaves just one #ifdef WILC_SDIO_IRQ_GPIO to preserve the previous behavior.

[PATCH 07/19] staging/wilc1000: remove linux_wlan_{device_power,device_detection}

The driver provides an interface for custom power management and detection that is meant to be filled by people customizing the driver. The default implementation of this is empty, and we don't actually want people to have to modify the source code. If anybody needs this, they need to describe

[PATCH 00/19] staging/wilc1000 cleanups

After I screwed up two or three times trying to get the dependencies with the SPI and SDIO backends right, I ended up taking the time to do the real fix and make both coexist. This is a series of patches that address various issues with the driver, but is mainly targetted at splitting out the spi

[PATCH] ath10k: add notice.txt references to WHENCE

This is to make sure that the notice files get automatically included. Signed-off-by: Kalle Valo --- WHENCE |5 + 1 file changed, 5 insertions(+) diff --git a/WHENCE b/WHENCE index 9b7e8e8526aa..5cc01414e915 100755 --- a/WHENCE +++ b/WHENCE @@ -2393,17 +2393,22

Re: [PATCH] ath10k: add notice.txt references to WHENCE

On Tue, 2015-10-20 at 17:53 +0300, Kalle Valo wrote: > This is to make sure that the notice files get automatically included. > > Signed-off-by: Kalle Valo While it isn't actually specified here, I expect the File: lines to refer to the firmware itself.  Please put these

Re: [PATCH] ath10k: add notice.txt references to WHENCE

Ben Hutchings writes: > On Tue, 2015-10-20 at 17:53 +0300, Kalle Valo wrote: >> This is to make sure that the notice files get automatically included. >> >> Signed-off-by: Kalle Valo > > While it isn't actually specified here, I expect the File:

Re: [RFC 01/16] rtl8xxxu: Treat REG_9346CR as an 8-bit wide register

Jakub Sitnicki writes: > rtl8723au driver treats REG_9346CR register as either 8- or 16-bit wide, > while rtl8192cu driver treats it as 8-bit wide, and so does rtl8188eu > driver. Use the lowest common denominator strategy. > > Signed-off-by: Jakub Sitnicki

Re: [RFC 02/16] rtl8xxxu: Use REG_EFUSE_TEST register only on multifunctional devices

Jakub Sitnicki writes: > rtl8192cu driver doesn't read/write the REG_EFUSE_TEST register. > Neither does the rtl8188eu driver. Do it only for RTL8192CU-based > devices. > > Signed-off-by: Jakub Sitnicki > --- >

Re: [RFC 00/16] rtl8xxxu: eFUSE parsing for RTL8188EU chips

Jakub Sitnicki writes: > Hi Jes, > > I'm posting this patch set to show in what direction the work is > going. Hopefully any changes (e.g. renames) which you wouldn't see > fit for this driver can be identified early this way. > > The work is nowhere near the end. These

Re: [RFC 04/16] rtl8xxxu: Skip disabled efuse words early

Jakub Sitnicki writes: > Avoid a negative conditional and an extra level of indentation in the > bigger part of the loop body. > > Signed-off-by: Jakub Sitnicki > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 28 > +++- >

Re: [RFC 03/16] rtl8xxxu: Don't check for illegal offset when reading from efuse

Jakub Sitnicki writes: > It is enough to check for either illegal offset or illegal map address > because map address is a value derived from an offset: > > map_addr = offset * 8 > EFUSE_MAP_LEN_8723A = EFUSE_MAX_SECTION_8723A * 8 > > Leave just the check for an illegal

Re: [RFC 08/16] rtl8xxxu: Extract TX power fields from struct rtl8xxu_priv

Jakub Sitnicki writes: > Signed-off-by: Jakub Sitnicki > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 95 > > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 30 > 2 files changed, 66 insertions(+), 59

Re: [RFC 16/16] rtl8xxxu: rtl8188eu: Implement parse_efuse()

Jakub Sitnicki writes: > As opposed to the vendor driver (rtl8188eu), extract from efuse only > these TX power values that are going to be of use later on. > > Signed-off-by: Jakub Sitnicki > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 35

Re: [RFC 05/16] rtl8xxxu: Unbreak a user-visible string

Jakub Sitnicki writes: > Signed-off-by: Jakub Sitnicki > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c >

Re: [RFC 06/16] rtl8xxxu: rtl8723au: Introduce a pointer to efuse

Jakub Sitnicki writes: > Signed-off-by: Jakub Sitnicki > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 26 > +--- > 1 file changed, 14 insertions(+), 12 deletions(-) This cleans up the code nicely - so I applied both of

Re: [RFC 14/16] rtl8xxxu: Introduce a device agnostic constant for efuse map length

Jakub Sitnicki writes: > Preparatory step for adding support for RTL8188EU devices. > > Signed-off-by: Jakub Sitnicki > --- > > Notes: > An alternative approach to keeping a buffer that is big enough to hold > the efuse map of any of the

Re: [RFC 13/16] rtl8xxxu: Make set_tx_power a chip-type-dependent operation

Jakub Sitnicki writes: > Preparatory step for adding support for RTL8188EU chips. > > Signed-off-by: Jakub Sitnicki > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 6 -- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 1 + > 2 files

RE: [PATCH] mwifiex: toggle carrier state in start_ap/stop_ap.

Hi Avinash > From: linux-wireless-ow...@vger.kernel.org [mailto:linux-wireless- > ow...@vger.kernel.org] On Behalf Of Avinash Patil > Sent: Wednesday, October 21, 2015 1:50 AM > To: Marty Faltesek > Cc: linux-wireless@vger.kernel.org; Amitkumar Karwar; Nishant > Sarmukadam; Denton Gentry; Avery

Re: [RFC 10/16] rtl8xxxu: Rename struct rtl8723au_idx

Jakub Sitnicki writes: > Signed-off-by: Jakub Sitnicki > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 16 +- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 38 > > 2 files changed, 27 insertions(+), 27

Re: [RFC 11/16] rtl8xxxu: Make efuse content length a parameter

Jakub Sitnicki writes: > This is a preparatory step for adding support for RTL8188EU chips > which have different efuse length. The way the EFUSES are stored, it doesn't matter that you try to read more data as the EFUSE will stop. I already added support for reading the