On 18 September 2014 18:11, Ben Greear <greearb@xxxxxxxxxxxxxxx> wrote: > On 09/18/2014 06:21 AM, Michal Kazior wrote: >> Firmware reports the number of RF chains so use >> that for initialization of supp_{tx,rx}_chainmask >> instead of using a macro for 3x3 chips. >> >> This should make tx/rx chainmask reports correct >> for chips other than 3x3. > >> diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c >> index 2c42bd5..772cca5 100644 >> --- a/drivers/net/wireless/ath/ath10k/wmi.c >> +++ b/drivers/net/wireless/ath/ath10k/wmi.c >> @@ -2198,6 +2198,9 @@ static void ath10k_wmi_service_ready_event_rx(struct ath10k *ar, >> ar->num_rf_chains = WMI_MAX_SPATIAL_STREAM; >> } >> >> + ar->supp_tx_chainmask = (1 << ar->num_rf_chains) - 1; >> + ar->supp_rx_chainmask = (1 << ar->num_rf_chains) - 1; > > Are there are no chips that support a different number > of tx and rx chains? I don't think current firmware interface is capable of advertising that to the host so I'm guessing there are none yet. Michał -- 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