On Mon, 2012-04-30 at 14:20 -0700, Ashok Nagarajan wrote:

+/** mesh_set_ht_prot_mode - set correct HT protection mode

I think you need a newline before the function name, but this isn't
really valid kernel-doc anyway so please just don't use **

+ *
+ * Section 9.23.3.5 of IEEE 80211s standard

There's no 80211s standard. 802.11s is an amendment, please correct the
reference to the -2012 version.

+	rcu_read_lock();
+	list_for_each_entry_rcu(sta, &local->sta_list, list) {
+		if (sdata == sta->sdata &&
+		    sta->plink_state == NL80211_PLINK_ESTAB) {

there would be less indentation if you'd add a continue for the mismatch
cases :-)

johannes