Re: [RFT 2/2] cfg80211: add 5 GHz UNII band regulatory hints

["Arend van Spriel" <arend@broadcom.com>]

On 08/11/2012 02:25 AM, Luis R. Rodriguez wrote:
> +static void reg_unii_5ghz_beacon_hint_unii(struct wiphy *wiphy,
> +					   struct ieee80211_channel *chan,
> +					   enum reg_5ghz_unii_band unii_band)
> +{
> +	struct ieee80211_supported_band *sband;
> +	unsigned int i;
> +
> +	assert_cfg80211_lock();
> +
> +	sband = wiphy->bands[chan->band];
> +
> +	/*
> +	 * Only process UNII 1 hints for UNII 1 channels
> +	 * Only process UNII 3 hints for UNII 3 channels
> +	 */
> +	for (i = 0; i < sband->n_channels; i++) {
> +		struct ieee80211_channel *c = &sband->channels[i];
> +
> +		switch (unii_band) {
> +		case REG_5GHZ_UNII_1:
> +			if (!reg_is_5ghz_unii_1(c))
> +				continue;
> +			break;
> +		case REG_5GHZ_UNII_3:
> +			if (!reg_is_5ghz_unii_3(c))
> +				continue;
> +			break;
> +		default:
> +			WARN_ONCE(1, "unexpected regulatory unii hint\n");
> +			continue;
> +		}
> +
> +		reg_chan_process_beacon_hint(wiphy, chan);

Not sure, but reading the description I think variable 'c' should be
passed here instead of 'chan'.

> +	}

Gr. AvS