On Thu, 2021-08-26 at 10:20 +0200, Johannes Berg wrote: > > struct cfg80211_chan_def { > > struct ieee80211_channel *chan; > > @@ -684,6 +685,7 @@ struct cfg80211_chan_def { > > u32 center_freq2; > > struct ieee80211_edmg edmg; > > u16 freq1_offset; > > + enum nl80211_ap_reg_power power_type; > > I'm not sure why this should be in the chandef, there's no way that > anything in cfg80211 is ever using it there, at least in your patches. Does it even *apply* to a channel? What if I'm connecting to two APs on the same channel (two client interfaces)? johannes