How about wcn36xx_smd_set_sta_default_ht_params? 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 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? On Wed, Nov 20, 2013 at 7:26 AM, Yeoh Chun-Yeow <yeohchunyeow@xxxxxxxxx> wrote: > Hi, Eugene > >> 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. > How about rename the function name to wcn36xx_smd_set_self_sta? > > ---- > Chun-Yeow -- 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