Re: odd msg using kvalo firmware-5.bin w/ KillerAC 1535(QCA6174)

2017-04-19 Thread Mohammed Shafi Shajakhan (Mohammed Shafi)
Hi, my quick analysis suggests this belongs to 'WMI_TLV_GRP_STATS' WMI_TLV_STATS_EXT_EVENTID = WMI_TLV_EV(WMI_TLV_GRP_STATS), equals to 90112 (0x16000) WMI_TLV_IFACE_LINK_STATS_EVENTID, 90113 WMI_TLV_PEER_LINK_STATS_EVENTID, 90114 WMI_TLV_RADIO_LINK_STATS_EVENTID, 90015

[PATCH] ath: Fix updating radar flags for coutry code India

2017-04-12 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> As per latest regulatory update for India, channel 52, 56, 60, 64 is no longer restricted to DFS. Enabling DFS/no infra flags in driver results in applying all DFS related restrictions (like doing CAC etc before this channel

Re: [PATCH] ath10k: Fix spectral scan for QCA99X0 family of chipsets

2017-04-10 Thread Mohammed Shafi Shajakhan (Mohammed Shafi)
/ regards, shafi On 10 April 2017 at 07:26, Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> wrote: > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > spectral_bin length (number of bins per fft sample) is usually > a value where (2^n = value), n is an inte

[PATCH v2] ath10k: Fix spectral scan for QCA99X0 family of chipsets

2017-04-10 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> spectral_bin length (number of bins per fft sample) is usually a value where (2^n = value), n is an integer. All of the QCA99X0 family of chipsets seems to report a spectral_bin length of 2^n + 'm' bytes, where m = 4, 12

[PATCH] ath10k: Fix spectral scan for QCA99X0 family of chipsets

2017-04-10 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> spectral_bin length (number of bins per fft sample) is usually a value where (2^n = value), n is an integer. All of the QCA99X0 family of chipsets seems to report a spectral_bin length of 2^n + 'm' bytes, where m = 4, 12

[PATCH] ath10k: Fix compile time sanity check for CE4 buffer size

2017-04-04 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> In 'ath10k_ce_alloc_pipe' the compile time sanity check to ensure that there is sufficient buffers in CE4 for HTT Tx MSDU descriptors, but this did not take into account of the case with 'peer flow control' enabled, fix this. Cc:

[PATCH] ath10k: Remove obselete Copy Engine comments

2017-03-31 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Remove obselete Copy Engine comments referring to the function ath10k_ce_sendlist_send as this function was removed long time back by the commit 2e761b5a5222 ("ath10k: remove ce_sendlist_send"). Signed-off-by: Mohammed

[PATCH] ath10k: Enable a HTC debug message during insufficient tx credits

2017-03-31 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Add an ath10k HTC debug message when insufficient tx credits are available to send the WMI commands. This is very useful in debugging issues like 'tx credit starvation' that could possibly happen with multiclient setup with co

[PATCH] ath10k: cancel coverage class work during stop and restart

2017-03-31 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> It seems set_coverage_class_work is not cancelled anywhere, though I could not find a crash/warning with this existing design, its safer to cancel it during stop() and also before restarting the hardware. Signed-off-by: Mohammed

[PATCH] ath10k: Fix NAPI enable/disable symmetry for AHB interface

2017-03-20 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Move NAPI enable to 'ath10k_ahb_hif_start' from 'ath10k_ahb_hif_power_up'. This is to maintain the symmetry of calling napi_enable() from ath10k_ahb_hif_start() so that it matches with napi_disable() being called from ath10k_pci_hi

Re: ath10k - spectral scan - and - channel change

2017-03-15 Thread Mohammed Shafi Shajakhan
eason I am trying to do both (control spectral_scan_ctl and change > channels) is because it works fine in ath9k monitor mode.. > > [shafi] ok, we can check that , i think for ath9k the complete support is in driver (no firmware) > > > - Original Message - > From: Mohammed Shafi

Re: ath10k - spectral scan - and - channel change

2017-03-14 Thread Mohammed Shafi Shajakhan
> Thank you for looking into this.. I'm really sorry for the confusion, my > second email reversed what the problem really is. Now clarified **below**: > > Is it possible to control spectral_scan_ctl whilst in **Monitor mode**? > my result: > > root at OpenWrt:~# echo disable > >

[PATCH] ath10k: Fix typo in wmi header file

2017-03-14 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Fix typo(spelling mistake) in wmi.h Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> --- drivers/net/wireless/ath/ath10k/wmi.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/

[PATCH] ath10k: Fix fetching channel during potential radar detection

2017-03-08 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Fetch target operating channel during potential radar detection when the interface is just brought up, but no channel is assigned from userspace. In this scenario rx_channel may not be having a valid pointer hence fetch the

Re: ath10k QCA988x firmware not supporting monitor mode

2017-02-28 Thread Mohammed Shafi Shajakhan
On Wed, Mar 01, 2017 at 10:38:48AM +0530, KAVITA MATHUR wrote: > > > Hi! > > We are using Atheros QCA9880 mPCIe module with ath10k drivers and firmware. > > Kernel: 3.12 > Backports: 4.4 > Firmware:10.2.4.70.59-2 [shafi] worth trying updated firmware

Re: Recent driver changes destabilized QCA9377 connection quality

2017-02-26 Thread Mohammed Shafi Shajakhan
Hi, On Sun, Feb 26, 2017 at 04:40:42PM +0100, Tobias Predel wrote: > Hello, > > as I still encounter some stability issues with my beloved QCA9377 chipset on > Linux, I decided to recompile the ath10k kernel module according to [1] in > order to enable debugging and tracing (see [2]) instead

[PATCH] ath10k: Disallow DFS simulation if DFS channel is not enabled

2017-02-22 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> If DFS is not enabled in hostapd (ieee80211h=0) DFS channels shall not be available for use even though the hardware may have the capability to support DFS. With this configuration (DFS disabled in hostapd) trying to bring up

[PATCH v2] ath10k: Fix a warning during channel switch with multiple vaps

2017-02-17 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Doing a channel switch via hostapd_cli seems to update the new channel context for each VAP's appropriately as below in 'ath10k_mac_update_vif_chan', hence we can safely suppress the warning that shows up during this operation an

[PATCH] ath10k: Fix a warning during channel switch with multiple vaps

2017-02-17 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Doing a channel switch via hostapd_cli seems to update the new channel context for each VAP's appropriately as below in 'ath10k_mac_update_vif_chan', hence we can safely suppress the warning that shows up during this operation an

Re: [PATCH v3] ath10k: Fix crash during rmmod when probe firmware fails

2017-02-06 Thread Mohammed Shafi Shajakhan
mm.com> writes: > > > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> writes: > > > >> From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > >> > >> This fixes the below crash when ath10k probe firmware fails, > >> NAPI pollin

Re: [PATCH v3] ath10k: Fix crash during rmmod when probe firmware fails

2017-02-05 Thread Mohammed Shafi Shajakhan
Hi Kalle, sorry for the delay On Wed, Jan 25, 2017 at 01:46:28PM +, Valo, Kalle wrote: > Kalle Valo <kv...@qca.qualcomm.com> writes: > > > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> writes: > > > >> From: Mohammed Shafi Shajakhan <moham

[PATCH] ath10k: Fix per station tx bit rate reporting

2017-01-13 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Not clearing the previous tx bit rate status results in a ambigous tx bit rate reporting to mac80211/cfg80211, for example the previous bit rate status would have been marked as legacy rate , while the current rate would have been

Re: QCA9984 firmware crash

2017-01-10 Thread Mohammed Shafi Shajakhan
On Tue, Jan 10, 2017 at 08:54:34PM +0500, Sergey Stepanov wrote: > Hello, all > > Thanks for helping, finally I got this thing working. [shafi] It would be good if you let us know what did you fix to make this working ? Is the one suggested by Adrian ? > > 2017-01-10 20:44 GMT+05:00 Adrian

Re: QCA9984 firmware crash

2017-01-10 Thread Mohammed Shafi Shajakhan
Hello, Any luck trying the updated one ? https://source.codeaurora.org/quic/qsdk/oss/firmware/ath10k-firmware/plain/ath10k/QCA9984/hw1.0/firmware-5.bin_10.4-3.3-00102 Btw, are you using backports (or) ath.git ? Have you made any private changes ? regards, shafi On Tue, Jan 10, 2017 at

Re: QCA8994 booting failed

2017-01-03 Thread Mohammed Shafi Shajakhan
On Tue, Jan 03, 2017 at 06:03:13PM +0530, K.Mani wrote: > I'm using Annapurna board(Alpine soc V1) on kernel-linux-3.10.20 openWRT. > The board has two QCA9984 wlan boards(2.4 & 5 Ghz). I downloaded the > firmware files from net. > Used, insmod ath10k_core.ko skip_otp=1 also tried without skip_otp

Re: [1/2] ath10k: add accounting for the extended peer statistics

2017-01-02 Thread Mohammed Shafi Shajakhan
Hi Christian / Kalle, On Fri, Dec 30, 2016 at 03:35:10PM +0100, Christian Lamparter wrote: > On Thu, Dec 29, 2016 at 3:11 PM, Kalle Valo wrote: > > Christian Lamparter wrote: > >> The 10.4 firmware adds extended peer information to the > >>

Re: [PATCH] ath10k: merge extended peer info data with existing peers info

2016-12-29 Thread Mohammed Shafi Shajakhan
Hi Christian, On Thu, Dec 22, 2016 at 06:58:41PM +0100, Christian Lamparter wrote: > Hello Shafi, > > On Thursday, December 22, 2016 9:18:01 PM CET Mohammed Shafi Shajakhan wrote: > > > On Monday, December 19, 2016 10:19:57 PM CET Mohammed Shafi Shajakhan > > > wr

[PATCH] ath10k: Enable advertising support for channel 169, 5Ghz

2016-12-27 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Enable advertising support for channel 169, 5Ghz so that based on the regulatory domain(country code) this channel shall be active for use. For example in countries like India this channel shall be available for use with latest regu

[PATCH] ath10k: Remove passing unused argument for ath10k_mac_tx

2016-12-27 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> 'ath10k_mac_tx' does not seems to use the per station table entry pointer 'sta' (struct ieee80211_sta), hence remove passing this unused argument Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> ---

Re: [PATCH] ath10k: merge extended peer info data with existing peers info

2016-12-22 Thread Mohammed Shafi Shajakhan
Hi Christian, once again sorry for the delay > > On Monday, December 19, 2016 10:19:57 PM CET Mohammed Shafi Shajakhan wrote: > > On Sat, Dec 17, 2016 at 06:46:34PM +0100, Christian Lamparter wrote: > > > The 10.4 firmware adds extended peer information to the > &

[PATCH v2] ath10k: Fix crash during rmmod when probe firmware fails

2016-12-20 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> This fixes the below crash when ath10k probe firmware fails, NAPI polling tries to access a rx ring resource which was never allocated, fix this by disabling NAPI right away once the probe firmware fails by calling 'ath10k_hif_stop

Re: [PATCH] ath10k: merge extended peer info data with existing peers info

2016-12-19 Thread Mohammed Shafi Shajakhan
On Mon, Dec 19, 2016 at 10:19:57PM +0530, Mohammed Shafi Shajakhan wrote: > Hi Christian, > > On Sat, Dec 17, 2016 at 06:46:34PM +0100, Christian Lamparter wrote: > > The 10.4 firmware adds extended peer information to the > > firmware's statistics payload. This addition

Re: [PATCH] ath10k: merge extended peer info data with existing peers info

2016-12-19 Thread Mohammed Shafi Shajakhan
the extended peer statistics" [0] > > Mohammed Shafi Shajakhan commented that the extended peer statistics > lists are of little use:"... there is not much use in appending > the extended peer stats (which gets periodically updated) to the > linked list '>debug.

Re: [PATCH 1/2] ath10k: add accounting for the extended peer statistics

2016-12-15 Thread Mohammed Shafi Shajakhan
Hi Christian, > On Thursday, December 15, 2016 10:13:39 PM CET Mohammed Shafi Shajakhan wrote: > > I am also thinking, as of now there is not much use in appending > > the extended peer stats (which gets periodically ) to the linked list > > '>debug.fw_stats.peers_extd)

Re: [PATCH 1/2] ath10k: add accounting for the extended peer statistics

2016-12-15 Thread Mohammed Shafi Shajakhan
xtd, >debug.fw_stats.peers_extd); since rx_duration is getting updated periodically to the per sta information. Kindly let me know your thoughts about this. regards, shafi On Thu, Dec 15, 2016 at 09:56:59PM +0530, Mohammed Shafi Shajakhan wrote: > Hello Christian, > > On Wed, Dec 14

