Re: [PATCH 1/2] ath9k: Check for identical channels when changing channels.

[Felix Fietkau <nbd@openwrt.org>]

On 2010-09-28 3:37 PM, Ben Greear wrote:
> On 09/28/2010 01:51 AM, Felix Fietkau wrote:
>> On 2010-09-28 2:07 AM, greearb@candelatech.com wrote:
>>> From: Ben Greear<greearb@candelatech.com>
>>>
>>> Don't do un-needed work.  Should be useful when scanning on
>>> only the working channel.
>>>
>>> Signed-off-by: Ben Greear<greearb@candelatech.com>
>>> ---
>>> :100644 100644 a133878... 4d34ba5... M	drivers/net/wireless/ath/ath9k/main.c
>>>   drivers/net/wireless/ath/ath9k/main.c |    5 +++++
>>>   1 files changed, 5 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
>>> index a133878..4d34ba5 100644
>>> --- a/drivers/net/wireless/ath/ath9k/main.c
>>> +++ b/drivers/net/wireless/ath/ath9k/main.c
>>> @@ -1602,6 +1602,11 @@ static int ath9k_config(struct ieee80211_hw *hw, u32 changed)
>>>   		struct ieee80211_channel *curchan = hw->conf.channel;
>>>   		int pos = curchan->hw_value;
>>>
>>> +		/* If channels are the same, then don't actually do anything.
>>> +		 */
>>> +		if (sc->sc_ah->curchan ==&sc->sc_ah->channels[pos])
>>> +			goto skip_chan_change;
>>> +
>> NACK. This ignores off-channel flag changes, which are important for
>> calibration.
> 
> Is it enough to just add a check to make sure no changes were made to
> the SC_OP_OFFCHANNEL flag?
Shouldn't this check be done in mac80211 then?

- Felix