On Tue, 08 Oct 2024 17:30:39 +0800, Lingbo Kong wrote:
> Currently, when ath12k retrieves values from the structure
> rx_msdu_end_wcn7850, the information obtained is incorrect. This is because
> the definition of the rx_msdu_end_wcn7850 structure in ath12k is incorrect.
> In fact, the rx_msdu_end structure used by WCN7850 is the same as the
> rx_msdu_end structure of QCN9274.
>
> Commit ed823fd113b7 ("wifi: ath12k: add msdu_end structure for WCN7850")
> introduced this structure. The original issue trying to be fixed with that
> commit is that the TID information retrieved from the rx_msdu_end structure
> was incorrect, and it was thought that the structure wasn't correct, so a
> supposedly correct structure was introduced. However it was subsequently
> discovered that the TID information is only guaranteed to be valid in the
> rx_msdu_start structure, so that commit added the logic to retrieve the TID
> from the rx_msdu_start, but unfortunately kept the rx_msdu_end_wcn7850
> structure.
>
> [...]
Applied, thanks!
[1/1] wifi: ath12k: remove msdu_end structure for WCN7850
commit: fd01ace1140476e97f1801a26e8e96a5626e3385
Best regards,
--
Jeff Johnson <quic_jjohnson@xxxxxxxxxxx>
