Raja Mani <rmani@xxxxxxxxxxxxxxxx> writes:
> New htt event table is added for 10.4 firmware. Following new htt
> events are available only 10.4. adding this to generic htt event
> table,
> HTT_T2H_MSG_TYPE_EN_STATS,
> HTT_T2H_MSG_TYPE_TX_FETCH_IND,
> HTT_T2H_MSG_TYPE_TX_FETCH_CONF,
> HTT_T2H_MSG_TYPE_TX_LOW_LATENCY_IND
>
> Signed-off-by: Raja Mani <rmani@xxxxxxxxxxxxxxxx>
[...]
> @@ -1146,6 +1147,8 @@ static int ath10k_core_init_firmware_features(struct ath10k *ar)
> ar->htt.op_version = ATH10K_FW_HTT_OP_VERSION_TLV;
> break;
> case ATH10K_FW_WMI_OP_VERSION_10_4:
> + ar->htt.op_version = ATH10K_FW_HTT_OP_VERSION_10_4;
> + break;
> case ATH10K_FW_WMI_OP_VERSION_UNSET:
> case ATH10K_FW_WMI_OP_VERSION_MAX:
> WARN_ON(1);
So this switch block is only to support older firmware versions which do
not have ATH10K_FW_IE_HTT_OP_VERSION yet. 10.4 firmware will always have
it set so we don't need to have any backwards compatibility code for
10.4. I changed this part with the diff below. Please review.
--- a/drivers/net/wireless/ath/ath10k/core.c
+++ b/drivers/net/wireless/ath/ath10k/core.c
@@ -1147,8 +1147,6 @@ static int ath10k_core_init_firmware_features(struct ath10k *ar)
ar->htt.op_version = ATH10K_FW_HTT_OP_VERSION_TLV;
break;
case ATH10K_FW_WMI_OP_VERSION_10_4:
- ar->htt.op_version = ATH10K_FW_HTT_OP_VERSION_10_4;
- break;
case ATH10K_FW_WMI_OP_VERSION_UNSET:
case ATH10K_FW_WMI_OP_VERSION_MAX:
WARN_ON(1);
--
Kalle Valo
--
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
