On 2019-07-11 18:24, Sergey Matyukevich wrote:
Introduce NL80211_TID_ATTR_CONFIG_TX_RATES in nl80211_tid_attr_config
to accept data TID specific TX bitrate configuration
through NL80211_CMD_SET_TID_CONFIG command. TID for which the
this configuration is to be applied is passed in
NL80211_TID_ATTR_CONFIG_TID attribute. TX bitrate mask values passed
in NL80211_ATTR_TX_RATES attribute and
NL80211_TID_ATTR_CONFIG_TX_RATES
attribute will have types of the TX rate should be applied.
When the user-space wants this configuration peer specific
rather than being applied for all the connected stations,
MAC address of the peer can be passed in NL80211_ATTR_MAC attribute.
Driver supporting this feature should advertise
NL80211_EXT_FEATURE_PER_TID_TX_BITRATE_MASK and supporting per-STA
data TID
TX bitrate configuration should advertise
NL80211_EXT_FEATURE_PER_STA_TX_BITRATE_MASK.
Signed-off-by: Tamizh chelvam <tamizhr@xxxxxxxxxxxxxx>
...
if (info->attrs[NL80211_ATTR_TX_RATES]) {
- err = nl80211_parse_tx_bitrate_mask(info,
¶ms.beacon_rate);
+ err = nl80211_parse_tx_bitrate_mask(info, info->attrs,
+
NL80211_ATTR_TX_RATES,
+
¶ms.beacon_rate);
if (err)
return err;
Could you please clarify this change of nl80211_parse_tx_bitrate_mask
arguments.
Unless I missing something, the appropriate change for
nl80211_parse_tx_bitrate_mask
is not included into this patch set.
Yeah. These arguments are missed in the function definition place:( I
will fix this rebase issue in next version.
Thanks,
Tamizh.
