On Wed, 2018-06-13 at 16:15 +0530, Tamizh chelvam wrote: > Change the NL80211_CMD_STA_MON RSSI threshold attribut to This seems weird - you just introduced it a few patches back, and now you change it and even worry about compatibility and have both cfg80211 callbacks etc. Just remove set_sta_mon_rssi_config() and ask that drivers/mac80211 implement set_sta_mon_rssi_range_config? > * @NL80211_EXT_FEATURE_STA_MON_RSSI_CONFIG: With this driver can set > * rssi threshold using %NL80211_ATTR_STA_MON_RSSI_THOLD attribute > * for a connected station. > + * @NL80211_EXT_FEATURE_STA_MON_RSSI_LIST: With this driver the > + * %NL80211_ATTR_STA_MON_RSSI_THOLD attribute accepts a list of zero or > + * more RSSI threshold values to monitor rather than exactly one threshold. And maybe not have two bits here either? Or do you expect somebody to actually really need the single threshold in the near future? It seems you're implementing it for mac80211 only, which doesn't care. johannes
