Re: [PATCH] mac80211: Add 802.11h CSA support

[Sujith <m.sujith@gmail.com>]

Johannes Berg wrote:
> On Wed, 2008-12-31 at 15:39 +0530, Sujith wrote:
> > Move to the advertised channel on reception of
> > a CSA element. This is needed for 802.11h compliance.
> > 
> > Signed-off-by: Sujith <Sujith.Manoharan@atheros.com>
> 
> One thing I've been wondering about though, should we have a hook into
> the driver to actually stop transmit at a lower level, where possible?
> For example Broadcom hardware we can actually set some value on the chip
> and then let it reject all frames, drain its FIFOs, and then in the
> driver we could schedule those frames again for afterwards.

I think that is needed only when handling Quiet IE, which I was planning to work
on separately, with a notification to the driver to stop transmission completely for a period of time.

> > +	sdata->local->oper_channel = new_ch;
> 
> Here you're setting oper_channel, but is that really the right thing to
> do? That means any hw config call that comes in between this and later
> will already switch to that channel, and hw config can happen for
> various other reasons, for instance PS mode stuff. I think we may need
> to have a separate variable?

Right, a *switch_to in sdata would suffice, I guess.
I'll send an updated patch.

Sujith