On Wed, 2018-01-17 at 13:01 +0530, tamizhr@xxxxxxxxxxxxxx wrote: > > +static void ieee80211_update_rssi_config(struct ieee80211_sub_if_data *sdata) [...] > + /* if AP, always tell the driver by checking beacon status */ > + if ((sdata->vif.type == NL80211_IFTYPE_AP || > + sdata->vif.type == NL80211_IFTYPE_AP_VLAN || > + sdata->vif.type == NL80211_IFTYPE_P2P_GO) && > + !sdata->vif.bss_conf.enable_beacon) nit: that line with !sdata should be indented one less, it doesn't belong into the parenthesized expression. Again though, this relies on a driver feature bit to not confuse the driver. johannes
