[PATCH v3 8/8] wifi: ath12k: Avoid explicit type cast in monitor status parse handler
Kalle Valo
kvalo at kernel.org
Tue Dec 17 11:55:51 PST 2024
Karthikeyan Periyasamy <quic_periyasa at quicinc.com> writes:
> Currently, monitor status parse procedure handles all the supported TLV
> tags. Each TLV tag has its own data structure for parsing. Now, this
> handler is passed the tlv_data as a u8 pointer, so explicit type cast
> conversion happens for every TLV tag parsing. Therefore, avoid the
> explicit type conversion by changing the tlv_data type from a u8 pointer
> to a const void pointer.
>
> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.3.1-00173-QCAHKSWPL_SILICONZ-1
> Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0.c5-00481-QCAHMTSWPL_V1.0_V2.0_SILICONZ-3
>
> Signed-off-by: Karthikeyan Periyasamy <quic_periyasa at quicinc.com>
> Acked-by: Jeff Johnson <quic_jjohnson at quicinc.com>
Nice, less void pointers the better.
Acked-by: Kalle Valo <kvalo at kernel.org>
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
More information about the ath12k
mailing list