Re: [ath9k-devel] [RFC/RFT 0/3] ath10k: fix beacon loss handling

Michal, could you please address Kalle's comments. From me ACK for the series (tested OK) Bartosz On 9 May 2013 12:33, Kalle Valo kv...@qca.qualcomm.com wrote: Bartosz Markowski bartosz.markow...@tieto.com writes: Kalle, could you please submit/review these? I will not commit any patches

[ath9k-devel] [PATCH] ath10k: strip protected flag in managment frames

From: Marek Kwaczynski marek.kwaczyn...@tieto.com Decryption and all crypto verification are implemented in HW. Mac80211 tries to decrypt RX CCMP managment frames without this patch. Signed-off-by: Marek Kwaczynski marek.kwaczyn...@tieto.com Signed-off-by: Bartosz Markowski bartosz.markow

Re: [ath9k-devel] [RFC/RFT 0/3] ath10k: fix beacon loss handling

ACK Kalle, could you please submit/review these? Fixes quite serious problem. Bartosz On 6 May 2013 16:03, Michal Kazior michal.kaz...@tieto.com wrote: FW is lying about tx status. First two patches do a little cleanup. The last one fixes beacon loss handling. This can be considered a

[ath9k-devel] [PATCH] ath10k: fix htt debug flag

Use HTT_DUMP for tx frag/msdu. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/htt_tx.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/htt_tx.c b/drivers/net/wireless/ath/ath10k

[ath9k-devel] [PATCH] ath10k: unify header inclusion guard names

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/htc.h |4 ++-- drivers/net/wireless/ath/ath10k/trace.h |6 +++--- drivers/net/wireless/ath/ath10k/txrx.h |4 ++-- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers

[ath9k-devel] [PATCH] ath10k: fix (void *) casts in wmi submodule

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/wmi.c | 42 - 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c index

[ath9k-devel] [PATCH 3/3] ath10k: kill obsolete CE functions

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/ce.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/ce.c b/drivers/net/wireless/ath/ath10k/ce.c index 7d77ad7..44c8947 100644 --- a/drivers/net

[ath9k-devel] [PATCH 1/3] ath10k: refactor CE_BASE_ADDRESS into inline function

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/ce.c | 12 ++-- drivers/net/wireless/ath/ath10k/ce.h |6 +++--- drivers/net/wireless/ath/ath10k/pci.h |2 +- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git

[ath9k-devel] [PATCH 0/3] cleanup CE macros

Bartosz Markowski (3): ath10k: refactor CE_BASE_ADDRESS into inline function ath10k: change CE macros to inline functions ath10k: kill obsolete CE functions drivers/net/wireless/ath/ath10k/ce.c | 273 + drivers/net/wireless/ath/ath10k/ce.h | 105

[ath9k-devel] [PATCH 2/3] ath10k: change CE macros to inline functions

Change CE direct addr access macros to inline functions. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/ce.c | 276 ++--- drivers/net/wireless/ath/ath10k/ce.h | 99 drivers/net/wireless/ath/ath10k

[ath9k-devel] [PATCH] ath10k: fix compilation warning

In function ‘ath10k_pci_probe’: /wireless-testing/drivers/net/wireless/ath/ath10k/pci.c:2150: warning: unused variable ‘hif_type’ Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers

[ath9k-devel] [PATCH 0/2] ath10k: killing things..

Bartosz Markowski (2): ath10k: kill pci_write32_v1_workaround function ath10k: kill TARGET_TYPE_AR9888 drivers/net/wireless/ath/ath10k/bmi.h |8 +++ drivers/net/wireless/ath/ath10k/core.c | 28 +-- drivers/net/wireless/ath/ath10k/core.h | 12

[ath9k-devel] [PATCH 1/2] ath10k: kill pci_write32_v1_workaround function

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.h | 38 +++-- 1 file changed, 17 insertions(+), 21 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.h b/drivers/net/wireless/ath/ath10k/pci.h index

[ath9k-devel] [PATCH 2/2] ath10k: kill TARGET_TYPE_AR9888

Get rid of target_type variable also. At the moment ath10k does not support anything else then QCA988x HW. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/bmi.h |8 +++- drivers/net/wireless/ath/ath10k/core.c | 28

