Adding wireless-regdb. Regulatory folks: if two cards are present on a system, in the worst case consider two different cards for AP mode, and one has a DFS region set for the country its on but the other does not, do we want to use the DFS region for both? DFS would not be allowed on system unless the DFS region is set. DFS operation requires a card to explicitly support DFS though so even though it can be set as an intersection each card would still require DFS suport for that region. As I see it this will depend on what we want cards to do if the DFS region is unknown for a region. If the DFS region is not known can we use any DFS algorithm? If we cannot then I think a DFS intersection would require agreement on the DFS region. That would also mean though that when shipping products if a system is built with one card that has DFS for ETSI for example, and then a secondary card is present and its regulatory domain does not have DFS then the first card would not be able to operate on the DFS. I think this is reasonable given that the two cards must at least agree on the regulatory domain, otherwise the folks doing system integration probably did a bad job at thinking of things ahead of time. Even though this can be technically true I foresee folks this misconfiguration happening in the future and folks beingp puzzled by this as an issue. This means this should be documented for folks selling devices in a combined wifi system. As such NACK for now. Luis On Wed, Jun 11, 2014 at 01:08:34PM -0700, greearb@xxxxxxxxxxxxxxx wrote: > From: Ben Greear <greearb@xxxxxxxxxxxxxxx> > > If one is UN-SET, use the other. Seems this would > be more correct that what we have now. > > Signed-off-by: Ben Greear <greearb@xxxxxxxxxxxxxxx> > --- > > Someone that understands this code better than I > do should review this well before it is considered > for upstream. I'm not sure it is actually needed. > > net/wireless/reg.c | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/net/wireless/reg.c b/net/wireless/reg.c > index 785e516..b87035f 100644 > --- a/net/wireless/reg.c > +++ b/net/wireless/reg.c > @@ -727,8 +727,15 @@ static enum nl80211_dfs_regions > reg_intersect_dfs_region(const enum nl80211_dfs_regions dfs_region1, > const enum nl80211_dfs_regions dfs_region2) > { > - if (dfs_region1 != dfs_region2) > + if (dfs_region1 != dfs_region2) { > + pr_info("intersect-dfs-region, region1: %d region2: %d\n", > + dfs_region1, dfs_region2); > + if (dfs_region1 == NL80211_DFS_UNSET) > + return dfs_region2; > + if (dfs_region2 == NL80211_DFS_UNSET) > + return dfs_region1; > return NL80211_DFS_UNSET; > + } > return dfs_region1; > } > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html