wifi: cfg80211: extend PMSR FTM response for proximity ranging

Applications need negotiated session parameters to interpret
proximity ranging results and perform post-processing. Currently,
the FTM response lacks LTF repetition counts, time constraints,
spatial stream configuration, and availability window parameters.

Extend the FTM response structure to report these negotiated
parameters, enabling applications to track session configuration
and use them in post-processing to increase ranging precision.

Signed-off-by: Peddolla Harshavardhan Reddy <peddolla.reddy@oss.qualcomm.com>
Link: https://patch.msgid.link/20260420090856.2152905-8-peddolla.reddy@oss.qualcomm.com
Signed-off-by: Johannes Berg <johannes.berg@intel.com>

diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h
index 28923539456e61ff8b290ab112b2d9b44f03b510..ad32353e87e054119853b20b0c08ae03cb279eec 100644 (file)
--- a/include/net/cfg80211.h
+++ b/include/net/cfg80211.h
@@ -4436,6 +4436,25 @@ struct cfg80211_ftm_responder_stats {
  *     (must have either this or @rtt_avg)
  * @dist_variance: variance of distances measured (see also @rtt_variance)
  * @dist_spread: spread of distances measured (see also @rtt_spread)
+ * @tx_ltf_repetition_count: negotiated value of number of tx ltf repetitions
+ *     in NDP frames
+ * @rx_ltf_repetition_count: negotiated value of number of rx ltf repetitions
+ *     in NDP frames
+ * @max_time_between_measurements: the negotiated maximum interval (in units of
+ *     10 ms) by which the ISTA must complete the next measurement cycle.
+ * @min_time_between_measurements: the negotiated minimum interval (in units of
+ *     100 us) between two consecutive range measurements initiated by the
+ *     ISTA.
+ * @num_tx_spatial_streams: number of Tx space-time streams used in the NDP
+ *     frame during the measurement sounding phase.
+ * @num_rx_spatial_streams: number of Rx space-time streams used in the NDP
+ *     frame during the measurement sounding phase.
+ * @nominal_time: negotiated nominal duration between adjacent availability
+ *     windows in units of milliseconds (u32).
+ * @availability_window: negotiated availability window time used in this
+ *     session in units of milliseconds (u8).
+ * @chan_width: band width used for measurement.
+ * @preamble: preamble used for measurement.
  * @num_ftmr_attempts_valid: @num_ftmr_attempts is valid
  * @num_ftmr_successes_valid: @num_ftmr_successes is valid
  * @rssi_avg_valid: @rssi_avg is valid
@@ -4448,6 +4467,18 @@ struct cfg80211_ftm_responder_stats {
  * @dist_avg_valid: @dist_avg is valid
  * @dist_variance_valid: @dist_variance is valid
  * @dist_spread_valid: @dist_spread is valid
+ * @tx_ltf_repetition_count_valid: @tx_ltf_repetition_count is valid
+ * @rx_ltf_repetition_count_valid: @rx_ltf_repetition_count is valid
+ * @max_time_between_measurements_valid: @max_time_between_measurements is valid
+ * @min_time_between_measurements_valid: @min_time_between_measurements is valid
+ * @num_tx_spatial_streams_valid: @num_tx_spatial_streams is valid
+ * @num_rx_spatial_streams_valid: @num_rx_spatial_streams is valid
+ * @nominal_time_valid: @nominal_time is valid
+ * @availability_window_valid: @availability_window is valid
+ * @chan_width_valid: @chan_width is valid.
+ * @preamble_valid: @preamble is valid.
+ * @is_delayed_lmr: indicates if the reported LMR is of the current burst or the
+ *     previous burst, flag.
  */
 struct cfg80211_pmsr_ftm_result {
        const u8 *lci;
@@ -4471,8 +4502,18 @@ struct cfg80211_pmsr_ftm_result {
        s64 dist_avg;
        s64 dist_variance;
        s64 dist_spread;
+       u32 tx_ltf_repetition_count;
+       u32 rx_ltf_repetition_count;
+       u32 max_time_between_measurements;
+       u32 min_time_between_measurements;
+       u8 num_tx_spatial_streams;
+       u8 num_rx_spatial_streams;
+       u32 nominal_time;
+       u8 availability_window;
+       enum nl80211_chan_width chan_width;
+       enum nl80211_preamble preamble;
 
-       u16 num_ftmr_attempts_valid:1,
+       u32 num_ftmr_attempts_valid:1,
            num_ftmr_successes_valid:1,
            rssi_avg_valid:1,
            rssi_spread_valid:1,
@@ -4483,7 +4524,18 @@ struct cfg80211_pmsr_ftm_result {
            rtt_spread_valid:1,
            dist_avg_valid:1,
            dist_variance_valid:1,
-           dist_spread_valid:1;
+           dist_spread_valid:1,
+           tx_ltf_repetition_count_valid:1,
+           rx_ltf_repetition_count_valid:1,
+           max_time_between_measurements_valid:1,
+           min_time_between_measurements_valid:1,
+           num_tx_spatial_streams_valid:1,
+           num_rx_spatial_streams_valid:1,
+           nominal_time_valid:1,
+           availability_window_valid:1,
+           chan_width_valid:1,
+           preamble_valid:1,
+           is_delayed_lmr:1;
 };
 
 /**

diff --git a/include/uapi/linux/nl80211.h b/include/uapi/linux/nl80211.h
index 6be89e6562769854b4d048481f26441f0f96edd8..08e8e4de650cd6c261fcc339ace4857914a3d0f2 100644 (file)
--- a/include/uapi/linux/nl80211.h
+++ b/include/uapi/linux/nl80211.h
@@ -8439,6 +8439,33 @@ enum nl80211_peer_measurement_ftm_failure_reasons {
  * @NL80211_PMSR_FTM_RESP_ATTR_PAD: ignore, for u64/s64 padding only
  * @NL80211_PMSR_FTM_RESP_ATTR_BURST_PERIOD: actual burst period used by
  *     the responder (similar to request, u16)
+ * @NL80211_PMSR_FTM_RESP_ATTR_TX_LTF_REPETITION_COUNT: negotiated value of
+ *     number of tx ltf repetitions in NDP frames (u32, optional)
+ * @NL80211_PMSR_FTM_RESP_ATTR_RX_LTF_REPETITION_COUNT: negotiated value of
+ *     number of rx ltf repetitions in NDP frames (u32, optional)
+ * @NL80211_PMSR_FTM_RESP_ATTR_MAX_TIME_BETWEEN_MEASUREMENTS: negotiated value
+ *     where latest time by which the ISTA needs to complete the next round of
+ *     measurements, in units of 10 ms (u32, optional)
+ * @NL80211_PMSR_FTM_RESP_ATTR_MIN_TIME_BETWEEN_MEASUREMENTS: negotiated
+ *     minimum time between two consecutive range measurements initiated by an
+ *     ISTA, in units of 100 us (u32, optional)
+ * @NL80211_PMSR_FTM_RESP_ATTR_NUM_TX_SPATIAL_STREAMS: number of Tx space-time
+ *     streams used in NDP frames during the measurement sounding phase
+ *     (u32, optional).
+ * @NL80211_PMSR_FTM_RESP_ATTR_NUM_RX_SPATIAL_STREAMS: number of Rx space-time
+ *     streams used in the NDP frames during the measurement sounding phase
+ *     (u32, optional)
+ * @NL80211_PMSR_FTM_RESP_ATTR_NOMINAL_TIME: negotiated nominal time used in
+ *     this session in milliseconds. (u32, optional)
+ * @NL80211_PMSR_FTM_RESP_ATTR_AVAILABILITY_WINDOW: negotiated availability
+ *     window time used in this session, in units of milli seconds.
+ *     (u32, optional)
+ * @NL80211_PMSR_FTM_RESP_ATTR_CHANNEL_WIDTH: u32 attribute indicating channel
+ *     width used for measurement, see &enum nl80211_chan_width (optional).
+ * @NL80211_PMSR_FTM_RESP_ATTR_PREAMBLE: u32 attribute indicating the preamble
+ *     type used for the measurement, see &enum nl80211_preamble (optional).
+ * @NL80211_PMSR_FTM_RESP_ATTR_IS_DELAYED_LMR: flag, indicates if the
+ *     current result is delayed LMR data.
  *
  * @NUM_NL80211_PMSR_FTM_RESP_ATTR: internal
  * @NL80211_PMSR_FTM_RESP_ATTR_MAX: highest attribute number
@@ -8468,6 +8495,17 @@ enum nl80211_peer_measurement_ftm_resp {
        NL80211_PMSR_FTM_RESP_ATTR_CIVICLOC,
        NL80211_PMSR_FTM_RESP_ATTR_PAD,
        NL80211_PMSR_FTM_RESP_ATTR_BURST_PERIOD,
+       NL80211_PMSR_FTM_RESP_ATTR_TX_LTF_REPETITION_COUNT,
+       NL80211_PMSR_FTM_RESP_ATTR_RX_LTF_REPETITION_COUNT,
+       NL80211_PMSR_FTM_RESP_ATTR_MAX_TIME_BETWEEN_MEASUREMENTS,
+       NL80211_PMSR_FTM_RESP_ATTR_MIN_TIME_BETWEEN_MEASUREMENTS,
+       NL80211_PMSR_FTM_RESP_ATTR_NUM_TX_SPATIAL_STREAMS,
+       NL80211_PMSR_FTM_RESP_ATTR_NUM_RX_SPATIAL_STREAMS,
+       NL80211_PMSR_FTM_RESP_ATTR_NOMINAL_TIME,
+       NL80211_PMSR_FTM_RESP_ATTR_AVAILABILITY_WINDOW,
+       NL80211_PMSR_FTM_RESP_ATTR_CHANNEL_WIDTH,
+       NL80211_PMSR_FTM_RESP_ATTR_PREAMBLE,
+       NL80211_PMSR_FTM_RESP_ATTR_IS_DELAYED_LMR,
 
        /* keep last */
        NUM_NL80211_PMSR_FTM_RESP_ATTR,

diff --git a/net/wireless/pmsr.c b/net/wireless/pmsr.c
index 951ba0b96da26b946118573a160b9cc6dc5cdca2..caffa2421c20cd1035e04038041a7b8226ee8017 100644 (file)
--- a/net/wireless/pmsr.c
+++ b/net/wireless/pmsr.c
@@ -562,6 +562,21 @@ static int nl80211_pmsr_send_ftm_res(struct sk_buff *msg,
        PUTOPT_U64(DIST_AVG, dist_avg);
        PUTOPT_U64(DIST_VARIANCE, dist_variance);
        PUTOPT_U64(DIST_SPREAD, dist_spread);
+       PUTOPT(u32, TX_LTF_REPETITION_COUNT, tx_ltf_repetition_count);
+       PUTOPT(u32, RX_LTF_REPETITION_COUNT, rx_ltf_repetition_count);
+       PUTOPT(u32, MAX_TIME_BETWEEN_MEASUREMENTS,
+              max_time_between_measurements);
+       PUTOPT(u32, MIN_TIME_BETWEEN_MEASUREMENTS,
+              min_time_between_measurements);
+       PUTOPT(u8, NUM_TX_SPATIAL_STREAMS, num_tx_spatial_streams);
+       PUTOPT(u8, NUM_RX_SPATIAL_STREAMS, num_rx_spatial_streams);
+       PUTOPT(u32, NOMINAL_TIME, nominal_time);
+       PUTOPT(u8, AVAILABILITY_WINDOW, availability_window);
+       PUTOPT(u32, CHANNEL_WIDTH, chan_width);
+       PUTOPT(u32, PREAMBLE, preamble);
+       if (res->ftm.is_delayed_lmr &&
+           nla_put_flag(msg, NL80211_PMSR_FTM_RESP_ATTR_IS_DELAYED_LMR))
+               goto error;
        if (res->ftm.lci && res->ftm.lci_len &&
            nla_put(msg, NL80211_PMSR_FTM_RESP_ATTR_LCI,
                    res->ftm.lci_len, res->ftm.lci))