osmo-bts-trx: do not run osmo_{fr,hr}_check_sid() on FACCH/U frames

It makes no sense to perform the SID codeword lookup in signalling
frames (FACCH), because it can be present only in speech frames.

Change-Id: I2f8137993acfe8a8add3fc2af276e5eb4da1605e
Related: SYS#5853
This commit is contained in:
Vadim Yanitskiy 2022-03-10 01:22:37 +03:00
parent 8ec3f28fad
commit dc17d1036b
2 changed files with 3 additions and 3 deletions

View File

@ -119,7 +119,7 @@ int rx_tchf_fn(struct l1sched_ts *l1ts, const struct trx_ul_burst_ind *bi)
: tch_mode) {
case GSM48_CMODE_SPEECH_V1: /* FR */
rc = gsm0503_tch_fr_decode(tch_data, *bursts_p, 1, 0, &n_errors, &n_bits_total);
if (rc >= 0)
if (rc == GSM_FR_BYTES) /* only for valid *speech* frames */
lchan_set_marker(osmo_fr_check_sid(tch_data, rc), lchan); /* DTXu */
break;
case GSM48_CMODE_SPEECH_EFR: /* EFR */

View File

@ -147,8 +147,8 @@ int rx_tchh_fn(struct l1sched_ts *l1ts, const struct trx_ul_burst_ind *bi)
*/
rc = gsm0503_tch_hr_decode(tch_data, *bursts_p,
fn_is_odd, &n_errors, &n_bits_total);
if (rc >= 0) /* DTXu */
lchan_set_marker(osmo_hr_check_sid(tch_data, rc), lchan);
if (rc == (GSM_HR_BYTES + 1)) /* only for valid *speech* frames */
lchan_set_marker(osmo_hr_check_sid(tch_data, rc), lchan); /* DTXu */
break;
case GSM48_CMODE_SPEECH_AMR: /* AMR */
/* the first FN 0,8,17 or 1,9,18 defines that CMI is included