Aloka Dixit <quic_alokad@xxxxxxxxxxx> writes: > FILS discovery and unsolicited broadcast probe response transmissions > are configured as part of NL80211_CMD_START_AP, however both stop > after userspace uses the NL80211_CMD_SET_BEACON command as these > attributes are not processed in that command. > > - Add the missing implementation. > - Modify the local variable in nl80211_set_beacon() and input parameter > to rdev_change_beacon() from type struct cfg80211_beacon_data to > type struct cfg80211_ap_settings to support the new processing. > > Signed-off-by: Aloka Dixit <quic_alokad@xxxxxxxxxxx> > --- > include/net/cfg80211.h | 2 +- > net/wireless/nl80211.c | 28 ++++++++++++++++++---- > net/wireless/rdev-ops.h | 2 +- > net/wireless/trace.h | 52 +++++++++++++++++++++++------------------ > 4 files changed, 55 insertions(+), 29 deletions(-) > > diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h > index 97a5804ccdcf..880ade4a0430 100644 > --- a/include/net/cfg80211.h > +++ b/include/net/cfg80211.h > @@ -4200,7 +4200,7 @@ struct cfg80211_ops { > int (*start_ap)(struct wiphy *wiphy, struct net_device *dev, > struct cfg80211_ap_settings *settings); > int (*change_beacon)(struct wiphy *wiphy, struct net_device *dev, > - struct cfg80211_beacon_data *info); > + struct cfg80211_ap_settings *info); Shouldn't patch 3 folded into patch 1? I don't see how patch 1 as is would compile without warnings. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches