[PATCH v3 02/36] PR: Update PR device configs and capabilities from driver

[Benjamin Berg]

Hi,

On Thu, 2025-08-28 at 21:12 +0530, Peddolla Harshavardhan Reddy wrote:
> Add changes to fetch device configurations, capabilities, and
> supported channels for ranging and store them in Proximity Ranging
> global context.
> 
> This includes propagation of Enhanced Distributed channel Access
> (EDCA-802.11mc) based ranging capabilities as well as Non-Trigger
> Based (NTB-802.11az) ranging capabilities.
> 
> Signed-off-by: Peddolla Harshavardhan Reddy
> <peddolla at qti.qualcomm.com>
> ---
>  src/common/proximity_ranging.h    | 43
> +++++++++++++++++++++++++++++++
>  src/drivers/driver.h              | 23 +++++++++++++++++
>  wpa_supplicant/config.c           |  2 ++
>  wpa_supplicant/config.h           |  9 +++++++
>  wpa_supplicant/config_file.c      |  4 +++
>  wpa_supplicant/pr_supplicant.c    | 36 ++++++++++++++++++++++++++
>  wpa_supplicant/wpa_supplicant.c   | 15 +++++++++++
>  wpa_supplicant/wpa_supplicant_i.h | 16 ++++++++++++
>  8 files changed, 148 insertions(+)
> 
> [SNIP]
> 
> diff --git a/wpa_supplicant/wpa_supplicant_i.h
> b/wpa_supplicant/wpa_supplicant_i.h
> index cc8ee34ad..3c9e02af1 100644
> --- a/wpa_supplicant/wpa_supplicant_i.h
> +++ b/wpa_supplicant/wpa_supplicant_i.h
> @@ -957,6 +957,22 @@ struct wpa_supplicant {
>  	unsigned int max_stations;
>  	unsigned int max_num_akms;
>  
> +	/* EDCA based ranging capabilities */
> +	u8 edca_format_and_bw;
> +	u8 max_tx_antenna;
> +	u8 max_rx_antenna;
> +
> +	/* NTB based ranging capabilities */
> +	u8 ntb_format_and_bw;
> +	u8 max_tx_ltf_repetations;
> +	u8 max_rx_ltf_repetations;
> +	u8 max_tx_ltf_total;
> +	u8 max_rx_ltf_total;
> +	u8 max_rx_sts_le_80;
> +	u8 max_rx_sts_gt_80;
> +	u8 max_tx_sts_le_80;
> +	u8 max_tx_sts_gt_80;
> +
>  	int pending_mic_error_report;
>  	int pending_mic_error_pairwise;
>  	int mic_errors_seen; /* Michael MIC errors with the current
> PTK */

Is there a need for these inside struct wpa_supplicant?

I think in the current patchset you could simply pass the driver
capabilities struct to wpas_pr_init instead of adding them here.

Benjamin