2013-11-20 07:55:23

by Eugene Krasnikov

Subject: Re: [PATCH] wcn36xx: set self sta entry as HT capable

Most of these values are taken from wcn_band_2ghz

Will try to remember:

sta_params->ht_capable = 1
sta_params->tx_channel_width_set = 1
sta_params->lsig_txop_protection = 1
sta_params->max_ampdu_size = 3 // Might be wrong but let's have this
value as a begining.
sta_params->max_ampdu_density = 5 // Might be wrong but let's have
this value as a begining.
sta_params->max_amsdu_size = 0
sta_params->sgi_20Mhz = 1
sta_params->sgi_40mhz = 1
sta_params->green_field_capable = 1
sta_params->delayed_ba_support = 0
sta_params->dsss_cck_mode_40mhz = 1

On Wed, Nov 20, 2013 at 7:43 AM, Yeoh Chun-Yeow <[email protected]> wrote:
>> This function will set all of the following fields to default values:
>> sta_params->ht_capable
>> sta_params->tx_channel_width_set
>> sta_params->lsig_txop_protection
>> sta_params->max_ampdu_size
>> sta_params->max_ampdu_density
>> sta_params->max_amsdu_size
>> sta_params->sgi_20Mhz
>> sta_params->sgi_40mhz
>> sta_params->green_field_capable
>> sta_params->delayed_ba_support
>> sta_params->dsss_cck_mode_40mhz
>
>
> I am alright with that. But I am not sure about the default value of other
> fields mentioned above. Are you able to provide that?
>
>>
>> And then in wcn36xx_smd_set_sta_params it is called like this:
>> wcn36xx_set_default_rates(&sta_params->supported_rates);
>> wcn36xx_smd_set_sta_default_ht_params
>>
>> How does that sound?
>
>
> ---
> Chun-Yeow



--
Best regards,
Eugene

2013-11-20 08:05:18

by Chun-Yeow Yeoh

Subject: Re: [PATCH] wcn36xx: set self sta entry as HT capable

Alright, I will resubmit the patch with these as default value.

---
Chun-Yeow

On Wed, Nov 20, 2013 at 3:55 PM, Eugene Krasnikov <[email protected]> wrote:
> Most of these values are taken from wcn_band_2ghz
>
> Will try to remember:
>
> sta_params->ht_capable = 1
> sta_params->tx_channel_width_set = 1
> sta_params->lsig_txop_protection = 1
> sta_params->max_ampdu_size = 3 // Might be wrong but let's have this
> value as a begining.
> sta_params->max_ampdu_density = 5 // Might be wrong but let's have
> this value as a begining.
> sta_params->max_amsdu_size = 0
> sta_params->sgi_20Mhz = 1
> sta_params->sgi_40mhz = 1
> sta_params->green_field_capable = 1
> sta_params->delayed_ba_support = 0
> sta_params->dsss_cck_mode_40mhz = 1