hi,
On Tue, Mar 18, 2014 at 3:53 PM, Michal Kazior <michal.kazior@xxxxxxxxx> wrote:
> The utility function has no uses yet. It is aimed
> at future chanctx reservation management and
> channel switching.
>
> Signed-off-by: Michal Kazior <michal.kazior@xxxxxxxxx>
> ---
[...]
> +int ieee80211_max_num_channels(struct ieee80211_local *local)
> +{
> + struct ieee80211_sub_if_data *sdata;
> + int num[NUM_NL80211_IFTYPES] = {};
> + struct ieee80211_chanctx *ctx;
> + int num_different_channels = 0;
> + u8 radar_detect = 0;
> + u32 max_num_different_channels = 1;
> + int err;
> +
> + lockdep_assert_held(&local->chanctx_mtx);
> +
> + list_for_each_entry(ctx, &local->chanctx_list, list) {
> + num_different_channels++;
> +
this doesn't seem correct - i think we can have multiple channel
contexts with the same channel (e.g. due to exclusive mode)?
Eliad.
--
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