Re: boot kernel panic with QCA6174 and kernel 4.8

2016-12-07 Thread Mohammed Shafi Shajakhan
Hi, On Wed, Dec 07, 2016 at 09:40:29AM +0100, Alexandre Maloteaux wrote: > Hi > Since recent kernel upgrade on Arch linux ; im facing boot kernel > panics with the ath10k_pci and QCA6174 802.11ac Wireless Network Adapter > on an Eurocom P5 Pro laptop. > > This is the main call stack dump : >

Re: [PATCH 1/2] ath10k: add accounting for the extended peer statistics

2016-12-06 Thread Mohammed Shafi Shajakhan
Hi Christian, On Mon, Dec 05, 2016 at 10:52:45PM +0100, Christian Lamparter wrote: > The 10.4 firmware adds extended peer information to the > firmware's statistics payload. This additional info is > stored as a separate data field and the elements are > stored in their own "peers_extd" list. >

[PATCH] ath10k: Avoid potential page alloc BUG_ON in tx free path

2016-12-06 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> 'ath10k_htt_tx_free_cont_txbuf' and 'ath10k_htt_tx_free_cont_frag_desc' have NULL pointer checks to avoid crash if they are called twice but this is as of now not sufficient as these pointers are not assigned to NULL once the cont

Re: ath10k firmware crashes in mesh mode on QCA9880

