Wen Gong <wgong@xxxxxxxxxxxxxx> writes:
> The transmission utilization ratio for sdio bus for small packet is
> slow, because the space and time cost for sdio bus is same for large
> length packet and small length packet. So the speed of data for large
> length packet is higher than small length.
>
> Test result of different length of data:
>
> data packet(byte) cost time(us) calculated rate(Mbps)
> 256 28 73
> 512 33 124
> 1024 35 234
> 1792 45 318
> 14336 168 682
> 28672 333 688
> 57344 660 695
>
> This patch change the TX packet from single packet to a large length
> bundle packet, max size is 32, it results in significant performance
> improvement on TX path.
>
> This patch only effect sdio chip, it will not effect PCI, SNOC etc.
> It only enable bundle for sdio chip.
>
> Tested with QCA6174 SDIO with firmware
> WLAN.RMH.4.4.1-00017-QCARMSWP-1.
>
> Signed-off-by: Wen Gong <wgong@xxxxxxxxxxxxxx>
[...]
> + if (ep->tx_credits < credits) {
> + ath10k_dbg(ar, ATH10K_DBG_HTC,
> + "htc insufficient credits ep %d required %d available %d consume %d\n",
> + eid, credits, ep->tx_credits, consume);
> + ret = -EAGAIN;
> + goto unlock;
> + }
> +
> + if (consume) {
> + ep->tx_credits -= credits;
> + ath10k_dbg(ar, ATH10K_DBG_HTC,
> + "htc ep %d consumed %d credits (total %d)\n",
"htc ep %d consumed %d credits total %d\n"
[...]
> + ath10k_dbg(ar, ATH10K_DBG_HTC, "bundle skb: len:%d\n", bundle_skb->len);
"htc bundle skb len %d\n"
In other words, start with "htc" and don't use colons or parenthesis.
This applies to most of debug messages in this patch.
--
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
