[S] Change in osmocom-bb[master]: trxcon/l1sched: fix UL FACCH regression in tx_tchh_fn()

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/33085 ) Change subject: trxcon/l1sched: fix UL FACCH regression in tx_tchh_fn() ...................................................................... trxcon/l1sched: fix UL FACCH regression in tx_tchh_fn() In ad8f7794 I introduced a regression by removing the line setting lchan->ul_facch_blocks, breaking transmission of Uplink FACCH/H. Change-Id: I4f2bbbd30c7d5dbb979b54455561df3c0cc12aac Fixes: ad8f7794 "trxcon/l1sched: remove redundant TCH/[FH] prim length checks" --- M src/host/trxcon/src/sched_lchan_tchh.c 1 file changed, 16 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve 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 acf362e..3832deb 100644 --- a/src/host/trxcon/src/sched_lchan_tchh.c +++ b/src/host/trxcon/src/sched_lchan_tchh.c @@ -456,6 +456,9 @@ goto send_burst; } + if (msgb_l2len(lchan->prim) == GSM_MACBLOCK_LEN) + lchan->ul_facch_blocks = 6; + /* populate the buffer with bursts */ switch (lchan->tch_mode) { case GSM48_CMODE_SIGN: -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/33085 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I4f2bbbd30c7d5dbb979b54455561df3c0cc12aac Gerrit-Change-Number: 33085 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged