On Tue, 2023-03-28 at 17:09 -0700, Aloka Dixit wrote: > + switch (wdev->iftype) { > + case NL80211_IFTYPE_AP: > + width = wdev->u.ap.preset_chandef.width; This seems a bit awkward and annoying, it means that we have to keep using the API that sets the preset_chandef first, and it also means it won't work for multi-link. Couldn't you link it with the config in start_ap/join_mesh? And per-link config in set_tx_bitrate_mask()? Also not sure about this at all in parse_tid_conf()? johannes