On 28 February 2014 10:00, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote: > Michal Kazior <michal.kazior@xxxxxxxxx> writes: > >> HTT Rx endpoint processes both frame rx >> indications and frame tx completion indications. >> >> Those completions typically come in batches and >> may be mixed so it makes sense to defer processing >> hoping to get a bunch of them and take advantage >> of hot caches. >> >> Signed-off-by: Michal Kazior <michal.kazior@xxxxxxxxx> > > [...] > >> +void ath10k_htt_txrx_compl_task(unsigned long ptr) >> +{ >> + struct ath10k_htt *htt = (struct ath10k_htt *)ptr; >> + struct htt_resp *resp; >> + struct sk_buff *skb; > > I think this can be a static function. Is it ok if I change this patch > with the diff below? I don't really mind. Michał -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html