On Fri, 2013-10-11 at 11:29 -0700, Chun-Yeow Yeoh wrote: > /* Channel Switching Support */ > + struct cfg80211_csa_settings *csa_settings; That should have an __rcu annotation now, I believe? > + if (ifmsh->csa_settings) { In which case this will result in a sparse warning. Does this even make sense here - it can't ever be non-NULL in this patch? 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