[PATCH] Add IEEE80211_TX_CTL_REQ_TX_STATUS support

Kalle Valo kvalo at qca.qualcomm.com
Wed Jun 5 02:17:14 EDT 2013


Pontus Fuchs <pontus.fuchs at gmail.com> writes:

> As there seems to be no packet id in the status indication message
> from the FW, I stop the queues in order to have only one frame with
> IEEE80211_TX_CTL_REQ_TX_STATUS submitted to the FW.
>
> Signed-off-by: Pontus Fuchs <pontus.fuchs at gmail.com>

[...]

> @@ -503,10 +551,9 @@ int wcn36xx_dxe_tx(struct wcn36xx *wcn,
>  	wcn36xx_prepare_tx_bd(ctl->bd_cpu_addr, skb->len, header_len);
>  	if (!is_high && WCN36XX_BSS_KEY == wcn->en_state) {
>  		wcn36xx_dbg(WCN36XX_DBG_DXE, "DXE Encription enabled");
> -		wcn36xx_fill_tx_bd(wcn, ctl->bd_cpu_addr, broadcast, 0);
> -	} else {
> -		wcn36xx_fill_tx_bd(wcn, ctl->bd_cpu_addr, broadcast, 1);
> -	}
> +		wcn36xx_fill_tx_bd(wcn, ctl->bd_cpu_addr, broadcast, 0, tx_ack);
> +	} else
> +		wcn36xx_fill_tx_bd(wcn, ctl->bd_cpu_addr, broadcast, 1, tx_ack);

Forgot to mention that I think that the coding style requires braces
also for the else branch here.

-- 
Kalle Valo



More information about the wcn36xx mailing list