On Fri, 2012-05-18 at 14:03 +0200, Michal Kazior wrote: > /** > + * struct ieee80211_channel_context - channel context that vifs may be tuned to > + * > + * @channel: the channel to tune to > + * @channel_type: the channel (HT) type > + * @vif_list: vifs bound to channel context > + */ > +struct ieee80211_channel_context { > + struct ieee80211_channel *channel; > + enum nl80211_channel_type channel_type; > + > + struct list_head vif_list; > +}; A few trivial comments on this particular struct: I think we should have an internal and an external struct, so that the list for example isn't accessible by drivers. OTOH, maybe it should be accessible? The other request I have would be for private space in here for the driver, like inside vif or hw structs. Anyway, I'll look over this in more detail, but might not get to it before the week after next. 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