2016-12-03 Thread Mohammed Shafi Shajakhan
https://patchwork.kernel.org/patch/9437519/ (sorry missed this in the previous thread) On Sat, Dec 03, 2016 at 06:13:58PM +0530, Mohammed Shafi Shajakhan wrote: > Hi Benjamin, > > On Fri, Dec 02, 2016 at 05:28:02PM -0800, Benjamin Morgan wrote: > > Just tried 10.2.4.70.58 fi

Re: ath10k firmware crashes in mesh mode on QCA9880

2016-12-03 Thread Mohammed Shafi Shajakhan
and ath.git Also worth trying this fix in ath10k (once the next firmware version) is available. I will keep you posted. thank you ! > > ~Benjamin > > On 11/30/2016 11:09 PM, Mohammed Shafi Shajakhan wrote: > >Hi, > > > >On Tue, Nov 29, 2016 at 11:22:12AM -0800, Benjami

Re: [PATCH v2] ath10k: Fix soft lockup during firmware crash/hw-restart

2016-12-01 Thread Mohammed Shafi Shajakhan
On Thu, Dec 01, 2016 at 10:35:38AM +, Valo, Kalle wrote: > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> writes: > > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > During firmware crash (or) user requested manual restart >

Re: ath10k firmware crashes in mesh mode on QCA9880

