On Mon, 2014-01-06 at 19:01 +0100, Karl Beldan wrote:
> + if (vif->csa_active) {
> + vp->csa_bcn_cnt++;
I don't see any readers of this variable?
> + if (vp->csa_finished) {
> + wiphy_debug(hw->wiphy,"%s extra CSA-beacon\n", __func__);
> + } else if (ieee80211_csa_is_complete(vif)) {
> + wiphy_debug(hw->wiphy,
> + "%s CSA complete after %d beacons\n",
> + __func__, vp->csa_bcn_cnt);
except for the debug, but is that really useful?
> + ieee80211_csa_finish(vif);
> + vp->csa_finished = true;
and if we remove the other debug we can also remove the csa_finished
variable, that seems reasonable to me since mac80211 would already warn
anyway, no?
johannes
--
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