[ath9k-devel] [PATCH v2 0/7] killing things - respin

Just to avoid many merge conflicts I'm re-sending prev 2 patches in the same sack. Bartosz Markowski (7): ath10k: kill pci_write32_v1_workaround function ath10k: kill TARGET_TYPE_AR9888 ath10k: replace AR9888 occurrences with QCA988x ath10k: kill obsolete defines in hw.h ath10k: remove

[ath9k-devel] [PATCH v2 2/7] ath10k: kill TARGET_TYPE_AR9888

Get rid of target_type variable also. At the moment ath10k does not support anything else then QCA988x HW. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/bmi.h |8 +++- drivers/net/wireless/ath/ath10k/core.c | 28

[ath9k-devel] [PATCH v2 4/7] ath10k: kill obsolete defines in hw.h

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hw.h |3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hw.h b/drivers/net/wireless/ath/ath10k/hw.h index 6c2a0cd..ddb9b58 100644 --- a/drivers/net/wireless/ath

[ath9k-devel] [PATCH v2 3/7] ath10k: replace AR9888 occurrences with QCA988x

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/bmi.h |4 +-- drivers/net/wireless/ath/ath10k/core.c | 38 +-- drivers/net/wireless/ath/ath10k/core.h |2 +- drivers/net/wireless/ath/ath10k/htt.h

[ath9k-devel] [PATCH v2 5/7] ath10k: remove odd line break

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hw.h |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hw.h b/drivers/net/wireless/ath/ath10k/hw.h index ddb9b58..2a624ea 100644 --- a/drivers

[ath9k-devel] [PATCH v2 6/7] ath10k: change FW folder name from AR9888 - QCA988X

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hw.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hw.h b/drivers/net/wireless/ath/ath10k/hw.h index 2a624ea..c9159d3 100644 --- a/drivers

[ath9k-devel] [PATCH v2 7/7] ath10k: add new PCI 'feature' to track QCA988x_1.0 hacks

This is to remove excessive bool variable from ar_pci struct and to not check directly device version, again. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c |8 +--- drivers/net/wireless/ath/ath10k/pci.h |8 2 files

[ath9k-devel] [PATCH] ath10k: fix comments style in rx_desc.h file

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/rx_desc.h | 1360 ++--- 1 file changed, 680 insertions(+), 680 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/rx_desc.h b/drivers/net/wireless/ath/ath10k/rx_desc.h

[ath9k-devel] [PATCH 0/3] ath10k: cleanup - respin

Couple of updated patches. Bartosz Markowski (3): ath10k: remove excessive CONFIG_PM_SLEEP checks ath10k: kill PCIE_LOCAL_REG_READ/WRITE macros ath10k: update supported firmware tag to .599 drivers/net/wireless/ath/ath10k/core.c |5 + drivers/net/wireless/ath/ath10k/hw.h |2

[ath9k-devel] [PATCH 1/3] ath10k: remove excessive CONFIG_PM_SLEEP checks

There's no big overhead and without those #ifdefs the code looks simpler. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/core.c b

[ath9k-devel] [PATCH 2/3] ath10k: kill PCIE_LOCAL_REG_READ/WRITE macros

Convert them into inline functions with better names. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c | 16 +++- drivers/net/wireless/ath/ath10k/pci.h | 11 ++- 2 files changed, 13 insertions(+), 14 deletions(-) diff

[ath9k-devel] [PATCH 3/3] ath10k: update supported firmware tag to .599

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hw.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath10k/hw.h b/drivers/net/wireless/ath/ath10k/hw.h index 82465a8..9049e4c 100644 --- a/drivers/net

[ath9k-devel] [PATCH 0/7] ath10k: more cleanup

Bartosz Markowski (7): ath10k: Print more elgant HW name/version ath10k: shift ath10k_generic_iter structure to mac header ath10k: kill HIF_TYPE_AR9888 ath10k: shift REG_DUMP_COUNT_AR9888 to pci block ath10k: move DIAG_TRANSFER_LIMIT to pci block ath10k: remove leftover prototypes

