On 7/25/2024 5:00 AM, kangyang wrote:
> From: Kang Yang <quic_kangyang@xxxxxxxxxxx>
>
> Currently mon reap timer is not used, and it is not needed anymore.
>
> So remove related code.
>
> Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0.c5-00481-QCAHMTSWPL_V1.0_V2.0_SILICONZ-3
>
> Signed-off-by: Kang Yang <quic_kangyang@xxxxxxxxxxx>
> ---
> drivers/net/wireless/ath/ath12k/core.h | 2 --
> drivers/net/wireless/ath/ath12k/dp.c | 25 -------------------------
> 2 files changed, 27 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath12k/core.h b/drivers/net/wireless/ath/ath12k/core.h
> index daf7c04bb728..83ef92ecdc39 100644
> --- a/drivers/net/wireless/ath/ath12k/core.h
> +++ b/drivers/net/wireless/ath/ath12k/core.h
> @@ -888,8 +888,6 @@ struct ath12k_base {
> struct ath12k_dbring_cap *db_caps;
> u32 num_db_cap;
>
> - struct timer_list mon_reap_timer;
> -
> struct completion htc_suspend;
>
> u64 fw_soc_drop_count;
> diff --git a/drivers/net/wireless/ath/ath12k/dp.c b/drivers/net/wireless/ath/ath12k/dp.c
> index 61aa78d8bd8c..547ecb419809 100644
> --- a/drivers/net/wireless/ath/ath12k/dp.c
> +++ b/drivers/net/wireless/ath/ath12k/dp.c
> @@ -974,8 +974,6 @@ void ath12k_dp_pdev_free(struct ath12k_base *ab)
> {
> int i;
>
> - del_timer_sync(&ab->mon_reap_timer);
> -
> for (i = 0; i < ab->num_radios; i++)
> ath12k_dp_rx_pdev_free(ab, i);
> }
> @@ -1020,27 +1018,6 @@ void ath12k_dp_hal_rx_desc_init(struct ath12k_base *ab)
> ab->hal_rx_ops->rx_desc_get_desc_size();
> }
>
> -static void ath12k_dp_service_mon_ring(struct timer_list *t)
> -{
> - struct ath12k_base *ab = from_timer(ab, t, mon_reap_timer);
> - int i;
> -
> - for (i = 0; i < ab->hw_params->num_rxdma_per_pdev; i++)
> - ath12k_dp_mon_process_ring(ab, i, NULL, DP_MON_SERVICE_BUDGET,
> - ATH12K_DP_RX_MONITOR_MODE);
> -
> - mod_timer(&ab->mon_reap_timer, jiffies +
> - msecs_to_jiffies(ATH12K_MON_TIMER_INTERVAL));
> -}
> -
> -static void ath12k_dp_mon_reap_timer_init(struct ath12k_base *ab)
> -{
> - if (ab->hw_params->rxdma1_enable)
> - return;
> -
> - timer_setup(&ab->mon_reap_timer, ath12k_dp_service_mon_ring, 0);
> -}
> -
> int ath12k_dp_pdev_alloc(struct ath12k_base *ab)
> {
> struct ath12k *ar;
> @@ -1051,8 +1028,6 @@ int ath12k_dp_pdev_alloc(struct ath12k_base *ab)
> if (ret)
> goto out;
>
> - ath12k_dp_mon_reap_timer_init(ab);
> -
> /* TODO: Per-pdev rx ring unlike tx ring which is mapped to different AC's */
> for (i = 0; i < ab->num_radios; i++) {
> ar = ab->pdevs[i].ar;
