fixeria submitted this change.

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved

osmo-bts-trx: rx_tchh_fn(): do not calculate BER10k for FACCH twice

We already have BER10k calculated in the generic code path, so do
not calculate it once again in the FACCH specific branch.

Change-Id: I5d3955d09990e280d11d687385eeaf5edf437395
---
M src/osmo-bts-trx/sched_lchan_tchh.c
1 file changed, 0 insertions(+), 1 deletion(-)

diff --git a/src/osmo-bts-trx/sched_lchan_tchh.c b/src/osmo-bts-trx/sched_lchan_tchh.c
index 27880c1..b6d83d0 100644
--- a/src/osmo-bts-trx/sched_lchan_tchh.c
+++ b/src/osmo-bts-trx/sched_lchan_tchh.c
@@ -269,7 +269,6 @@
 	/* FACCH */
 	if (rc == GSM_MACBLOCK_LEN) {
 		chan_state->ul_ongoing_facch = 1;
-		uint16_t ber10k = compute_ber10k(n_bits_total, n_errors);
 		if (lchan->nr == 0)
 			fn_begin = gsm0502_fn_remap(bi->fn, FN_REMAP_FACCH_H0);
 		else

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

To view, visit change 27483. To unsubscribe, or for help writing mail filters, visit settings.