Re: [PATCH 2/7] cfg80211: Add new NL command to configure peer specific rssi threshold
From: Johannes Berg <johannes@sipsolutions.net>
Date: 2018-07-06 11:40:55
From: Johannes Berg <johannes@sipsolutions.net>
Date: 2018-07-06 11:40:55
On Wed, 2018-07-04 at 11:39 +0530, Tamizh chelvam wrote:
On 2018-06-29 14:59, Johannes Berg wrote:quoted
On Wed, 2018-06-13 at 16:15 +0530, Tamizh chelvam wrote:quoted
+ * @NL80211_ATTR_STA_MON: Station's connection monitor configuration in a + * nested attribute with %NL80211_ATTR_STA_MON_* sub-attributes.Can't we reuse the existing attributes in this nesting?This new attributes introduced as per the previous discussion. https://patchwork.kernel.org/patch/10240697/ Is my understanding correct ?
Yeah ok ... I'm thinking perhaps we could share some part of it, but it's not really worth making it more complex due to that.
quoted
Wait, you went to all this effort in the first patch and now you don't add this to the list?No, the list and the rssi_config structure is for multi threshold value and not for single threshold.
Yeah, though I'm still not sure why we need the single threshold thing at all. List should always be a superset. johannes