fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/32919 )
Change subject: trxcon/l1sched: use sched_tchh_dl_facch_map[] in rx_tchh_fn() ......................................................................
trxcon/l1sched: use sched_tchh_dl_facch_map[] in rx_tchh_fn()
This is a more effecient way to check if FACCH/H is allowed. The mapping was added in a53e93fe9 and is already used for AMR.
Change-Id: Ib4b409956326480679a436d28224b539d00779e2 Related: a53e93fe9 "trxcon: Initial support for forwarding AMR" Related: OS#4396 --- M src/host/trxcon/src/sched_lchan_tchh.c 1 file changed, 16 insertions(+), 2 deletions(-)
Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved
diff --git a/src/host/trxcon/src/sched_lchan_tchh.c b/src/host/trxcon/src/sched_lchan_tchh.c index 04a8bff..acf362e 100644 --- a/src/host/trxcon/src/sched_lchan_tchh.c +++ b/src/host/trxcon/src/sched_lchan_tchh.c @@ -302,8 +302,8 @@ case GSM48_CMODE_SIGN: case GSM48_CMODE_SPEECH_V1: /* HR */ rc = gsm0503_tch_hr_decode(l2, buffer, - !l1sched_tchh_facch_end(lchan->type, bi->fn, 0), - &n_errors, &n_bits_total); + !sched_tchh_dl_facch_map[bi->fn % 26], + &n_errors, &n_bits_total); break; case GSM48_CMODE_SPEECH_AMR: /* AMR */ /* See comment in function rx_tchf_fn() */