Dmitry Antipov <dmantipov@xxxxxxxxx> wrote: > Since 'mod_timer()' assumes expire time in jiffies and not milliseconds, > 'msecs_to_jiffies()' should be used in 'ath10k_pci_rx_post_pipe()' and > 'ath10k_snoc_rx_post_pipe()'. > > This patch also introduces a minor refactoring and fixes number of loop > iterations in 'ath10k_pci_rx_post_pipe()' (I assume that the number of > calls of '__ath10k_pci_rx_post_buf()' should be equal to number of buffers > returned by '__ath10k_ce_rx_num_free_bufs()' and not plus one). > > Fixes: a6e149a9ff03 ("ath10k: add hif start/stop methods for wcn3990 snoc layer") > Fixes: 728f95eef523 ("ath10k: rework posting pci rx buffers") > > Signed-off-by: Dmitry Antipov <dmantipov@xxxxxxxxx> > Signed-off-by: Kalle Valo <quic_kvalo@xxxxxxxxxxx> This is too intrusive patch to take without any testing. Patch set to Rejected. -- https://patchwork.kernel.org/project/linux-wireless/patch/20230626151636.314868-1-dmantipov@xxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches