Re: [PATCH v2 01/18] wl1271: Add AP related configuration to conf_drv_settings

[Luciano Coelho <luciano.coelho@nokia.com>]

On Wed, 2010-12-22 at 16:27 +0200, ext Arik Nemtsov wrote:
> Rate class configuration has been split up for AP and STA modes.
> Template related configuration likewise separated.
> 
> Signed-off-by: Arik Nemtsov <arik@wizery.com>
> ---

[...]

> @@ -153,6 +153,45 @@ static struct conf_drv_settings default_conf = {
>  				.tx_op_limit = 1504,
>  			},
>  		},
> +		.ap_rc_conf                  = {
> +			[0] = {
> +				.enabled_rates = 0x1EFF,
> +				.short_retry_limit = 10,
> +				.long_retry_limit = 10,
> +				.aflags      = 0,
> +			},
> +			[1] = {
> +				.enabled_rates = 0x1EFF,
> +				.short_retry_limit = 10,
> +				.long_retry_limit = 10,
> +				.aflags      = 0,
> +			},
> +			[2] = {
> +				.enabled_rates = 0x1EFF,
> +				.short_retry_limit = 10,
> +				.long_retry_limit = 10,
> +				.aflags      = 0,
> +			},
> +			[3] = {
> +				.enabled_rates = 0x1EFF,
> +				.short_retry_limit = 10,
> +				.long_retry_limit = 10,
> +				.aflags      = 0,
> +			},
> +		},
> +		.ap_mgmt_conf = {
> +			.enabled_rates       = 0x7,
> +			.short_retry_limit   = 10,
> +			.long_retry_limit    = 10,
> +			.aflags              = 0,
> +		},
> +		.ap_bcst_conf = {
> +			.enabled_rates       = 0x1,
> +			.short_retry_limit   = 10,
> +			.long_retry_limit    = 10,
> +			.aflags              = 0,
> +		},
> +

All the enabled_rates values here look quite magic.  Is there any way
these could be changed to macros or something more descriptive?

Also, how do you deal with the different basic rates used by 11bg and
11a?

-- 
Cheers,
Luca.