On 1-3-2016 9:42, Eliad Peller wrote: > On Mon, Feb 29, 2016 at 11:44 AM, Arend van Spriel <arend@xxxxxxxxxxxx> wrote: >> Introducing a new feature that the driver can use to >> indicate the driver/firmware supports configuration of BSS >> selection criteria upon CONNECT command. This can be useful >> when multiple BSS-es are found belonging to the same ESS, >> ie. Infra-BSS with same SSID. The criteria can then be used to >> offload selection of a preferred BSS. >> > [...] > >> + /* only do bss selection when no BSSID is specified. */ >> + if (!connect.bssid && info->attrs[NL80211_ATTR_BSS_SELECT]) { >> + if (connect.bssid) { > > i guess you wanted to remove the !connect.bssid condition in the outer if :) > other than that, looks go to me. Good guess. Will fix that. Regards, Arend -- 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