On Fri, 22 Jan 2010 10:34:38 +0100, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Thu, 2010-01-21 at 18:06 -0500, Luis R. Rodriguez wrote: > >> +#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)) >> +static bool ieee80211_all_queues_stopped(struct ieee80211_hw *hw) > > You want "all queues started" :) > Are you certain of this? I stared and stared trying to see what case the function would behave as a "started" check. It loops through all queues, returning false if it finds any started queues. If it makes it through all the queues and hasn't found a single started queue, it returns true (to indicate all queues are stopped). Or do you mean, as a second function? >> +{ >> + unsigned int queue; >> + >> + for (queue = 0; queue < hw->queues; queue++) >> + if (!ieee80211_queue_stopped(hw, queue)) >> + return false; >> + return true; >> +} >> +#endif >> + > johannes Pat -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html