qcacmn: Do not update source ring hp in completion process
ce_send_entries_done_nolock_srng() source ring Tx completion processing
callback function making the redundant update of previous HP. Further
when actual Tx updates the HP results in 2nd HP update.
Because of race in hardware HP update detection, second write is lost.
Avoid this scenario by not updating HP in Tx completion processing.
Also remove unused ce_send_entries_done and ce_recv_entries_done APIs.
Change-Id: I1eb288cd7fb0ff418d4cec4f8d127cff8fc9822d
CRs-Fixed: 2407394
diff --git a/hif/src/ce/ce_service_srng.c b/hif/src/ce/ce_service_srng.c
index 56175a3..d3f5790 100644
--- a/hif/src/ce/ce_service_srng.c
+++ b/hif/src/ce/ce_service_srng.c
@@ -328,7 +328,7 @@
count = hal_srng_src_done_val(scn->hal_soc, src_ring->srng_ctx);
- hal_srng_access_end(scn->hal_soc, src_ring->srng_ctx);
+ hal_srng_access_end_reap(scn->hal_soc, src_ring->srng_ctx);
return count;
}