Re: [PATCH v2] rt2x00: fix monitor mode regression

[Stanislaw Gruszka <sgruszka@redhat.com>]

On Thu, Jan 14, 2016 at 12:07:12AM +0800, Eli Cooper wrote:
> Since commit df1404650ccbfeb76a84f301f22316be0d00a864 monitor mode for rt2x00
> has been made effectively useless because the hardware filter is configured to
> drop packets whose intended recipient is not the device, regardless of the
> presence of monitor mode interfaces.
> 
> This patch fixes this regression by adding explicit monitor mode support, and
> configuring the hardware filter accordingly.
> 
> Signed-off-by: Eli Cooper <elicooper@gmx.com>
[snip]
>  	unsigned int intf_associated;
>  	unsigned int intf_beaconing;
> +	bool is_monitoring;

Could you change to use rt2x00dev->flags instead of new bool
variable i.e. introduce CONFIG_MONITORING in rt2x00_state_flags ?

> +	if (ieee80211_flags & IEEE80211_CONF_CHANGE_MONITOR) {
> +		if (conf->flags & IEEE80211_CONF_MONITOR) {
> +			rt2x00_dbg(rt2x00dev, "Monitor mode is enabled\n");

I don't think those debug messages are actually needed.

Thanks
Stanislaw