2016-11-30 Thread Mohammed Shafi Shajakhan
Hi, On Tue, Nov 29, 2016 at 11:22:12AM -0800, Benjamin Morgan wrote: > When we try to transmit traffic (ping) between two meshed ath10k > devices running latest lede we keep experiencing ath10k firmware > crashes. This seems to only happen when running in 802.11n/ac mode > but not in 802.11a/g

Re: [PATCH] ath10k: Fix Tx DMA alloc failure during continuous wifi down/up

2016-11-30 Thread Mohammed Shafi Shajakhan
egards, shafi > > > > On 30 November 2016 at 01:50, Mohammed Shafi Shajakhan > <moham...@qti.qualcomm.com> wrote: > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > With maximum number of vap's configured in a two radio supporte

Re: ath10k: Fix soft lockup during firmware crash/hw-restart

2016-11-30 Thread Mohammed Shafi Shajakhan
thanks a lot Ben. > > On 11/29/2016 06:46 AM, Mohammed Shafi Shajakhan wrote: > >From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > >During firmware crash (or) user requested manual restart > >the system gets into a soft lock up state becaus

[PATCH] ath10k: Fix Tx DMA alloc failure during continuous wifi down/up

2016-11-30 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> With maximum number of vap's configured in a two radio supported systems of ~256 Mb RAM, doing a continuous wifi down/up and intermittent traffic streaming from the connected stations results in failure to allocate contiguous

Re: ath10k: Fix soft lockup during firmware crash/hw-restart

2016-11-29 Thread Mohammed Shafi Shajakhan
AM, Mohammed Shafi Shajakhan wrote: > >From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > >During firmware crash (or) user requested manual restart > >the system gets into a soft lock up state because of the > >below root cause. > > >

[PATCH v2] ath10k: Fix soft lockup during firmware crash/hw-restart

2016-11-29 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> During firmware crash (or) user requested manual restart the system gets into a soft lock up state because of the below root cause. During user requested hardware restart / firmware crash the system goes into a soft lockup

ath10k: Fix soft lockup during firmware crash/hw-restart

2016-11-29 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> During firmware crash (or) user requested manual restart the system gets into a soft lock up state because of the below root cause. During user requested hardware restart / firmware crash the system goes into a soft lockup

Re: Kernel deadlock when ath10k firmware crashes in hacked 4.7+ kernel.

2016-11-21 Thread Mohammed Shafi Shajakhan
Hi Ben, Is this completely sorted out (after changing the locking to the original existing code ) ? spin_lock_bh(>txqs_lock); rcu_read_lock(); regards, shafi On Wed, Nov 09, 2016 at 05:02:09PM -0800, Ben Greear wrote: > I should have looked a bit longer. I was still running the patch >

Re: [BUG] Kernel crashes with ath10k radio and Nexus 5X clients

2016-11-21 Thread Mohammed Shafi Shajakhan
Hi Timo, sorry had I missed the exact kernel crash call trace ? I could see only the warnings, can you please post the call trace of the kernel crash please ? regards, shafi On Sun, Nov 20, 2016 at 02:35:27AM +0100, Timo Sigurdsson wrote: > Hi Adrian, > > sure - here's the bug report on

[PATCH] ath10k: Fix crash during rmmod when probe firmware fails

2016-11-04 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> This fixes the below crash when ath10k probe firmware fails, NAPI polling tries to access a rx ring resource which was never allocated, fix this by disabling NAPI right away once the probe firmware fails BUG: unable to handle kerne

Re: QCA9888 fails to work with latest linux-ath tree, crashes kernel on rmmod.

