Re: [PATCH] mac80211: support FTM responder configuration/statistics

[Pradeep Kumar Chitrapu <pradeepc@xxxxxxxxxxxxxx>]

>  	memset(&params, 0, sizeof(params));
> +	params.beacon_csa.ftm_responder = -1;
Hi Johannes,

Agree with the rest, however, I think this may not be needed because 
ftm_responder is already being set to -1,
if NL80211_ATTR_FTM_RESPONDER attribute is not included, in 
nl80211_parse_beacon, which will be called from
nl80211_channel_switch. Please let me know if I may have missed 
something..

        if (attrs[NL80211_ATTR_FTM_RESPONDER]) {

        } else {
                bcn->ftm_responder = -1;
        }

>  	if (!info->attrs[NL80211_ATTR_WIPHY_FREQ] ||
>  	    !info->attrs[NL80211_ATTR_CH_SWITCH_COUNT])
>
>
> If that seems good to you I'll submit the patch.
Sure..Thanks.
> johannes