Re: [PATCH] wireless: Add channel/frequency conversions to ieee80211.h

[Dave <kilroyd@googlemail.com>]

Richard Farina wrote:
> David Kilroy wrote:
>> Added mappings for FHSS, DSSS and OFDM channels - with macros to point
>> HR DSSS and ERP to the DSSS mappings. Currently just static inline
>> functions.

>> +/**
>> + * ieee80211_ofdm_chan_to_freq - get channel center frequency
>> + * @s_freq: starting frequency == (dotChannelStartingFactor/2) MHz
>> + * @channel: the OFDM channel
>> + *
>> + * Convert IEEE802.11 OFDM channel to center frequency (MHz)
>> + * Ref IEEE 802.11-2007 section 17.3.8.3.2
>> + */
>> +static inline int ieee80211_ofdm_chan_to_freq(int s_freq, int channel)
>> +{
>> +	if ((channel > 0) && (channel <= 200) &&
>> +	    (s_freq >= 4000))
>> +		return s_freq + (channel * 5);
>> +	else
>> +		return -1;
>> +}

> Any desire to make 184-196 work? These are the 4.9 GHz channels.

Do you see a reason why these channels wouldn't work with these
conversion routines? Or are you asking about corresponding driver support?

>From the ack I assume the latter. My main motivation is to put the
frequency conversions in a common place. In this case for old drivers
which only use the 2.4 GHz band. The 5GHz conversions are just for
completeness.

If drivers don't currently support those channels, I suspect there's a
more significant issue to be addressed first.

> I checked this all out pretty specifically, seems to work well. As if 
> anyone cares but:
> 
> ACKED-By: Rick Farina

Thanks for having a look.



Dave.