Michal Kazior <[email protected]> writes: > It doesn't make much sense to overwrite send_cb > and recv_cb callbacks over and over again whenever > transport starts. Just make sure to unmask copy > engine interrupts when starting. > > Signed-off-by: Michal Kazior <[email protected]>
The patch looks, just a followup question for the future: > @@ -1674,7 +1639,9 @@ static int ath10k_pci_ce_init(struct ath10k *ar) > pipe_info->hif_ce_state = ar; > attr = &host_ce_config_wlan[pipe_num]; > > - ret = ath10k_ce_init_pipe(ar, pipe_num, attr); > + ret = ath10k_ce_init_pipe(ar, pipe_num, attr, > + ath10k_pci_ce_send_done, > + ath10k_pci_ce_recv_data); As we call ath10k_ce_init_pipe() only once and seem to have only one set of functions, why even bother bother with function pointers? What if we just call the functions directly? -- Kalle Valo _______________________________________________ ath10k mailing list [email protected] http://lists.infradead.org/mailman/listinfo/ath10k
