Why would the function that is setting default HT params will also set
default rates? There is no connection between HT params and default
rates.
But the idea to set HT by default in AP mode is something that we
definitely need.
On Tue, Nov 19, 2013 at 2:13 PM, Chun-Yeow Yeoh <yeohchunyeow@xxxxxxxxx> wrote:
> Set the HT capable for self STA entry for bss configuration. Otherwise,
> only legacy rate will be used for transmitting, especially for mesh
> mode.
>
> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@xxxxxxxxx>
> ---
> drivers/net/wireless/ath/wcn36xx/smd.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/ath/wcn36xx/smd.c b/drivers/net/wireless/ath/wcn36xx/smd.c
> index 19f43de..5160b04 100644
> --- a/drivers/net/wireless/ath/wcn36xx/smd.c
> +++ b/drivers/net/wireless/ath/wcn36xx/smd.c
> @@ -115,6 +115,13 @@ static void wcn36xx_smd_set_sta_ht_params(struct ieee80211_sta *sta,
> }
> }
>
> +static void wcn36xx_smd_set_sta_default_ht_params(
> + struct wcn36xx_hal_config_sta_params *sta_params)
> +{
> + wcn36xx_set_default_rates(&sta_params->supported_rates);
> + sta_params->ht_capable = 1;
> +}
> +
> static void wcn36xx_smd_set_sta_params(struct wcn36xx *wcn,
> struct ieee80211_vif *vif,
> struct ieee80211_sta *sta,
> @@ -171,7 +178,7 @@ static void wcn36xx_smd_set_sta_params(struct wcn36xx *wcn,
> memcpy(&sta_params->supported_rates, &priv_sta->supported_rates,
> sizeof(priv_sta->supported_rates));
> } else {
> - wcn36xx_set_default_rates(&sta_params->supported_rates);
> + wcn36xx_smd_set_sta_default_ht_params(sta_params);
> }
> }
>
> --
> 1.7.9.5
>
--
Best regards,
Eugene
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
