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