Re: [PATCH] iw: info: print missing extended feature flag information

["Toke Høiland-Jørgensen" <toke@redhat.com>]

Markus Theil <markus.theil@tu-ilmenau.de> writes:

> Some extended feature flags were not considered before.
>
> Signed-off-by: Markus Theil <markus.theil@tu-ilmenau.de>
> ---
>  info.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/info.c b/info.c
> index 38362d8..0f24dae 100644
> --- a/info.c
> +++ b/info.c
> @@ -676,7 +676,17 @@ broken_combination:
>  		ext_feat_print(tb, DFS_OFFLOAD, "DFS offload");
>  		ext_feat_print(tb, CONTROL_PORT_OVER_NL80211,
>  			       "control port over nl80211");
> +		ext_feat_print(tb, ACK_SIGNAL_SUPPORT,
> +			       "ack signal level support");
>  		ext_feat_print(tb, TXQS, "FQ-CoDel-enabled intermediate TXQs");
> +		ext_feat_print(tb, SCAN_RANDOM_SN,
> +			       "use random sequence numbers in scans");
> +		ext_feat_print(tb, SCAN_MIN_PREQ_CONTENT,
> +			       "use probe request with only rate IEs in scans");
> +		ext_feat_print(tb, CAN_REPLACE_PTK0,
> +			       "can safely replace PTK 0 when rekeying");
> +		ext_feat_print(tb, ENABLE_FTM_RESPONDER,
> +			       "enable FTM (Fine Time Measurement) responder");
>  		ext_feat_print(tb, AIRTIME_FAIRNESS,
>  			       "airtime fairness scheduling");

I would prefer to keep TXQS and AIRTIME_FAIRNESS next to each other.

Also, while you're at it, could you please add the newly-added AQL
feature? Something like

  		ext_feat_print(tb, AIRTIME_FAIRNESS,
  			       "airtime fairness scheduling");
 +		ext_feat_print(tb, AQL,
 +			       "Airtime Queue Limits (AQL)");


-Toke