Re: [PATCH 6/7] cfg80211: Accept multiple RSSI threholds for STA_MON command

2018-07-06 Thread Johannes Berg
On Thu, 2018-07-05 at 13:07 +0530, Tamizh chelvam wrote: > First I've done patch to have single threshold value for AP mode and as > per the previous discussion extended to multiple thresholds also. Here > the intention is to accept one or more than one RSSI thresholds to > monitor. Any thought

Re: [PATCH 6/7] cfg80211: Accept multiple RSSI threholds for STA_MON command

2018-07-05 Thread Tamizh chelvam
On 2018-06-29 15:09, Johannes Berg wrote: 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 cfg8

Re: [PATCH 6/7] cfg80211: Accept multiple RSSI threholds for STA_MON command

2018-06-29 Thread Johannes Berg
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_

[PATCH 6/7] cfg80211: Accept multiple RSSI threholds for STA_MON command

2018-06-13 Thread Tamizh chelvam
Change the NL80211_CMD_STA_MON RSSI threshold attribut to accept any number of thresholds as a sorted array. If user send the configuration with single RSSI threshold then the old mechanism is enabled. Same netlink event will be generated in both cases. This patch introduced set_sta_mon_rssi_range_