On Fri, 2012-04-06 at 13:35 -0700, Thomas Pedersen wrote: > /* > + * cfg80211_ch_switch_notify - update wdev channel and notify userspace > + * @dev: the device which switched channels > + * @freq: new channel frequency (in MHz) > + * @type: channel type > + * > + * Acquires wdev_lock, so must only be called from sleepable driver context! > + */ > +void cfg80211_ch_switch_notify(struct net_device *dev, int freq, > + enum nl80211_channel_type type); I think this could be a bit more detailed -- the driver authors won't know what wdev_lock is (nor care), but the other consequence of it is that the function absolutely must not be called in the context of a cfg80211 call into the driver. 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
