On 10/11/2010 07:48 PM, Bruno Randolf wrote: > On Tue October 12 2010 11:10:43 Ben Greear wrote: >> On 10/11/2010 06:44 PM, Bruno Randolf wrote: >>> On Sat October 9 2010 04:01:15 gree...@candelatech.com wrote: >>>> + >>>> + /* Calculate combined mode - when APs are active, operate in AP mode. >>>> + * Otherwise use the mode of the new interface. This can currently >>>> + * only deal with combinations of APs and STAs. Only one ad-hoc >>>> + * interfaces is allowed above. >>> >>> the comment reference to "above" does not make sense here any more. >> >> This patch is already in..but I can send a followup cleanup patch. > > just for the comment it's probably not worth it, but... > >>>> ieee80211_vif *vif) +static void ath_do_set_opmode(struct ath5k_softc >>>> *sc) >>>> +{ >>>> + struct ath5k_hw *ah = sc->ah; >>>> + ath5k_hw_set_opmode(ah, sc->opmode); >>>> + ATH5K_DBG(sc, ATH5K_DEBUG_MODE, "mode setup opmode %d (%s)\n", >>>> + sc->opmode, >>>> + ath_opmode_to_string(sc->opmode) ? >>>> + ath_opmode_to_string(sc->opmode) : "UKNOWN"); >>>> +} >>> >>> what's the point of this function? just to add the debug print? >> >> Yes. At one point, I had it being called from several places..but now it's >> called from only one place currently, I think. I could make it explicitly >> inline code if someone cared. > > i think i do care. could you please just open-code it at the one place where > you call this function?
I just posted a patch to do these two things. Thanks for the suggestions! Ben -- Ben Greear <gree...@candelatech.com> Candela Technologies Inc http://www.candelatech.com _______________________________________________ ath5k-devel mailing list ath5k-devel@lists.ath5k.org https://lists.ath5k.org/mailman/listinfo/ath5k-devel