xref: /freebsd/sys/contrib/dev/iwlwifi/mvm/sf.c (revision a4128aad8503277614f2d214011ef60a19447b83)
1bfcc09ddSBjoern A. Zeeb // SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause
2bfcc09ddSBjoern A. Zeeb /*
3*a4128aadSBjoern A. Zeeb  * Copyright (C) 2013-2014, 2018-2019, 2022-2024 Intel Corporation
4bfcc09ddSBjoern A. Zeeb  * Copyright (C) 2013-2014 Intel Mobile Communications GmbH
5bfcc09ddSBjoern A. Zeeb  */
6bfcc09ddSBjoern A. Zeeb #include "mvm.h"
7bfcc09ddSBjoern A. Zeeb 
8bfcc09ddSBjoern A. Zeeb /* For counting bound interfaces */
9bfcc09ddSBjoern A. Zeeb struct iwl_mvm_active_iface_iterator_data {
10bfcc09ddSBjoern A. Zeeb 	struct ieee80211_vif *ignore_vif;
119af1bba4SBjoern A. Zeeb 	struct ieee80211_sta *sta_vif_ap_sta;
12bfcc09ddSBjoern A. Zeeb 	enum iwl_sf_state sta_vif_state;
13bfcc09ddSBjoern A. Zeeb 	u32 num_active_macs;
14bfcc09ddSBjoern A. Zeeb };
15bfcc09ddSBjoern A. Zeeb 
16bfcc09ddSBjoern A. Zeeb /*
17bfcc09ddSBjoern A. Zeeb  * Count bound interfaces which are not p2p, besides data->ignore_vif.
18bfcc09ddSBjoern A. Zeeb  * data->station_vif will point to one bound vif of type station, if exists.
19bfcc09ddSBjoern A. Zeeb  */
20bfcc09ddSBjoern A. Zeeb static void iwl_mvm_bound_iface_iterator(void *_data, u8 *mac,
21bfcc09ddSBjoern A. Zeeb 					 struct ieee80211_vif *vif)
22bfcc09ddSBjoern A. Zeeb {
23bfcc09ddSBjoern A. Zeeb 	struct iwl_mvm_active_iface_iterator_data *data = _data;
24bfcc09ddSBjoern A. Zeeb 	struct iwl_mvm_vif *mvmvif = iwl_mvm_vif_from_mac80211(vif);
25bfcc09ddSBjoern A. Zeeb 
269af1bba4SBjoern A. Zeeb 	if (vif == data->ignore_vif || !mvmvif->deflink.phy_ctxt ||
27bfcc09ddSBjoern A. Zeeb 	    vif->type == NL80211_IFTYPE_P2P_DEVICE)
28bfcc09ddSBjoern A. Zeeb 		return;
29bfcc09ddSBjoern A. Zeeb 
30bfcc09ddSBjoern A. Zeeb 	data->num_active_macs++;
31bfcc09ddSBjoern A. Zeeb 
32bfcc09ddSBjoern A. Zeeb 	if (vif->type == NL80211_IFTYPE_STATION) {
339af1bba4SBjoern A. Zeeb 		data->sta_vif_ap_sta = mvmvif->ap_sta;
349af1bba4SBjoern A. Zeeb 		if (vif->cfg.assoc)
35bfcc09ddSBjoern A. Zeeb 			data->sta_vif_state = SF_FULL_ON;
36bfcc09ddSBjoern A. Zeeb 		else
37bfcc09ddSBjoern A. Zeeb 			data->sta_vif_state = SF_INIT_OFF;
38bfcc09ddSBjoern A. Zeeb 	}
39bfcc09ddSBjoern A. Zeeb }
40bfcc09ddSBjoern A. Zeeb 
41bfcc09ddSBjoern A. Zeeb /*
42bfcc09ddSBjoern A. Zeeb  * Aging and idle timeouts for the different possible scenarios
43bfcc09ddSBjoern A. Zeeb  * in default configuration
44bfcc09ddSBjoern A. Zeeb  */
45bfcc09ddSBjoern A. Zeeb static const
46bfcc09ddSBjoern A. Zeeb __le32 sf_full_timeout_def[SF_NUM_SCENARIO][SF_NUM_TIMEOUT_TYPES] = {
47bfcc09ddSBjoern A. Zeeb 	{
48bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_SINGLE_UNICAST_AGING_TIMER_DEF),
49bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_SINGLE_UNICAST_IDLE_TIMER_DEF)
50bfcc09ddSBjoern A. Zeeb 	},
51bfcc09ddSBjoern A. Zeeb 	{
52bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_AGG_UNICAST_AGING_TIMER_DEF),
53bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_AGG_UNICAST_IDLE_TIMER_DEF)
54bfcc09ddSBjoern A. Zeeb 	},
55bfcc09ddSBjoern A. Zeeb 	{
56bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_MCAST_AGING_TIMER_DEF),
57bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_MCAST_IDLE_TIMER_DEF)
58bfcc09ddSBjoern A. Zeeb 	},
59bfcc09ddSBjoern A. Zeeb 	{
60bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_BA_AGING_TIMER_DEF),
61bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_BA_IDLE_TIMER_DEF)
62bfcc09ddSBjoern A. Zeeb 	},
63bfcc09ddSBjoern A. Zeeb 	{
64bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_TX_RE_AGING_TIMER_DEF),
65bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_TX_RE_IDLE_TIMER_DEF)
66bfcc09ddSBjoern A. Zeeb 	},
67bfcc09ddSBjoern A. Zeeb };
68bfcc09ddSBjoern A. Zeeb 
69bfcc09ddSBjoern A. Zeeb /*
70bfcc09ddSBjoern A. Zeeb  * Aging and idle timeouts for the different possible scenarios
71bfcc09ddSBjoern A. Zeeb  * in single BSS MAC configuration.
72bfcc09ddSBjoern A. Zeeb  */
73bfcc09ddSBjoern A. Zeeb static const __le32 sf_full_timeout[SF_NUM_SCENARIO][SF_NUM_TIMEOUT_TYPES] = {
74bfcc09ddSBjoern A. Zeeb 	{
75bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_SINGLE_UNICAST_AGING_TIMER),
76bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_SINGLE_UNICAST_IDLE_TIMER)
77bfcc09ddSBjoern A. Zeeb 	},
78bfcc09ddSBjoern A. Zeeb 	{
79bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_AGG_UNICAST_AGING_TIMER),
80bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_AGG_UNICAST_IDLE_TIMER)
81bfcc09ddSBjoern A. Zeeb 	},
82bfcc09ddSBjoern A. Zeeb 	{
83bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_MCAST_AGING_TIMER),
84bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_MCAST_IDLE_TIMER)
85bfcc09ddSBjoern A. Zeeb 	},
86bfcc09ddSBjoern A. Zeeb 	{
87bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_BA_AGING_TIMER),
88bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_BA_IDLE_TIMER)
89bfcc09ddSBjoern A. Zeeb 	},
90bfcc09ddSBjoern A. Zeeb 	{
91bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_TX_RE_AGING_TIMER),
92bfcc09ddSBjoern A. Zeeb 		cpu_to_le32(SF_TX_RE_IDLE_TIMER)
93bfcc09ddSBjoern A. Zeeb 	},
94bfcc09ddSBjoern A. Zeeb };
95bfcc09ddSBjoern A. Zeeb 
96bfcc09ddSBjoern A. Zeeb static void iwl_mvm_fill_sf_command(struct iwl_mvm *mvm,
97bfcc09ddSBjoern A. Zeeb 				    struct iwl_sf_cfg_cmd *sf_cmd,
98bfcc09ddSBjoern A. Zeeb 				    struct ieee80211_sta *sta)
99bfcc09ddSBjoern A. Zeeb {
100bfcc09ddSBjoern A. Zeeb 	int i, j, watermark;
1019af1bba4SBjoern A. Zeeb 	u8 max_rx_nss = 0;
1029af1bba4SBjoern A. Zeeb 	bool is_legacy = true;
1039af1bba4SBjoern A. Zeeb 	struct ieee80211_link_sta *link_sta;
1049af1bba4SBjoern A. Zeeb 	unsigned int link_id;
105bfcc09ddSBjoern A. Zeeb 
106bfcc09ddSBjoern A. Zeeb 	sf_cmd->watermark[SF_LONG_DELAY_ON] = cpu_to_le32(SF_W_MARK_SCAN);
107bfcc09ddSBjoern A. Zeeb 
108bfcc09ddSBjoern A. Zeeb 	/*
109bfcc09ddSBjoern A. Zeeb 	 * If we are in association flow - check antenna configuration
110bfcc09ddSBjoern A. Zeeb 	 * capabilities of the AP station, and choose the watermark accordingly.
111bfcc09ddSBjoern A. Zeeb 	 */
112bfcc09ddSBjoern A. Zeeb 	if (sta) {
1139af1bba4SBjoern A. Zeeb 		/* find the maximal NSS number among all links (if relevant) */
1149af1bba4SBjoern A. Zeeb 		rcu_read_lock();
1159af1bba4SBjoern A. Zeeb 		for (link_id = 0; link_id < ARRAY_SIZE(sta->link); link_id++) {
1169af1bba4SBjoern A. Zeeb 			link_sta = rcu_dereference(sta->link[link_id]);
1179af1bba4SBjoern A. Zeeb 			if (!link_sta)
1189af1bba4SBjoern A. Zeeb 				continue;
1199af1bba4SBjoern A. Zeeb 
1209af1bba4SBjoern A. Zeeb 			if (link_sta->ht_cap.ht_supported ||
1219af1bba4SBjoern A. Zeeb 			    link_sta->vht_cap.vht_supported ||
1229af1bba4SBjoern A. Zeeb 			    link_sta->eht_cap.has_eht ||
1239af1bba4SBjoern A. Zeeb 			    link_sta->he_cap.has_he) {
1249af1bba4SBjoern A. Zeeb 				is_legacy = false;
1259af1bba4SBjoern A. Zeeb 				max_rx_nss = max(max_rx_nss, link_sta->rx_nss);
1269af1bba4SBjoern A. Zeeb 			}
1279af1bba4SBjoern A. Zeeb 		}
1289af1bba4SBjoern A. Zeeb 		rcu_read_unlock();
1299af1bba4SBjoern A. Zeeb 
1309af1bba4SBjoern A. Zeeb 		if (!is_legacy) {
1319af1bba4SBjoern A. Zeeb 			switch (max_rx_nss) {
132bfcc09ddSBjoern A. Zeeb 			case 1:
133bfcc09ddSBjoern A. Zeeb 				watermark = SF_W_MARK_SISO;
134bfcc09ddSBjoern A. Zeeb 				break;
135bfcc09ddSBjoern A. Zeeb 			case 2:
136bfcc09ddSBjoern A. Zeeb 				watermark = SF_W_MARK_MIMO2;
137bfcc09ddSBjoern A. Zeeb 				break;
138bfcc09ddSBjoern A. Zeeb 			default:
139bfcc09ddSBjoern A. Zeeb 				watermark = SF_W_MARK_MIMO3;
140bfcc09ddSBjoern A. Zeeb 				break;
141bfcc09ddSBjoern A. Zeeb 			}
142bfcc09ddSBjoern A. Zeeb 		} else {
143bfcc09ddSBjoern A. Zeeb 			watermark = SF_W_MARK_LEGACY;
144bfcc09ddSBjoern A. Zeeb 		}
145bfcc09ddSBjoern A. Zeeb 	/* default watermark value for unassociated mode. */
146bfcc09ddSBjoern A. Zeeb 	} else {
147bfcc09ddSBjoern A. Zeeb 		watermark = SF_W_MARK_MIMO2;
148bfcc09ddSBjoern A. Zeeb 	}
149bfcc09ddSBjoern A. Zeeb 	sf_cmd->watermark[SF_FULL_ON] = cpu_to_le32(watermark);
150bfcc09ddSBjoern A. Zeeb 
151bfcc09ddSBjoern A. Zeeb 	for (i = 0; i < SF_NUM_SCENARIO; i++) {
152bfcc09ddSBjoern A. Zeeb 		for (j = 0; j < SF_NUM_TIMEOUT_TYPES; j++) {
153bfcc09ddSBjoern A. Zeeb 			sf_cmd->long_delay_timeouts[i][j] =
154bfcc09ddSBjoern A. Zeeb 					cpu_to_le32(SF_LONG_DELAY_AGING_TIMER);
155bfcc09ddSBjoern A. Zeeb 		}
156bfcc09ddSBjoern A. Zeeb 	}
157bfcc09ddSBjoern A. Zeeb 
158bfcc09ddSBjoern A. Zeeb 	if (sta) {
159bfcc09ddSBjoern A. Zeeb 		BUILD_BUG_ON(sizeof(sf_full_timeout) !=
160bfcc09ddSBjoern A. Zeeb 			     sizeof(__le32) * SF_NUM_SCENARIO *
161bfcc09ddSBjoern A. Zeeb 			     SF_NUM_TIMEOUT_TYPES);
162bfcc09ddSBjoern A. Zeeb 
163bfcc09ddSBjoern A. Zeeb 		memcpy(sf_cmd->full_on_timeouts, sf_full_timeout,
164bfcc09ddSBjoern A. Zeeb 		       sizeof(sf_full_timeout));
165bfcc09ddSBjoern A. Zeeb 	} else {
166bfcc09ddSBjoern A. Zeeb 		BUILD_BUG_ON(sizeof(sf_full_timeout_def) !=
167bfcc09ddSBjoern A. Zeeb 			     sizeof(__le32) * SF_NUM_SCENARIO *
168bfcc09ddSBjoern A. Zeeb 			     SF_NUM_TIMEOUT_TYPES);
169bfcc09ddSBjoern A. Zeeb 
170bfcc09ddSBjoern A. Zeeb 		memcpy(sf_cmd->full_on_timeouts, sf_full_timeout_def,
171bfcc09ddSBjoern A. Zeeb 		       sizeof(sf_full_timeout_def));
172bfcc09ddSBjoern A. Zeeb 	}
173bfcc09ddSBjoern A. Zeeb }
174bfcc09ddSBjoern A. Zeeb 
1759af1bba4SBjoern A. Zeeb static int iwl_mvm_sf_config(struct iwl_mvm *mvm, struct ieee80211_sta *sta,
176bfcc09ddSBjoern A. Zeeb 			     enum iwl_sf_state new_state)
177bfcc09ddSBjoern A. Zeeb {
178bfcc09ddSBjoern A. Zeeb 	struct iwl_sf_cfg_cmd sf_cmd = {
179bfcc09ddSBjoern A. Zeeb 		.state = cpu_to_le32(new_state),
180bfcc09ddSBjoern A. Zeeb 	};
181bfcc09ddSBjoern A. Zeeb 	int ret = 0;
182bfcc09ddSBjoern A. Zeeb 
183bfcc09ddSBjoern A. Zeeb 	/*
184bfcc09ddSBjoern A. Zeeb 	 * If an associated AP sta changed its antenna configuration, the state
185bfcc09ddSBjoern A. Zeeb 	 * will remain FULL_ON but SF parameters need to be reconsidered.
186bfcc09ddSBjoern A. Zeeb 	 */
187bfcc09ddSBjoern A. Zeeb 	if (new_state != SF_FULL_ON && mvm->sf_state == new_state)
188bfcc09ddSBjoern A. Zeeb 		return 0;
189bfcc09ddSBjoern A. Zeeb 
190bfcc09ddSBjoern A. Zeeb 	switch (new_state) {
191bfcc09ddSBjoern A. Zeeb 	case SF_UNINIT:
192bfcc09ddSBjoern A. Zeeb 		iwl_mvm_fill_sf_command(mvm, &sf_cmd, NULL);
193bfcc09ddSBjoern A. Zeeb 		break;
194bfcc09ddSBjoern A. Zeeb 	case SF_FULL_ON:
1959af1bba4SBjoern A. Zeeb 		if (!sta) {
196bfcc09ddSBjoern A. Zeeb 			IWL_ERR(mvm,
197bfcc09ddSBjoern A. Zeeb 				"No station: Cannot switch SF to FULL_ON\n");
198bfcc09ddSBjoern A. Zeeb 			return -EINVAL;
199bfcc09ddSBjoern A. Zeeb 		}
200bfcc09ddSBjoern A. Zeeb 		iwl_mvm_fill_sf_command(mvm, &sf_cmd, sta);
201bfcc09ddSBjoern A. Zeeb 		break;
202bfcc09ddSBjoern A. Zeeb 	case SF_INIT_OFF:
203bfcc09ddSBjoern A. Zeeb 		iwl_mvm_fill_sf_command(mvm, &sf_cmd, NULL);
204bfcc09ddSBjoern A. Zeeb 		break;
205bfcc09ddSBjoern A. Zeeb 	default:
206bfcc09ddSBjoern A. Zeeb 		WARN_ONCE(1, "Invalid state: %d. not sending Smart Fifo cmd\n",
207bfcc09ddSBjoern A. Zeeb 			  new_state);
208bfcc09ddSBjoern A. Zeeb 		return -EINVAL;
209bfcc09ddSBjoern A. Zeeb 	}
210bfcc09ddSBjoern A. Zeeb 
211bfcc09ddSBjoern A. Zeeb 	ret = iwl_mvm_send_cmd_pdu(mvm, REPLY_SF_CFG_CMD, CMD_ASYNC,
212bfcc09ddSBjoern A. Zeeb 				   sizeof(sf_cmd), &sf_cmd);
213bfcc09ddSBjoern A. Zeeb 	if (!ret)
214bfcc09ddSBjoern A. Zeeb 		mvm->sf_state = new_state;
215bfcc09ddSBjoern A. Zeeb 
216bfcc09ddSBjoern A. Zeeb 	return ret;
217bfcc09ddSBjoern A. Zeeb }
218bfcc09ddSBjoern A. Zeeb 
219bfcc09ddSBjoern A. Zeeb /*
220bfcc09ddSBjoern A. Zeeb  * Update Smart fifo:
221bfcc09ddSBjoern A. Zeeb  * Count bound interfaces that are not to be removed, ignoring p2p devices,
222bfcc09ddSBjoern A. Zeeb  * and set new state accordingly.
223bfcc09ddSBjoern A. Zeeb  */
224bfcc09ddSBjoern A. Zeeb int iwl_mvm_sf_update(struct iwl_mvm *mvm, struct ieee80211_vif *changed_vif,
225bfcc09ddSBjoern A. Zeeb 		      bool remove_vif)
226bfcc09ddSBjoern A. Zeeb {
227bfcc09ddSBjoern A. Zeeb 	enum iwl_sf_state new_state;
228bfcc09ddSBjoern A. Zeeb 	struct iwl_mvm_vif *mvmvif = NULL;
229bfcc09ddSBjoern A. Zeeb 	struct iwl_mvm_active_iface_iterator_data data = {
230bfcc09ddSBjoern A. Zeeb 		.ignore_vif = changed_vif,
231bfcc09ddSBjoern A. Zeeb 		.sta_vif_state = SF_UNINIT,
232bfcc09ddSBjoern A. Zeeb 	};
2339af1bba4SBjoern A. Zeeb 	struct ieee80211_sta *sta = NULL;
234bfcc09ddSBjoern A. Zeeb 
235*a4128aadSBjoern A. Zeeb 	if (fw_has_api(&mvm->fw->ucode_capa,
236*a4128aadSBjoern A. Zeeb 		       IWL_UCODE_TLV_API_SMART_FIFO_OFFLOAD))
237*a4128aadSBjoern A. Zeeb 		return 0;
238bfcc09ddSBjoern A. Zeeb 	/*
239bfcc09ddSBjoern A. Zeeb 	 * Ignore the call if we are in HW Restart flow, or if the handled
240bfcc09ddSBjoern A. Zeeb 	 * vif is a p2p device.
241bfcc09ddSBjoern A. Zeeb 	 */
242bfcc09ddSBjoern A. Zeeb 	if (test_bit(IWL_MVM_STATUS_IN_HW_RESTART, &mvm->status) ||
243bfcc09ddSBjoern A. Zeeb 	    (changed_vif && changed_vif->type == NL80211_IFTYPE_P2P_DEVICE))
244bfcc09ddSBjoern A. Zeeb 		return 0;
245bfcc09ddSBjoern A. Zeeb 
246bfcc09ddSBjoern A. Zeeb 	ieee80211_iterate_active_interfaces_atomic(mvm->hw,
247bfcc09ddSBjoern A. Zeeb 						   IEEE80211_IFACE_ITER_NORMAL,
248bfcc09ddSBjoern A. Zeeb 						   iwl_mvm_bound_iface_iterator,
249bfcc09ddSBjoern A. Zeeb 						   &data);
250bfcc09ddSBjoern A. Zeeb 
251bfcc09ddSBjoern A. Zeeb 	/* If changed_vif exists and is not to be removed, add to the count */
252bfcc09ddSBjoern A. Zeeb 	if (changed_vif && !remove_vif)
253bfcc09ddSBjoern A. Zeeb 		data.num_active_macs++;
254bfcc09ddSBjoern A. Zeeb 
255bfcc09ddSBjoern A. Zeeb 	switch (data.num_active_macs) {
256bfcc09ddSBjoern A. Zeeb 	case 0:
257bfcc09ddSBjoern A. Zeeb 		/* If there are no active macs - change state to SF_INIT_OFF */
258bfcc09ddSBjoern A. Zeeb 		new_state = SF_INIT_OFF;
259bfcc09ddSBjoern A. Zeeb 		break;
260bfcc09ddSBjoern A. Zeeb 	case 1:
261bfcc09ddSBjoern A. Zeeb 		if (remove_vif) {
262bfcc09ddSBjoern A. Zeeb 			/* The one active mac left is of type station
263bfcc09ddSBjoern A. Zeeb 			 * and we filled the relevant data during iteration
264bfcc09ddSBjoern A. Zeeb 			 */
265bfcc09ddSBjoern A. Zeeb 			new_state = data.sta_vif_state;
2669af1bba4SBjoern A. Zeeb 			sta = data.sta_vif_ap_sta;
267bfcc09ddSBjoern A. Zeeb 		} else {
268bfcc09ddSBjoern A. Zeeb 			if (WARN_ON(!changed_vif))
269bfcc09ddSBjoern A. Zeeb 				return -EINVAL;
270bfcc09ddSBjoern A. Zeeb 			if (changed_vif->type != NL80211_IFTYPE_STATION) {
271bfcc09ddSBjoern A. Zeeb 				new_state = SF_UNINIT;
2729af1bba4SBjoern A. Zeeb 			} else if (changed_vif->cfg.assoc &&
273bfcc09ddSBjoern A. Zeeb 				   changed_vif->bss_conf.dtim_period) {
274bfcc09ddSBjoern A. Zeeb 				mvmvif = iwl_mvm_vif_from_mac80211(changed_vif);
2759af1bba4SBjoern A. Zeeb 				sta = mvmvif->ap_sta;
276bfcc09ddSBjoern A. Zeeb 				new_state = SF_FULL_ON;
277bfcc09ddSBjoern A. Zeeb 			} else {
278bfcc09ddSBjoern A. Zeeb 				new_state = SF_INIT_OFF;
279bfcc09ddSBjoern A. Zeeb 			}
280bfcc09ddSBjoern A. Zeeb 		}
281bfcc09ddSBjoern A. Zeeb 		break;
282bfcc09ddSBjoern A. Zeeb 	default:
283bfcc09ddSBjoern A. Zeeb 		/* If there are multiple active macs - change to SF_UNINIT */
284bfcc09ddSBjoern A. Zeeb 		new_state = SF_UNINIT;
285bfcc09ddSBjoern A. Zeeb 	}
2869af1bba4SBjoern A. Zeeb 
2879af1bba4SBjoern A. Zeeb 	return iwl_mvm_sf_config(mvm, sta, new_state);
288bfcc09ddSBjoern A. Zeeb }
289