Wen Gong <wgong@xxxxxxxxxxxxxx> writes: > On 2020-04-09 23:05, Kalle Valo wrote: > >>> + 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. > > I will change the log and other log and sent v10. > but "ath10k: disable TX complete indication of htt for sdio" and > "ath10k: change ATH10K_SDIO_BUS_REQUEST_MAX_NUM from 64 to 1024" has > appied to ath-next, > so I only need to send the left 2 patches: > "ath10k: add htt TX bundle for sdio" and "ath10k: enable alt data of > TX path for sdio", right? Correct, I already applied patches 1 and 2. But before you resend patches 3-4 did you see my question about creating a new thread, is it really necessary? -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
