fixeria submitted this change.

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

osmo-bts-trx: fix scheduling of dummy FACCH/H and FACCH/F

* dl_ongoing_facch must be set for correct FACCH/H scheduling;
* dl_facch_bursts must be set for FACCH overpower to work.

Change-Id: Ief12eb67ad80de3b71f5226858dc2e0c8ae76948
Related: SYS#5919, OS#4823
---
M src/osmo-bts-trx/sched_lchan_tchf.c
M src/osmo-bts-trx/sched_lchan_tchh.c
2 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/osmo-bts-trx/sched_lchan_tchf.c b/src/osmo-bts-trx/sched_lchan_tchf.c
index 8fee874..a4ca9bf 100644
--- a/src/osmo-bts-trx/sched_lchan_tchf.c
+++ b/src/osmo-bts-trx/sched_lchan_tchf.c
@@ -559,6 +559,7 @@
 
 		LOGL1SB(DL1P, LOGL_DEBUG, l1ts, br, "No TCH or FACCH prim for transmit.\n");
 		gsm0503_tch_fr_encode(*bursts_p, dummy, sizeof(dummy), 1);
+		chan_state->dl_facch_bursts = 8;
 		goto send_burst;
 	}
 
diff --git a/src/osmo-bts-trx/sched_lchan_tchh.c b/src/osmo-bts-trx/sched_lchan_tchh.c
index 0db9c18..55a185e 100644
--- a/src/osmo-bts-trx/sched_lchan_tchh.c
+++ b/src/osmo-bts-trx/sched_lchan_tchh.c
@@ -408,6 +408,8 @@
 
 		LOGL1SB(DL1P, LOGL_INFO, l1ts, br, "No TCH or FACCH prim for transmit.\n");
 		gsm0503_tch_hr_encode(*bursts_p, dummy, sizeof(dummy));
+		chan_state->dl_ongoing_facch = 1;
+		chan_state->dl_facch_bursts = 6;
 		goto send_burst;
 	}

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