On 2020-04-16 20:38, Kalle Valo wrote:
Wen Gong <wgong@xxxxxxxxxxxxxx> writes:
+static inline int ath10k_hif_set_mbox_sleep(struct ath10k *ar, bool enable_sleep)+{ + if (ar->hif.ops->set_mbox_sleep) + return ar->hif.ops->set_mbox_sleep(ar, enable_sleep); + return 0; +}I don't think we need to add another hif op for this. I sent v2 which uses existing op.
Yes, I see it. but I see it removed ath10k_hif_set_mbox_sleep in ath10k_core_start, expected it will not effect the patch's power save.