On Thu, 2009-07-30 at 17:53 +0400, Igor Perminov wrote:
> @@ -245,6 +247,14 @@ static int ieee80211_open(struct net_device *dev)
>
> ieee80211_start_mesh(sdata);
> }
> + else if (sdata->vif.type == NL80211_IFTYPE_AP) {
Put 'else if' on the same line as '}' please
> + if (sdata->u.mntr_flags & MONITOR_FLAG_CONTROL) {
> + local->fif_pspoll--;
> local->fif_control--;
> + }
Yeah, that's exactly what I was thinking of :)
> @@ -497,6 +509,13 @@ static int ieee80211_stop(struct net_device *dev)
> }
> /* fall through */
> default:
> + if (sdata->vif.type == NL80211_IFTYPE_AP) {
I'd probably put this after a
case NL80211_IFTYPE_AP:
label but you need the if () anyway ... hmm I guess we really should
clean up this code and split it into two switch statements...
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
