On Thursday 15 February 2007 15:15, Jiri Benc wrote: > > + local->hw.conf.channel = chan->chan; > > + local->hw.conf.channel_val = chan->val; > > + local->hw.conf.power_level = chan->power_level; > > + local->hw.conf.freq = chan->freq; > > + local->hw.conf.phymode = mode->mode; > > + local->hw.conf.antenna_max = chan->antenna_max; > > What about passing ieee80211_channel and ieee80211_hw_mode structures > instead of a ton of variables? (Just an idea, not a problem with the > patch.) > I was thinking that too, but I was trying to avoid driver api changes. I might make a patch for it later if it still makes sense then. > > + if (local->curr_rates != mode->rates) > > + rate_control_clear(local); > > + local->curr_rates = mode->rates; > > + local->num_curr_rates = mode->num_rates; > > + ieee80211_prepare_rates(local); > > This will trigger rate control reinitialization when scanning on abg > cards. It's needed but not obvious at first sight. Perhaps some comment > would be useful here? (Again, not a problem with the patch, just > something I realized looking at the patch and thinking why the hell do > we do the reinitialization here?) > Yeah, this is why rate control never seems to work right on my system - NetworkManager keeps scanning and resetting the rate control. I'm not sure exactly how to fix it, so I just left it.. -Michael Wu
Attachment:
pgpPtI9G3b8sp.pgp
Description: PGP signature
