ath10k: report mgmt tx retry status as no_ack

Retry status may mean FW either didn't send the
frame or there was no ACK for it. Since we don't
really support retrying (why should we?) lets
report these frames as not acked.

This way if NullFunc frame is sent through mgmt tx
path we can report it's status correctly to
mac80211.

Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
ath10k: cleanup tx completion

The default values are false anyway. No need to
reassign.

Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
ath10k: pass NullFunc frames through mgmt tx path

FW was reporting incorrect tx status for NullFunc
frames sent through data tx path.

mac80211 uses tx status when dealing with
link/beacon loss. Using mgmt tx path allows us to
get reliable tx status. This patch fixes
connection loss handling.

Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>