[PATCHv2] ath10k: Update available channel list for 5G radio

Raja, Tamizh Chelvam c_traja at qti.qualcomm.com
Sun Feb 19 21:07:53 PST 2017


Please ignore this patch:(

> -----Original Message-----
> From: Raja, Tamizh Chelvam
> Sent: Monday, February 20, 2017 10:37 AM
> To: ath10k at lists.infradead.org
> Cc: linux-wireless at vger.kernel.org; Raja, Tamizh Chelvam
> <c_traja at qti.qualcomm.com>
> Subject: [PATCHv2] ath10k: Update available channel list for 5G radio
> 
> From: Tamizh chelvam <c_traja at qti.qualcomm.com>
> 
> If a 5 GHz radio is calibrated for operation in both the low band (channels 36 to
> 64) and high band(channels 100 to 169), hardware allows operations in all the
> listed channels. However, if the chip has been calibrated only for the low/high
> band and a high/low band channel is configured, due to lack of calibration
> there will be potentially invalid signal on those non calibrated channels.
> To avoid this problem this patch introduces ath10k_low_5ghz_channel and
> ath10k_high_5ghz_channel lists as separate arrays.
> The channel list will be initialized depending on the low_5ghz_chan and
> high_5ghz_chan values which we get from target through wmi service ready
> event.
> 
> Signed-off-by: Tamizh chelvam <c_traja at qti.qualcomm.com>
> ---
> v2:
>   * Addressed Jouni and Kalle's comments
> 
>  drivers/net/wireless/ath/ath10k/core.h |    2 ++
>  drivers/net/wireless/ath/ath10k/mac.c  |   55
> +++++++++++++++++++++++++++++---
>  drivers/net/wireless/ath/ath10k/wmi.c  |    6 ++++
>  drivers/net/wireless/ath/ath10k/wmi.h  |    2 ++
>  4 files changed, 61 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/wireless/ath/ath10k/core.h
> b/drivers/net/wireless/ath/ath10k/core.h
> index 88d14be..d07d567 100644
> --- a/drivers/net/wireless/ath/ath10k/core.h
> +++ b/drivers/net/wireless/ath/ath10k/core.h
> @@ -774,6 +774,8 @@ struct ath10k {
>  	u32 vht_cap_info;
>  	u32 num_rf_chains;
>  	u32 max_spatial_stream;
> +	u32 low_5ghz_chan;
> +	u32 high_5ghz_chan;
>  	/* protected by conf_mutex */
>  	bool ani_enabled;
> 
> diff --git a/drivers/net/wireless/ath/ath10k/mac.c
> b/drivers/net/wireless/ath/ath10k/mac.c
> index 3029f25..0840efb 100644
> --- a/drivers/net/wireless/ath/ath10k/mac.c
> +++ b/drivers/net/wireless/ath/ath10k/mac.c
> @@ -7623,6 +7623,38 @@ static void
> ath10k_mac_op_sta_pre_rcu_remove(struct ieee80211_hw *hw,
>  	CHAN2G(14, 2484, 0),
>  };
> 
> +static const struct ieee80211_channel ath10k_low_5ghz_channels[] = {
> +	CHAN5G(36, 5180, 0),
> +	CHAN5G(40, 5200, 0),
> +	CHAN5G(44, 5220, 0),
> +	CHAN5G(48, 5240, 0),
> +	CHAN5G(52, 5260, 0),
> +	CHAN5G(56, 5280, 0),
> +	CHAN5G(60, 5300, 0),
> +	CHAN5G(64, 5320, 0),
> +};
> +
> +static const struct ieee80211_channel ath10k_high_5ghz_channels[] = {
> +	CHAN5G(100, 5500, 0),
> +	CHAN5G(104, 5520, 0),
> +	CHAN5G(108, 5540, 0),
> +	CHAN5G(112, 5560, 0),
> +	CHAN5G(116, 5580, 0),
> +	CHAN5G(120, 5600, 0),
> +	CHAN5G(124, 5620, 0),
> +	CHAN5G(128, 5640, 0),
> +	CHAN5G(132, 5660, 0),
> +	CHAN5G(136, 5680, 0),
> +	CHAN5G(140, 5700, 0),
> +	CHAN5G(144, 5720, 0),
> +	CHAN5G(149, 5745, 0),
> +	CHAN5G(153, 5765, 0),
> +	CHAN5G(157, 5785, 0),
> +	CHAN5G(161, 5805, 0),
> +	CHAN5G(165, 5825, 0),
> +	CHAN5G(169, 5845, 0),
> +};
> +
>  static const struct ieee80211_channel ath10k_5ghz_channels[] = {
>  	CHAN5G(36, 5180, 0),
>  	CHAN5G(40, 5200, 0),
> @@ -8014,6 +8046,7 @@ int ath10k_mac_register(struct ath10k *ar)
>  	};
>  	struct ieee80211_supported_band *band;
>  	void *channels;
> +	int n_channels;
>  	int ret;
> 
>  	SET_IEEE80211_PERM_ADDR(ar->hw, ar->mac_addr); @@ -8049,16
> +8082,30 @@ int ath10k_mac_register(struct ath10k *ar)
>  	}
> 
>  	if (ar->phy_capability & WHAL_WLAN_11A_CAPABILITY) {
> -		channels = kmemdup(ath10k_5ghz_channels,
> -				   sizeof(ath10k_5ghz_channels),
> -				   GFP_KERNEL);
> +		if (ar->high_5ghz_chan <= 5350) {
> +			channels = kmemdup(ath10k_low_5ghz_channels,
> +					   sizeof(ath10k_low_5ghz_channels),
> +					   GFP_KERNEL);
> +			n_channels = ARRAY_SIZE(ath10k_low_5ghz_channels);
> +		} else if (ar->low_5ghz_chan >= 5490 &&
> +			   ar->high_5ghz_chan <= 5885) {
> +			channels = kmemdup(ath10k_high_5ghz_channels,
> +					   sizeof(ath10k_high_5ghz_channels),
> +					   GFP_KERNEL);
> +			n_channels =
> ARRAY_SIZE(ath10k_high_5ghz_channels);
> +		} else {
> +			channels = kmemdup(ath10k_5ghz_channels,
> +					   sizeof(ath10k_5ghz_channels),
> +					   GFP_KERNEL);
> +			n_channels = ARRAY_SIZE(ath10k_5ghz_channels);
> +		}
>  		if (!channels) {
>  			ret = -ENOMEM;
>  			goto err_free;
>  		}
> 
>  		band = &ar->mac.sbands[NL80211_BAND_5GHZ];
> -		band->n_channels = ARRAY_SIZE(ath10k_5ghz_channels);
> +		band->n_channels = n_channels;
>  		band->channels = channels;
>  		band->n_bitrates = ath10k_a_rates_size;
>  		band->bitrates = ath10k_a_rates;
> diff --git a/drivers/net/wireless/ath/ath10k/wmi.c
> b/drivers/net/wireless/ath/ath10k/wmi.c
> index 2f1743e..b31c4f7 100644
> --- a/drivers/net/wireless/ath/ath10k/wmi.c
> +++ b/drivers/net/wireless/ath/ath10k/wmi.c
> @@ -4593,6 +4593,8 @@ static int ath10k_wmi_alloc_host_mem(struct ath10k
> *ar, u32 req_id,
>  	arg->phy_capab = ev->phy_capability;
>  	arg->num_rf_chains = ev->num_rf_chains;
>  	arg->eeprom_rd = ev->hal_reg_capabilities.eeprom_rd;
> +	arg->low_5ghz_chan = ev->hal_reg_capabilities.low_5ghz_chan;
> +	arg->high_5ghz_chan = ev->hal_reg_capabilities.high_5ghz_chan;
>  	arg->num_mem_reqs = ev->num_mem_reqs;
>  	arg->service_map = ev->wmi_service_bitmap;
>  	arg->service_map_len = sizeof(ev->wmi_service_bitmap); @@ -4629,6
> +4631,8 @@ static int ath10k_wmi_alloc_host_mem(struct ath10k *ar, u32
> req_id,
>  	arg->phy_capab = ev->phy_capability;
>  	arg->num_rf_chains = ev->num_rf_chains;
>  	arg->eeprom_rd = ev->hal_reg_capabilities.eeprom_rd;
> +	arg->low_5ghz_chan = ev->hal_reg_capabilities.low_5ghz_chan;
> +	arg->high_5ghz_chan = ev->hal_reg_capabilities.high_5ghz_chan;
>  	arg->num_mem_reqs = ev->num_mem_reqs;
>  	arg->service_map = ev->wmi_service_bitmap;
>  	arg->service_map_len = sizeof(ev->wmi_service_bitmap); @@ -4682,6
> +4686,8 @@ static void ath10k_wmi_event_service_ready_work(struct
> work_struct *work)
>  	ar->phy_capability = __le32_to_cpu(arg.phy_capab);
>  	ar->num_rf_chains = __le32_to_cpu(arg.num_rf_chains);
>  	ar->hw_eeprom_rd = __le32_to_cpu(arg.eeprom_rd);
> +	ar->low_5ghz_chan = __le32_to_cpu(arg.low_5ghz_chan);
> +	ar->high_5ghz_chan = __le32_to_cpu(arg.high_5ghz_chan);
> 
>  	ath10k_dbg_dump(ar, ATH10K_DBG_WMI, NULL, "wmi svc: ",
>  			arg.service_map, arg.service_map_len); diff --git
> a/drivers/net/wireless/ath/ath10k/wmi.h
> b/drivers/net/wireless/ath/ath10k/wmi.h
> index 386aa51..78350d4 100644
> --- a/drivers/net/wireless/ath/ath10k/wmi.h
> +++ b/drivers/net/wireless/ath/ath10k/wmi.h
> @@ -6335,6 +6335,8 @@ struct wmi_svc_rdy_ev_arg {
>  	__le32 num_rf_chains;
>  	__le32 eeprom_rd;
>  	__le32 num_mem_reqs;
> +	__le32 low_5ghz_chan;
> +	__le32 high_5ghz_chan;
>  	const __le32 *service_map;
>  	size_t service_map_len;
>  	const struct wlan_host_mem_req
> *mem_reqs[WMI_MAX_MEM_REQS];
> --
> 1.7.9.5




More information about the ath10k mailing list