[ath9k-devel] [PATCH 3/7] ath10k: kill HIF_TYPE_AR9888

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hif.h |2 -- drivers/net/wireless/ath/ath10k/pci.c |3 --- 2 files changed, 5 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hif.h b/drivers/net/wireless/ath/ath10k/hif.h index

[ath9k-devel] [PATCH 6/7] ath10k: remove leftover prototypes

* ath10k_register_host_reg_table() * ath10k_register_target_reg_table() Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hif.h |3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hif.h b/drivers/net/wireless

[ath9k-devel] [PATCH 7/7] ath10k: rename ath10k_pci_wait_for_target_to_awake

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c index 6fe0508..790f23a 100644

[ath9k-devel] [PATCH 4/7] ath10k: shift REG_DUMP_COUNT_AR9888 to pci block

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hif.h |3 --- drivers/net/wireless/ath/ath10k/pci.h |3 +++ 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hif.h b/drivers/net/wireless/ath

[ath9k-devel] [PATCH 5/7] ath10k: move DIAG_TRANSFER_LIMIT to pci block

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hif.h |6 -- drivers/net/wireless/ath/ath10k/pci.h |5 + 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hif.h b/drivers/net/wireless

[ath9k-devel] [PATCH 0/2] rebase patch 6 and 7 from 'various fixes' patch-set

* [PATCH v2 6/7] ath10k: reindent ath10k_pci_suspend * [PATCH v2 7/7] ath10k: return error if suspend command fails Markowski Bartosz (2): ath10k: reindent ath10k_pci_suspend ath10k: return error if suspend command fails drivers/net/wireless/ath/ath10k/pci.c | 50

[ath9k-devel] [PATCH 1/2] ath10k: reindent ath10k_pci_suspend

From: Markowski Bartosz bartosz.markow...@tieto.com Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c | 50 + 1 file changed, 26 insertions(+), 24 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k

[ath9k-devel] [PATCH 2/2] ath10k: return error if suspend command fails

From: Markowski Bartosz bartosz.markow...@tieto.com In case the WMI suspend commands fails, return -EBUSY up. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net

[ath9k-devel] [PATCH] ath10k: tag ath10k_target_ps as static

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c |2 +- drivers/net/wireless/ath/ath10k/pci.h |2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k

[ath9k-devel] [PATCH] ath10k: use config_enabled() for CONFIG_PM_SLEEP

Make use of config_enabled() check insted of raw #ifdef's within function bodies when checking CONFIG_PM_SLEEP option. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions

[ath9k-devel] [PATCH 0/3] ath10k: kill unnecessary macros

* use iowrite32/ioread32 directly * fix comment Bartosz Markowski (3): ath10k: kill A_PCIE_LOCAL_REG_READ ath10k: kill A_PCIE_LOCAL_REG_WRITE ath10k: fix comment to reflect time in mili-seconds drivers/net/wireless/ath/ath10k/pci.c | 19 --- drivers/net/wireless/ath

[ath9k-devel] [PATCH 1/3] ath10k: kill A_PCIE_LOCAL_REG_READ

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c |8 +++- drivers/net/wireless/ath/ath10k/pci.h |3 --- 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless

[ath9k-devel] [PATCH 2/3] ath10k: kill A_PCIE_LOCAL_REG_WRITE

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c | 11 +-- drivers/net/wireless/ath/ath10k/pci.h |4 2 files changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net

[ath9k-devel] [PATCH 3/3] ath10k: fix comment to reflect time in mili-seconds

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.h b/drivers/net/wireless/ath/ath10k/pci.h index a1a498b..4b0351f 100644

[ath9k-devel] [PATCH] ath10k: fix HW version/revision names

In this patch only the string values we print during hw_params initialization. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/core.c

[ath9k-devel] [PATCH] ath10k: move QCA9880_1.0 PCI-specific hacks to pci sub-module

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.h |3 --- drivers/net/wireless/ath/ath10k/pci.c |4 ++-- drivers/net/wireless/ath/ath10k/pci.h | 16 +++- 3 files changed, 13 insertions(+), 10 deletions(-) diff --git

[ath9k-devel] [PATCH v2] ath10k: fix HW version/revision names

In this patch we change only the string values we print during hw_params initialization. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath

[ath9k-devel] [PATCH v2 0/7] various fixes

