[PATCH v3 0/7] wcn36xx: Add a set of helpers to enable VHT parameter passing
Bryan O'Donoghue
bryan.odonoghue at linaro.org
Tue Sep 8 14:24:30 EDT 2020
This series is two in a set of seven to add support for wcn3680 at 802.11ac
data-rates.
V3:
- Enables MU-MIMO bits in wcn36xx_smd_set_sta_vht_params()
- Adds a set of helper functions. In the previous set these unused
declarations would cause a warning during compile because they were
declared static with the intention to add code later on to use them.
To resolve that, I do not declare the functions static. Static
declaration is done in a later patch after the functions have been used.
- There are two good reasons for this:
1. Granularity. I think its important and useful to have a patch per
function that clearly sets out what it does and why.
2. Patch size. Declaring seven functions at the time of their first
use is not good practice.
I think maintaining patch granularity is a good enough reason to not squash
these commits down. It is pretty trivial to declare the functions static
later on without generating compile warnings in the interregnum.
V2:
This set makes a series of modifications to enable passing of an extended
V1 data-structure to the firmware.
A series of of helper functions are added to facilitate the setup.
The final step in the process is enabling the extended data-structure
parameter passing for the wcn3680 only.
Only a firmware that is capable of 80211.ac will accept the larger V1
messages and only after explicitly enabling the DOT11AC firmware feature
bit.
- No functional difference from V1 for these patches
Breaking into smaller chunks to enable easier review/merging
V1:
https://lore.kernel.org/linux-wireless/87eensldhi.fsf@codeaurora.org/T/#t
Bryan O'Donoghue (7):
wcn36xx: Add wcn36xx_set_default_rates_v1
wcn36xx: Add wcn36xx_smd_set_sta_default_vht_params()
wcn36xx: Add wcn36xx_smd_set_sta_default_ht_ldpc_params()
wcn36xx: Add wcn36xx_smd_set_sta_vht_params()
wcn36xx: Add wcn36xx_smd_set_sta_ht_ldpc_params()
wcn36xx: Add wcn36xx_smd_set_bss_vht_params()
wcn36xx: Add wrapper function wcn36xx_smd_set_sta_params_v1()
drivers/net/wireless/ath/wcn36xx/main.c | 8 ++
drivers/net/wireless/ath/wcn36xx/smd.c | 88 ++++++++++++++++++++++
drivers/net/wireless/ath/wcn36xx/wcn36xx.h | 1 +
3 files changed, 97 insertions(+)
--
2.27.0
More information about the wcn36xx
mailing list