2016-10-24 Thread Mohammed Shafi Shajakhan
le to re-create the issue easily by removing the firmware files from /lib/firmware/ath10k/QCA988X (or) QCA9888 and doing an rmmod regards, shafi > > On 10/24/2016 06:46 AM, Mohammed Shafi Shajakhan wrote: > >Hi Ben, > > > >I had confirmed that the change is addressing th

Re: QCA9888 fails to work with latest linux-ath tree, crashes kernel on rmmod.

2016-10-24 Thread Mohammed Shafi Shajakhan
Hi Ben, I had confirmed that the change is addressing the crash, thanks for reporting. I will soon send the change properly for everyone to review. regards, shafi On Mon, Oct 24, 2016 at 02:02:26PM +0530, Mohammed Shafi Shajakhan wrote: > Hi Ben, > > I think its best to do 'nap

Re: QCA9888 fails to work with latest linux-ath tree, crashes kernel on rmmod.

2016-10-24 Thread Mohammed Shafi Shajakhan
Hi Ben, I think its best to do 'napi_disable' in 'hif_power_down' because 'hif_power_up does 'napi_enable'. regards, shafi On Thu, Oct 20, 2016 at 11:51:44AM +0530, Mohammed Shafi Shajakhan wrote: > Hi, > > On Wed, Oct 19, 2016 at 02:42:39PM -0700, Ben Greear wrote: > > Code-aur

Re: QCA9888 fails to work with latest linux-ath tree, crashes kernel on rmmod.

2016-10-20 Thread Mohammed Shafi Shajakhan
Hi, On Wed, Oct 19, 2016 at 02:42:39PM -0700, Ben Greear wrote: > Code-aura's board-2.bin for 9888 seems to fix the most obvious errors, > but that does not excuse crashing the kernel on rmmod i guess it could be crashing @ idx = __le32_to_cpu(*htt->rx_ring.alloc_idx.vaddr); in

[PATCH v3 1/2] ath10k: clean up HTT tx buffer allocation and free

2016-10-12 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> cleanup 'ath10k_htt_tx_alloc' by introducing the API's 'ath10k_htt_tx_alloc/free_{cont_txbuf, txdone_fifo} and re-use them whereever needed Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> --- drivers/net/

[PATCH v3 2/2] ath10k: Remove extraneous error message in tx alloc

2016-10-12 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Remove extraneous error message in 'ath10k_htt_tx_alloc_cont_frag_desc' as the caller 'ath10k_htt_tx_alloc' already dumps a proper error message Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> ---

Re: [v2,1/2] ath10k: clean up HTT tx buffer allocation and free

2016-10-12 Thread Mohammed Shafi Shajakhan
Hi Kalle, On Tue, Oct 11, 2016 at 01:36:22PM +0200, Kalle Valo wrote: > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> wrote: > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > cleanup 'ath10k_htt_tx_alloc' by introducing the API

[PATCH v2 2/2] ath10k: Remove extraneous error message in tx alloc