Fix since V1: * rename core suspend/resume functions * split reindentation and new error path * remove all excessive CONFIG_PM_SLEEP in one patch. Bartosz Markowski (7): ath10k: move suspend/resume exports to core.c file ath10k: remove excessive ifdef checks ath10k: fix pointer casts

[ath9k-devel] [PATCH v2 2/7] ath10k: remove excessive ifdef checks

No need to check CONFIG_PM_SLEEP everywhere, the PCI suspend hooks is the only place we need to check it. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |2 -- drivers/net/wireless/ath/ath10k/core.h |2 -- drivers/net/wireless/ath

[ath9k-devel] [PATCH v2 1/7] ath10k: move suspend/resume exports to core.c file

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c | 25 + drivers/net/wireless/ath/ath10k/core.h |3 +++ drivers/net/wireless/ath/ath10k/pci.c |6 +++--- drivers/net/wireless/ath/ath10k/wmi.c |7

[ath9k-devel] [PATCH v2 3/7] ath10k: fix pointer casts in debug code

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/debug.h |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/debug.h b/drivers/net/wireless/ath/ath10k/debug.h index dc4847b..681e400 100644

[ath9k-devel] [PATCH v2 5/7] ath10k: remove unnecessary NULL check

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/htc.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/htc.c b/drivers/net/wireless/ath/ath10k/htc.c index ff4b1bf..299b9e6 100644

[ath9k-devel] [PATCH v2 4/7] ath10k: shift extern variable where it belongs

ath10k_target_ps is PCI specific Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hif.h |2 -- drivers/net/wireless/ath/ath10k/pci.h |2 ++ 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hif.h

[ath9k-devel] [PATCH v2 6/7] ath10k: reindent ath10k_pci_suspend

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c | 50 + 1 file changed, 26 insertions(+), 24 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c index

[ath9k-devel] [PATCH v2 7/7] ath10k: return error if suspend command fails

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c index 3e841dd..22b533b 100644 --- a/drivers

[ath9k-devel] [PATCH v3] ath10k: move suspend/resume exports to core.c file

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c | 25 + drivers/net/wireless/ath/ath10k/core.h |3 +++ drivers/net/wireless/ath/ath10k/pci.c |6 +++--- drivers/net/wireless/ath/ath10k/wmi.c |7

[ath9k-devel] [PATCH] ath10k: simplify FW binary names

If we have a folder tree to store the fw files, there's no need to keep the names so complicated. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hw.h |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net

[ath9k-devel] [PATCH] ath10k: split ath10k_pci_target_ps_control

Divide the target_ps_control into 2 logical pieces: - ath10k_pci_wake - ath10k_pci_sleep Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c | 117 - drivers/net/wireless/ath/ath10k/pci.h | 18 + 2 files

[ath9k-devel] [PATCH] ath10k: fix double-prefixed functions

A few functions have been double-prefixed 'ath10k_ath10k_'. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |4 ++-- drivers/net/wireless/ath/ath10k/mac.c | 10 +- drivers/net/wireless/ath/ath10k/mac.h |6 +++--- drivers

[ath9k-devel] [PATCH 1/6] ath10k: remove obsolete function prototype

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/htt.h |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/htt.h b/drivers/net/wireless/ath/ath10k/htt.h index 96178ea..7349606 100644

[ath9k-devel] [PATCH 0/6] various cleanup patches

Bartosz Markowski (6): ath10k: remove obsolete function prototype ath10k: fix typo in ath10k Kconfig ath10k: use if statement instead of cond. operator ath10k: remove pointless goto label ath10k: rename PCI target PS contol functions ath10k: remove fw event chain for target dump

[ath9k-devel] [PATCH 4/6] ath10k: remove pointless goto label

There's nothing more than return call inside the label. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/ce.c |8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/ce.c b/drivers/net

[ath9k-devel] [PATCH 5/6] ath10k: rename PCI target PS contol functions

* TARGET_ACCESS_BEGIN() - ath10k_pci_wake() * TARGET_ACCESS_END() - ath10k_pci_sleep() Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/ce.c | 48 - drivers/net/wireless/ath/ath10k/pci.c | 22

[ath9k-devel] [PATCH 6/6] ath10k: remove fw event chain for target dump notification

