On Mon, Nov 18, 2019 at 11:15:40PM +0100, Markus Theil wrote: > +static void mt76x2u_channel_switch_beacon(struct ieee80211_hw *hw, > + struct ieee80211_vif *vif, > + struct cfg80211_chan_def *chandef) > +{ > +} > + > const struct ieee80211_ops mt76x2u_ops = { > .tx = mt76x02_tx, > .start = mt76x2u_start, > @@ -121,4 +127,5 @@ const struct ieee80211_ops mt76x2u_ops = { > .get_survey = mt76_get_survey, > .set_tim = mt76_set_tim, > .release_buffered_frames = mt76_release_buffered_frames, > + .channel_switch_beacon = mt76x2u_channel_switch_beacon, Is this needed ? Seems mac80211 check against this op being NULL in drv_channel_switch_beacon() and it is not used otherwise. Stanislaw