2016-10-06 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Remove extraneous error message in 'ath10k_htt_tx_alloc_cont_frag_desc' as the caller 'ath10k_htt_tx_alloc' already dumps a proper error message Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> --- [v2 reba

[PATCH v2 1/2] ath10k: clean up HTT tx buffer allocation and free

2016-10-06 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> cleanup 'ath10k_htt_tx_alloc' by introducing the API's 'ath10k_htt_tx_alloc/free_{cont_txbuf, txdone_fifo} and re-use them whereever needed Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> --- [v2 reba

Re: [1/2] ath10k: clean up HTT tx buffer allocation and free

2016-10-06 Thread Mohammed Shafi Shajakhan
On Thu, Oct 06, 2016 at 09:31:41AM +0200, Kalle Valo wrote: > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> wrote: > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > cleanup 'ath10k_htt_tx_alloc' by introducing the API's > >

Re: ath10k: Cleanup calling ath10k_htt_rx_h_unchain

2016-10-04 Thread Mohammed Shafi Shajakhan
On Tue, Oct 04, 2016 at 05:30:17PM +0200, Kalle Valo wrote: > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> wrote: > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > 'ath10k_htt_rx_h_unchain' is need to be called

[PATCH 2/2] ath10k: Remove extraneous error message in tx alloc

2016-10-04 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Remove extraneous error message in 'ath10k_htt_tx_alloc_cont_frag_desc' as the caller 'ath10k_htt_tx_alloc' already dumps a proper error message Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> ---

[PATCH] ath10k: Cleanup calling ath10k_htt_rx_h_unchain

2016-09-27 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> 'ath10k_htt_rx_h_unchain' is need to be called only if the return value from 'ath10k_htt_rx_amsdu_pop' is 1('chained msdu's'), this change makes it more explicit and avoids doing a skb_peek, fetching rx descriptor pointer, check

[PATCH v4] ath10k: Cleanup calling ath10k_htt_rx_h_unchain

2016-09-27 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> 'ath10k_htt_rx_h_unchain' is need to be called only if the return value from 'ath10k_htt_rx_amsdu_pop' is 1('chained msdu's'), this change makes it more explicit and avoids doing a skb_peek, fetching rx descriptor pointer, check

Re: [v5] ath10k: Fix broken NULL func data frame status for 10.4

2016-09-08 Thread Mohammed Shafi Shajakhan
On Thu, Sep 08, 2016 at 05:09:03AM +, Valo, Kalle wrote: > "Shajakhan, Mohammed Shafi (Mohammed Shafi)" > writes: > > please revert this patch, Raj suggested this patch introduced a > > regression in sending NULL func frames :( > > > > We will fix the wrong tx

[PATCH] ath10k: Remove unnecessary error code assignment

2016-09-01 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> The error assigned does not seems to be used anywhere, fixes nothing just a small cleanup Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> --- drivers/net/wireless/ath/ath10k/wmi.c | 1 - 1 file changed

[PATCH v5] ath10k: Fix broken NULL func data frame status for 10.4

2016-08-26 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Older firmware with HTT delivers incorrect tx status for null func frames to driver, but this fixed in 10.2 and 10.4 firmware versions. Also this workaround results in reporting of incorrect null func status for 10.

Re: [PATCH 3/3] ath10k: Improve logging message.

2016-08-19 Thread Mohammed Shafi Shajakhan
Hi Ben, On Thu, Aug 18, 2016 at 06:26:35PM -0700, gree...@candelatech.com wrote: > From: Ben Greear > > Helps to know the sta pointer. > > Signed-off-by: Ben Greear > --- > drivers/net/wireless/ath/ath10k/mac.c | 4 ++-- > 1 file changed, 2

[PATCH] ath10k: Remove driver log suggesting QCA9887 support is experimental

2016-07-27 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Support for QCA9887 is no longer experimental and if there are any issues we need to address them Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> --- drivers/net/wireless/ath/ath10k/pci.c | 1 - 1 fil

Re: [PATCH 2/2] ath10k: Fix sending NULL/ Qos NULL data frames for QCA99X0 and later

2016-07-20 Thread Mohammed Shafi Shajakhan
On Wed, Jul 20, 2016 at 02:22:29PM +0200, Michal Kazior wrote: > On 20 July 2016 at 13:43, Shajakhan, Mohammed Shafi (Mohammed Shafi) > wrote: > > Michal, > > > > Can you please let me know if this change is fine or not ? > > I am waiting infinitely for your reply long

[RESEND PATCH v4] ath10k: Fix sending NULL/ Qos NULL data frames for QCA99X0 and later

2016-07-20 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> For chipsets like QCA99X0, IPQ4019 and later we are not getting proper NULL func status (always acked/successs !!) when hostapd does a PROBE_CLIENT via nullfunc frames when the station is powered off abruptly (inactive timer probes

Re: [PATCH 2/2] ath10k: Fix sending NULL/ Qos NULL data frames for QCA99X0 and later

2016-07-08 Thread Mohammed Shafi Shajakhan
Hi Michal / Kalle / Ben, is this patch is good to go (or) should i re-work ? I had replied to Michal's comment of introducing a new firmware feature flag will not address the issue in older firmware / code. Let me know if i had missed something very obvious. On Tue, Jul 05, 2016 at 08:21:01AM

Re: [PATCH v3] ath10k: Fix 10.4 extended peer stats update

2016-06-30 Thread Mohammed Shafi Shajakhan
On Thu, Jun 30, 2016 at 11:09:31AM +, Valo, Kalle wrote: > Mohammed Shafi Shajakhan <moham...@codeaurora.org> writes: > > > Hi Kalle, > > > > On Thu, Jun 30, 2016 at 10:49:02AM +, Valo, Kalle wrote: > >> Kalle Valo <kv...@qca.qualcomm.com> wri

Re: [PATCH v3] ath10k: Fix 10.4 extended peer stats update

2016-06-30 Thread Mohammed Shafi Shajakhan
Hi Kalle, On Thu, Jun 30, 2016 at 10:49:02AM +, Valo, Kalle wrote: > Kalle Valo writes: > > >> @@ -261,6 +263,7 @@ static const struct ath10k_hw_params > >> ath10k_hw_params_list[] = { > >>.board = QCA4019_HW_1_0_BOARD_DATA_FILE, > >>

Re: [PATCH 2/2] ath10k: Fix sending NULL/ Qos NULL data frames for QCA99X0 and later

2016-06-29 Thread Mohammed Shafi Shajakhan
Hello Michal/ Kalle, On Tue, Jun 28, 2016 at 08:48:38AM +0200, Michal Kazior wrote: > On 27 June 2016 at 16:36, Mohammed Shafi Shajakhan > <moham...@codeaurora.org> wrote: > > Hi Michal, > > > > thanks for the review .. > > > > On Mon, Jun 27, 201

Re: [PATCH 2/2] ath10k: Fix sending NULL/ Qos NULL data frames for QCA99X0 and later

2016-06-27 Thread Mohammed Shafi Shajakhan
Hi Michal, thanks for the review .. On Mon, Jun 27, 2016 at 11:27:27AM +0200, Michal Kazior wrote: > On 23 June 2016 at 18:40, Mohammed Shafi Shajakhan > <moham...@qti.qualcomm.com> wrote: > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > >

Re: [PATCH 2/2] ath10k: Fix sending NULL/ Qos NULL data frames for QCA99X0 and later

2016-06-25 Thread Mohammed Shafi Shajakhan
Hello Ben, On Thu, Jun 23, 2016 at 10:12:01AM -0700, Ben Greear wrote: > On 06/23/2016 09:40 AM, Mohammed Shafi Shajakhan wrote: > >From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > >For chipsets like QCA99X0, IPQ4019 and later we are not getting proper

[PATCH 1/2] ath10k: Replace warning with an error message if HTT op version is unset

2016-06-23 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Print an ath10k error message rather a call trace when HTT op version is not found from firmware META data (IE). This should be sufficient to figure out what went wrong. Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qua

Re: Bug with: ath10k: enable parsing per station rx duration for 10.4?

2016-06-21 Thread Mohammed Shafi Shajakhan
On Tue, Jun 21, 2016 at 07:10:28AM -0700, Ben Greear wrote: > > > On 06/20/2016 10:18 PM, Mohammed Shafi Shajakhan wrote: > >Hi Ben, > > > >thanks for reporting ... > > > >On Mon, Jun 20, 2016 at 04:06:52PM -0700, Ben Greear wrote: > >>I'm wo

Re: Bug with: ath10k: enable parsing per station rx duration for 10.4?

2016-06-20 Thread Mohammed Shafi Shajakhan
I somehow screwed up when rebasing [shafi] i will check this and get back to you by end of day. regards, shafi > > > Author: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> 2016-03-16 > 05:43:34 > Committer: Kalle Valo <kv...@qca.qualcomm.com> 2016-04-04

Re: [PATCH] ath10k: Fix a typo in spectral code commments

2016-06-16 Thread Mohammed Shafi Shajakhan
On Thu, Jun 16, 2016 at 11:59:59AM -0400, Bob Copeland wrote: > On Tue, Jun 14, 2016 at 11:03:54AM +0530, Mohammed Shafi Shajakhan wrote: > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > Found this obvious typo while going through the spectra

[PATCH v4 2/2] ath10k: Remove debugfs support for Per STA total rx duration

2016-06-14 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> cfg80211/nl80211 interface changes for per STA total rx-duration and very basic 'ath10k_sta_statistics' mac80211 callback is implemented to extend support for per station statistics from the driver. Also provision in 'iw dev

[PATCH] ath10k: Fix a typo in spectral code commments

2016-06-13 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Found this obvious typo while going through the spectral code design in ath10k Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> --- drivers/net/wireless/ath/ath10k/spectral.c |2 +- 1 file changed,

[PATCH] ath10k: Remove unneccessary WARN_ON_ONCE in rx during ACS

2016-06-09 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> The below warning message seems to hit occasionally with the following combination (IPQ4019 + ACS scan) where we receive packets as a self peer when hostapd does ACS when we bring up AP mode . ath10k has the below fall back mec

Re: [PATCH 0/2] ath10k: Add support for QCA9887

2016-06-07 Thread Mohammed Shafi Shajakhan
Hi Sven, On Tue, Jun 07, 2016 at 06:54:54PM +0200, Sven Eckelmann wrote: > On Tuesday 07 June 2016 20:20:02 Mohammed Shafi Shajakhan wrote: > [...] > > [shafi] it would be helpful if you can share your basic test results (if its > > possible to share). Have you tried to bring u

Re: Bug 119151 - [regression] ath10k no longer authenitcates and freezes system

2016-06-02 Thread Mohammed Shafi Shajakhan
On Thu, Jun 02, 2016 at 08:21:41AM -0700, Ben Greear wrote: > On 06/02/2016 07:24 AM, Valo, Kalle wrote: > >Kalle Valo writes: > > > >>there's a regression in ath10k: > >> > >>https://bugzilla.kernel.org/show_bug.cgi?id=119151 > >> > >>Reporter bisected it to this: > >> >

[PATCH v3 2/2] ath10k: fix CCK h/w rates for QCA99X0 and newer chipsets

2016-06-02 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> CCK hardware table mapping from QCA99X0 onwards got revised. The CCK hardware rate values are in a proper order wrt. to rate and preamble as below ATH10K_HW_RATE_REV2_CCK_LP_1M = 1, ATH10K_HW_RATE_REV2_CCK_LP_

Re: [PATCH v2 1/2] ath10k: remove duplicate and unused rx rate flags

2016-06-02 Thread Mohammed Shafi Shajakhan
On Thu, Jun 02, 2016 at 01:45:15PM +, Valo, Kalle wrote: > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> writes: > > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > All these flags are not used and their use is completely

[PATCH v2 1/2] ath10k: remove duplicate and unused rx rate flags

2016-06-02 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> All these flags are not used and their use is completely covered by 'ath10k_hw_rate_ofdm', 'ath10k_hw_rate_cck', and RX_PPDU_START_RATE_FLAG Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Patchwork-Id: 91

[PATCH v2 2/2] ath10k: fix CCK h/w rates for QCA99X0 and newer chipsets

2016-06-02 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> CCK hardware table mapping from QCA99X0 onwards got revised. The CCK hardware rate values are in a proper order wrt. to rate and preamble as below ATH10K_HW_RATE_REV2_CCK_LP_1M = 1, ATH10K_HW_RATE_REV2_CCK_LP_

[PATCH v3] ath10k: Fix 10.4 extended peer stats update

2016-06-02 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> 10.4 'extended peer stats' will be not be appended with normal peer stats data and they shall be coming in separate chunks. Fix this by maintaining a separate linked list 'extender peer stats' for 10.4 and update rx_duration f

Re: [PATCH v2] ath10k: Fix 10.4 extended peer stats update

2016-06-01 Thread Mohammed Shafi Shajakhan
On Wed, Jun 01, 2016 at 01:00:40PM +, Valo, Kalle wrote: > Mohammed Shafi Shajakhan <moham...@codeaurora.org> writes: > > > Hello, > > > > On Tue, May 31, 2016 at 06:57:52PM +, Valo, Kalle wrote: > >> Mohammed Shafi Shajakhan <moham...@cod

Re: [PATCH v2] ath10k: Fix 10.4 extended peer stats update

2016-06-01 Thread Mohammed Shafi Shajakhan
Hello, On Tue, May 31, 2016 at 06:57:52PM +, Valo, Kalle wrote: > Mohammed Shafi Shajakhan <moham...@codeaurora.org> writes: > > >> > +void ath10k_sta_update_rx_duration(struct ath10k *ar, > >> > +

Re: [PATCH v2] ath10k: Fix 10.4 extended peer stats update

2016-05-31 Thread Mohammed Shafi Shajakhan
Hi Kalle, thanks for the review On Tue, May 31, 2016 at 12:18:49PM +, Valo, Kalle wrote: > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> writes: > > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > 10.4 'extended peer stats'

Re: [PATCH v1] ath10k: Fix 10.4 extended peer stats update

2016-05-31 Thread Mohammed Shafi Shajakhan
Hi Kalle, On Tue, May 31, 2016 at 11:09:38AM +, Valo, Kalle wrote: > "Shajakhan, Mohammed Shafi (Mohammed Shafi)" > writes: > > seems the change got merged in pending branch > >

Re: [PATCH v1] ath10k: Fix 10.4 extended peer stats update

2016-05-27 Thread Mohammed Shafi Shajakhan
On Fri, May 27, 2016 at 10:12:45AM +0200, Sven Eckelmann wrote: > On Wednesday 11 May 2016 11:54:30 Mohammed Shafi Shajakhan wrote: > > #else > > -static inline void ath10k_sta_update_rx_duration(struct ath10k *ar, > > -s

[PATCH v2] ath10k: Fix 10.4 extended peer stats update

2016-05-27 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> 10.4 'extended peer stats' will be not be appended with normal peer stats data and they shall be coming in separate chunks. Fix this by maintaining a separate linked list 'extender peer stats' for 10.4 and update rx_duration f

[PATCH] ath10k: Fix error while writing 'simulate_fw_crash' debugfs

2016-05-26 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> Fix invalid argument error while writing 'simulate_fw_crash', though the funcionality is working fine we get an error 'invalid argument' because 'count' value is not returned properly (no reason to reduce the count value for re

Re: [PATCH 1/2] ath10k: add QCA9887 chipset support

2016-05-23 Thread Mohammed Shafi Shajakhan
On Mon, May 23, 2016 at 11:52:11AM +0530, Mohammed Shafi Shajakhan wrote: > Hi Sven, > > On Fri, May 20, 2016 at 04:41:11PM +0200, Sven Eckelmann wrote: > > Add the hardware name, revision, firmware names and update the pci_id > > table. > > > > QA9887 HW1.0 is

  1   2   >