On 08/29/2011 03:23 PM, Jouni Malinen wrote: > +static int ath6kl_mgmt_tx(struct wiphy *wiphy, struct net_device *dev, > + struct ieee80211_channel *chan, bool offchan, > + enum nl80211_channel_type channel_type, > + bool channel_type_valid, unsigned int wait, > + const u8 *buf, size_t len, u64 *cookie) > +{ > + struct ath6kl *ar = ath6kl_priv(dev); > + u32 id; > + > + id = ar->send_action_id++; > + if (id == 0) > + id = ar->send_action_id++; > + > + *cookie = id; > + return ath6kl_wmi_send_action_cmd(ar->wmi, id, chan->center_freq, wait, > + buf, len); > +} I don't understand the "if (id == 0)" check. It would be nice to get a comment giving some background info about the situation. Kalle -- 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