On Fri, 2019-04-26 at 21:50 +0200, John Crispin wrote: > On 26/04/2019 11:41, John Crispin wrote: > > @@ -6105,6 +6106,7 @@ static int nl80211_set_bss(struct sk_buff *skb, struct genl_info *info) > > params.ht_opmode = -1; > > params.p2p_ctwindow = -1; > > params.p2p_opp_ps = -1; > > + params.twt_responder = -1; > > > > if (info->attrs[NL80211_ATTR_BSS_CTS_PROT]) > > params.use_cts_prot = > > @@ -6149,6 +6151,10 @@ static int nl80211_set_bss(struct sk_buff *skb, struct genl_info *info) > > return -EINVAL; > > } > > > > + if (info->attrs[NL80211_ATTR_TWT_RESPONDER]) > > + params.twt_responder = > > + nla_get_u8(info->attrs[NL80211_ATTR_TWT_RESPONDER]); > > + > > if (!rdev->ops->change_bss) > > return -EOPNOTSUPP; > > Hi, > > this should probably be moved to nl80211_start_ap() instead of > nl80211_set_bss() as we probably dont want to change this at runtime ? I have no idea, can we change it at runtime? Is it a capability or an operational state? johannes
