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

_______________________________________________
Devel mailing list
[email protected]
http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel

Reply via email to