| Message ID | 20240430091414.2486196-1-quic_tamizhr@quicinc.com |
|---|---|
| Headers | show |
| Series | wifi: ath12k: Remove unsupported and unused ring configurations | expand |
On 4/30/2024 2:14 AM, Tamizh Chelvam Raja wrote: > Currently in ath12k_dp_tx_htt_monitor_mode_ring_config() > ath12k_dp_tx_htt_tx_monitor_mode_ring_config() function wrongly called twice. > Fix that by calling ath12k_dp_tx_htt_rx_monitor_mode_ring_config(). > > Currently monitor mode is disabled in driver so the change is compile > tested and boot sequence verified. > > Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Tamizh Chelvam Raja <quic_tamizhr@quicinc.com> Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com> (although very confusing that there is preexisting RX functionality in dp_tx.c)
Currently in driver doing memory allocation for tx_monitor, tcl_cmd_ring and tcl_status ring. Here driver support for tx_monitor mode is not there and memory for tcl_cmd and tcl_status rings are allocated by firmware and it uses that memory instead of host allocated. So avoid these unused ring setup configuration. Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1 Tamizh Chelvam Raja (3): wifi: ath12k: fix calling correct function for rx monitor mode wifi: ath12k: Remove unsupported tx monitor handling wifi: ath12k: Remove unused tcl_*_ring configuration v4: * Fixed ath12k-check warnings v3: * Addressed Karthikeyan's review comment of modifying ath12k_dp_tx_htt_rx_monitor_mode_ring_config in correct place. v2: * Rebased on top of ToT drivers/net/wireless/ath/ath12k/dp.c | 16 ------- drivers/net/wireless/ath/ath12k/dp.h | 2 - drivers/net/wireless/ath/ath12k/dp_mon.c | 40 +--------------- drivers/net/wireless/ath/ath12k/dp_rx.c | 59 +----------------------- drivers/net/wireless/ath/ath12k/dp_tx.c | 44 +----------------- drivers/net/wireless/ath/ath12k/dp_tx.h | 3 +- 6 files changed, 4 insertions(+), 160 deletions(-) base-commit: 861928281f22a08e3d258ebcfe7159d729ec07d4