Re: [PATCH v5] cfg80211: Provision to allow the support for different beacon intervals

[Johannes Berg <johannes@sipsolutions.net>]

> +static bool diff_beacon_interval_supported(struct wiphy *wiphy, u16
> types,
> +					   u32 beacon_int)
> +{
> +	const struct ieee80211_iface_combination *c;
> +	u16 all_types;
> +	int i, j;
> +
> +	for (i = 0; i < wiphy->n_iface_combinations; i++) {
> +		c = &wiphy->iface_combinations[i];
> +
> +		if (!c->diff_beacon_int_gcd ||
> +		    (beacon_int % c->diff_beacon_int_gcd))
> +			continue;
> +
> +		all_types = 0;
> +		for (j = 0; j < c->n_limits; j++)
> +			all_types |= c->limits[j].types;
> +
> +		if (all_types & types)
> +			return true;
> +	}
> +
> +	return false;
> +}

This seems odd. Shouldn't it use cfg80211_iter_combinations()? Might
need some additional helper to understand which interface types are
used right now - seems we mostly moved that to mac80211, but for this
case it should probably stay in cfg80211? Or should it?

johannes