On 12/20/2024 12:24 AM, Kalle Valo wrote:
Karthikeyan Periyasamy <quic_periyasa@xxxxxxxxxxx> writes:
On 12/19/2024 5:04 PM, Kalle Valo wrote:
Karthikeyan Periyasamy <quic_periyasa@xxxxxxxxxxx> writes:
Currently, ath12k_dp_mon_rx_parse_status_tlv() takes the TLV tag, TLV data
and TLV userid as separate arguments from the caller. These argument can
be fetched from the TLV header itself. Therefore, pass the TLV header and
retrieve the necessary fields from the header instead of passing them from
the caller.
The commit message doesn't reply to "why?". Is there a clear benefit
in
the future or are you just doing random code cleanup?
It benefit in the future for supporting EHT monitor functionality.
How does it help exactly? Having even just one sentence explaining the
motivation in the commit message would help a lot. Otherwise these just
look random changes.
Currently, ath12k_dp_mon_rx_parse_status_tlv() takes the TLV tag, TLV
data and TLV user ID as separate arguments from the caller. In the
future, the TLV length will be needed for the EHT monitor TLV tags.
These arguments can be fetched from the TLV header itself. Therefore,
instead of increasing the arguments, pass the TLV header and retrieve
the necessary fields from the TLV header instead of passing them from
the caller.
Above commit is ok ?
--
Karthikeyan Periyasamy
--
கார்த்திகேயன் பெரியசாமி