mac80211: fix parameter reading for AC_BE tx bursting
authorAlberto Martinez-Alvarez <amteza@gmail.com>
Fri, 5 Aug 2022 21:10:54 +0000 (07:10 +1000)
committerChristian Lamparter <chunkeey@gmail.com>
Sat, 24 Sep 2022 15:41:38 +0000 (17:41 +0200)
The "tx_burst" option which should control the value was
expecting more of a list and hence tx_queue_data2_burst
value wasn't updated.

Yes, it would make sense to have a list for this, the
existing  code only updates tx_queue_data2_burst and
not the other tx_queue_data[0134]_burst values.

Signed-off-by: Alberto Martinez-Alvarez <amteza@gmail.com>
(formatted commit message, wrote extra information into commit,
moved tx_burst to existing json_get_vars)
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh

index f5249408546771e2c0dee27f11578c28444214d1..f462fc8ad9751db436a4bae372d7a7bef469c9ef 100644 (file)
@@ -138,8 +138,8 @@ mac80211_hostapd_setup_base() {
        [ -n "$acs_exclude_dfs" ] && [ "$acs_exclude_dfs" -gt 0 ] &&
                append base_cfg "acs_exclude_dfs=1" "$N"
 
-       json_get_vars noscan ht_coex min_tx_power:0
-       json_get_values ht_capab_list ht_capab tx_burst
+       json_get_vars noscan ht_coex min_tx_power:0 tx_burst
+       json_get_values ht_capab_list ht_capab
        json_get_values channel_list channels
 
        [ "$auto_channel" = 0 ] && [ -z "$channel_list" ] && \