mbox series

[v3,0/3] wcn36xx: Implement explicit beacon filter tables

Message ID 20211214134630.2214840-1-bryan.odonoghue@linaro.org
Headers show
Series wcn36xx: Implement explicit beacon filter tables | expand

Message

Bryan O'Donoghue Dec. 14, 2021, 1:46 p.m. UTC 
V3:
Adds check to updating beacon filter for NL80211_IFTYPE_STATION only - Loic
Adds Reviewed-by as indicated - Loic

V2:
Removes duplicate definition of mask field defines

V1:
Downstream provides the ability to directly program the beacon filter
tables. Currently in upstream we rely on whatever is the default filtration
table in firmware.

A trivial packing fixup is required for the SMD structure. The downstream
filtration table from the Linux driver is applied but, we are not
necessarily constrained to using this table forever.

Tested on wcn3620 and wcn3680b.

Bryan O'Donoghue (3):
  wcn36xx: Fix beacon filter structure definitions
  wcn36xx: Fix physical location of beacon filter comment
  wcn36xx: Implement downstream compliant beacon filtering

 drivers/net/wireless/ath/wcn36xx/hal.h  | 30 +++++++--
 drivers/net/wireless/ath/wcn36xx/main.c |  2 +
 drivers/net/wireless/ath/wcn36xx/smd.c  | 87 +++++++++++++++++++++++++
 drivers/net/wireless/ath/wcn36xx/smd.h  |  3 +
 4 files changed, 116 insertions(+), 6 deletions(-)

Comments

Bryan O'Donoghue Dec. 16, 2021, 1:21 p.m. UTC | #1 
On 16/12/2021 10:31, Kalle Valo wrote:
> I don't think the term downstream is clear for everyone, I guess with
> that you mean the prima driver? I would prefer to use "prima driver" in
> commit logs, I can change it and no need to resend because of this.

OK np thanks for zapping the comments directly.

Noted for future reference :g/downstream/s//prima\ driver/g

---
bod