[snip] I think this makes sense, but it's not clear to me why you add two counters and keep the old one? It seems to me that it would be sufficient to have a single counter per AP/VLAN interface? The usage in __ieee80211_request_smps_ap() can just be removed since it goes to iterate the stations next. That should be a separate, first, patch in the series, but after that I don't see a need to keep num_mcast_sta, or rather, I see no reason not to remove the VLAN stations from the AP's num_mcast_sta, and add a new per-VLAN num_mcast_sta. > +/** > + * @returns number of multicast stations connected > + * -1 if unsupported (no-AP, 4addr mode) > + */ > +static inline int > +ieee80211_vif_get_num_mcast_if(struct ieee80211_sub_if_data *sdata) That's not a valid kernel-doc comment, but you've tagged it as one with the /** - please fix by removing the /** and the @, and writing a real sentence out of that, or by making it a real kernel-doc comment. johannes
