fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/33730 )
Change subject: osmo-bts-trx: rx_tchf_fn(): move compute_ber10k() above ......................................................................
osmo-bts-trx: rx_tchf_fn(): move compute_ber10k() above
... for the sake of consistency with rx_tchh_fn().
Change-Id: Ie331da78eb3831e35d255583466e0d09b093b132 --- M src/osmo-bts-trx/sched_lchan_tchf.c 1 file changed, 13 insertions(+), 2 deletions(-)
Approvals: laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
diff --git a/src/osmo-bts-trx/sched_lchan_tchf.c b/src/osmo-bts-trx/sched_lchan_tchf.c index 69ce488..4832311 100644 --- a/src/osmo-bts-trx/sched_lchan_tchf.c +++ b/src/osmo-bts-trx/sched_lchan_tchf.c @@ -291,6 +291,8 @@ return -EINVAL; }
+ ber10k = compute_ber10k(n_bits_total, n_errors); + /* average measurements of the last N (depends on mode) bursts */ trx_sched_meas_avg(chan_state, &meas_avg, meas_avg_mode); /* meas_avg.fn now contains TDMA frame number of the first burst */ @@ -306,8 +308,6 @@ rc = 0; /* this is how we signal BFI to l1sap */ }
- ber10k = compute_ber10k(n_bits_total, n_errors); - /* FACCH */ if (rc == GSM_MACBLOCK_LEN) { _sched_compose_ph_data_ind(l1ts, fn_begin, bi->chan,
-
fixeria