On 10/17/2018 3:37 PM, Johannes Berg wrote:
[...]
Looks mostly good to me, some minor comments below...
> +/**
> + * struct cfg80211_pmsr_ftm_result - FTM result
> + * @failure_reason: if this measurement failed (PMSR status is
> + * %NL80211_PMSR_STATUS_FAILURE), this gives a more precise
> + * reason than just "failure"
> + * @burst_index: if reporting partial results, this is the index
> + * in [0 .. num_bursts-1] of the burst that's being reported
> + * @num_ftmr_attempts: number of FTM request frames transmitted
> + * @num_ftmr_successes: number of FTM request frames acked
> + * @busy_retry_time: if failure_reason is
> %NL80211_PMSR_FTM_FAILURE_PEER_BUSY,
> + * fill this to indicate in how many seconds a retry is deemed possible
> + * by the responder
> + * @num_bursts_exp: actual number of bursts exponent negotiated
> + * @burst_duration: actual burst duration negotiated
> + * @ftms_per_burst: actual frames per burst negotiated
actual frames -> actual ftms
> +/**
> + * struct cfg80211_pmsr_ftm_request_peer - FTM request data
> + * @requested: indicates FTM is requested
> + * @preamble: frame preamble to use
> + * @burst_period: burst period to use
> + * @asap: indicates to use ASAP mode
> + * @num_bursts_exp: number of bursts exponent
> + * @burst_duration: burst duration
> + * @ftms_per_burst: number of frames per burst
frames -> ftms
>
> +/**
> + * struct cfg80211_pmsr_capabilities - cfg80211 peer measurement capabilities
> + * @max_peers: maximum number of peers in a single measurement
> + * @report_ap_tsf: can report assoc AP's TSF for radio resource measurement
> + * @randomize_mac_addr: can randomize MAC address for measurement
> + * @ftm.supported: FTM measurement is supported
> + * @ftm.asap: ASAP-mode is supported
> + * @ftm.non_asap: non-ASAP-mode is supported
> + * @ftm.request_lci: can request LCI data
> + * @ftm.request_civicloc: can request civic location data
> + * @ftm.preambles: bitmap of preambles supported (&enum nl80211_preamble)
> + * @ftm.bandwidths: bitmap of bandwidths supported (&enum nl80211_chan_width)
> + * @ftm.max_bursts_exponent: maximum burst exponent supported
> + * (set to 0 if not limited; note that setting this will necessarily
> + * forbid using the value 15 to let the responder pick)
How can driver specify it only supports a single burst? (This is what our
internal wil6210 implementation currently supports...)
Thanks,
Lior
