John Crispin <john@xxxxxxxxxxx> writes:
> This allows us to pass HE rates down into the stack.
>
> Signed-off-by: John Crispin <john@xxxxxxxxxxx>
> ---
> drivers/net/wireless/ath/ath11k/dp_tx.c | 24 +++++++++++++++++++-----
> 1 file changed, 19 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath11k/dp_tx.c b/drivers/net/wireless/ath/ath11k/dp_tx.c
> index 7b532bf9acd8..66a6cfd54ad9 100644
> --- a/drivers/net/wireless/ath/ath11k/dp_tx.c
> +++ b/drivers/net/wireless/ath/ath11k/dp_tx.c
> @@ -357,9 +357,12 @@ static void ath11k_dp_tx_complete_msdu(struct ath11k *ar,
> struct sk_buff *msdu,
> struct hal_tx_status *ts)
> {
> + struct ieee80211_tx_status status = { 0 };
This adds a sparse warning:
drivers/net/wireless/ath/ath11k/dp_tx.c:350:47: warning: Using plain integer as NULL pointer
Seems like a false warning, no? But not sure how to shut up the warning,
using '{ NULL }' would do that but just feels wrong. Any opinions?
There was also a trivial checkpatch warning:
drivers/net/wireless/ath/ath11k/dp_tx.c:419: Please don't use multiple blank lines
--
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