We throw the FW dump registers straight from the bottom layers, there's no recovery or any logic up, so get rid of the callbacks chain for now. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |6 -- drivers/net/wireless/ath/ath10k

[ath9k-devel] [PATCH 2/7] ath10k: move suspend/resume exports to core.c file

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c | 25 + drivers/net/wireless/ath/ath10k/core.h |3 +++ drivers/net/wireless/ath/ath10k/pci.c |6 +++--- drivers/net/wireless/ath/ath10k/wmi.c |7

[ath9k-devel] [PATCH 1/7] ath10k: rename struct wmi_struct - struct ath10k_wmi

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/wmi.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c index 533e25f

[ath9k-devel] [PATCH 4/7] ath10k: reindent ath10k_pci_suspend

Also throw an error to PCI if WMI suspend command fails. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/pci.c | 50 + 1 file changed, 26 insertions(+), 24 deletions(-) diff --git a/drivers/net/wireless/ath

[ath9k-devel] [PATCH 3/7] ath10k: remove excessive ifdef checks

No need to check CONFIG_PM_SLEEP everywhere, the PCI suspend hooks is the only place we need to check it. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |2 -- drivers/net/wireless/ath/ath10k/wmi.c |2 -- 2 files changed, 4

[ath9k-devel] [PATCH 5/7] ath10k: fix pointer casts in debug code

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.h |2 -- drivers/net/wireless/ath/ath10k/debug.h |6 +++--- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/core.h b/drivers/net/wireless

[ath9k-devel] [PATCH 6/7] ath10k: shift extern variable where it belongs

ath10k_target_ps is PCI specific Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/hif.h |2 -- drivers/net/wireless/ath/ath10k/pci.h |2 ++ 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/hif.h

[ath9k-devel] [PATCH 7/7] ath10k: remove unnecessary NULL check

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/htc.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/htc.c b/drivers/net/wireless/ath/ath10k/htc.c index ff4b1bf..299b9e6 100644

[ath9k-devel] [PATCH] ath10k: split way to long line of code

split into functional blocks, to reduce almost 200 char line. Now it's easier to read both, the code and the logs produced. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/wmi.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions

[ath9k-devel] [PATCH 0/3] remaining ath10k_ prefixes for sub-modules

After this patch only core, mac and txrx files are not explicitly prefixed. And I think we can leave it like that for now. Bartosz Markowski (3): ath10k: add ath10k_ prefixes into wmi functions ath10k: add ath10k_ prefixes into htt_tx functions ath10k: add ath10k_ prefixes into htt_rx

[ath9k-devel] [PATCH 3/3] ath10k: add ath10k_ prefixes into htt_rx functions

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/htt.c| 10 +-- drivers/net/wireless/ath/ath10k/htt.h|6 +- drivers/net/wireless/ath/ath10k/htt_rx.c | 121 +++--- 3 files changed, 68 insertions(+), 69 deletions

[ath9k-devel] [PATCH 0/4] small cleanups

Bartosz Markowski (4): ath10k: fix indentation ath10k: remove odd 'todo' comment ath10k: remove 'todo' comments ath10k: remove obsolete define drivers/net/wireless/ath/ath10k/core.c |6 +- drivers/net/wireless/ath/ath10k/htc-priv.h |3 --- drivers/net/wireless/ath/ath10k

[ath9k-devel] [PATCH 3/4] ath10k: remove 'todo' comments

* firmware download Revisited, seems we handle error paths properly. * core registering Revisited. As for current design, fallback shall be safe. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/core.c |6 +- 1 file changed, 1

[ath9k-devel] [PATCH 2/4] ath10k: remove odd 'todo' comment

Seems like it was inherited from properiatery driver. Does not make sense to in ath10k context. Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/wmi.h |4 1 file changed, 4 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/wmi.h

[ath9k-devel] [PATCH 4/4] ath10k: remove obsolete define

Signed-off-by: Bartosz Markowski bartosz.markow...@tieto.com --- drivers/net/wireless/ath/ath10k/htc-priv.h |3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/htc-priv.h b/drivers/net/wireless/ath/ath10k/htc-priv.h index 89bd91a..f47dabc 100644 --- a/drivers