On Wed, 2018-01-17 at 13:01 +0530, [email protected] 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

Reply via email to