[PATCH ath-next v2] wifi: ath12k: Add support for transmit histogram stats
Vasanthakumar Thiagarajan
quic_vthiagar at quicinc.com
Thu Jun 26 20:44:00 PDT 2025
On 6/26/2025 11:55 PM, Jeff Johnson wrote:
> On 6/26/2025 11:18 AM, Jeff Johnson wrote:
>> On 6/26/2025 10:20 AM, Vasanthakumar Thiagarajan wrote:
>>> On 6/17/2025 4:51 PM, Roopni Devanathan wrote:
>>>> From: Saleemuddin Shaik <quic_saleemud at quicinc.com>
>>>> +struct ath12k_htt_tx_histogram_stats_tlv {
>>>> + __le32 rate_retry_mcs_drop_cnt;
>>>> + __le32 mcs_drop_rate[ATH12K_HTT_TX_PDEV_STATS_NUM_MCS_DROP_COUNTERS];
>>>> + __le32 per_histogram_cnt[ATH12K_HTT_TX_PDEV_STATS_NUM_PER_COUNTERS];
>>>> + __le32 low_latency_rate_cnt;
>>>> + __le32 su_burst_rate_drop_cnt;
>>>> + __le32 su_burst_rate_drop_fail_cnt;
>>>> +};
>>>
>>> Since this strcuture represents the message format used between host and firmware, pls add
>>> __packed annotation even though it may not have any effect in this case.
>>
>> looks like I missed this in a few other structs as well, at least:
>> ath12k_htt_tx_pdev_rate_stats_tlv
>> ath12k_htt_rx_pdev_rate_ext_stats_tlv
>>
>> perhaps i'll take this as-is and then have another patch that adds __packed to
>> all _tlv structs that are missing it?
>
> actually let me fix this instance in pending, and we can fix other existing
> ones separately
Sounds good, thanks.
Vasanth
More information about the ath12k
mailing list