Change in ...osmo-bts[master]: osmo-bts-trx/scheduler: rx_rach_fn(): clarify handover RACH handling

[laforge]

laforge has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/osmo-bts/+/14746 )

Change subject: osmo-bts-trx/scheduler: rx_rach_fn(): clarify handover RACH handling
......................................................................

osmo-bts-trx/scheduler: rx_rach_fn(): clarify handover RACH handling

Change-Id: I3da39d48052af1759297f4ad75c220b3046c0691
---
M src/osmo-bts-trx/scheduler_trx.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/osmo-bts-trx/scheduler_trx.c b/src/osmo-bts-trx/scheduler_trx.c
index 00d7bbb..cd3dbf4 100644
--- a/src/osmo-bts-trx/scheduler_trx.c
+++ b/src/osmo-bts-trx/scheduler_trx.c
@@ -793,7 +793,8 @@
 	enum rach_synch_seq_t synch_seq = RACH_SYNCH_SEQ_TS0;
 	int best_score = 127 * RACH_SYNCH_SEQ_LEN;
 
-	/* Handover RACH cannot be extended (11-bit) */
+	/* If chan != TRXC_RACH, this is a handover RACH, which is always encoded
+	 * as 8-bit and should contain the generic training sequence (TS0). */
 	if (chan == TRXC_RACH) {
 		if (bi->flags & TRX_BI_F_TS_INFO)
 			synch_seq = (enum rach_synch_seq_t) bi->tsc;

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14746
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I3da39d48052af1759297f4ad75c220b3046c0691
Gerrit-Change-Number: 14746
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190716/8c10cac7/attachment.htm>