fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/14688
Change subject: osmo-bts-trx/scheduler: rx_rach_fn(): provide actual C/I ratio to L1SAP ...................................................................... osmo-bts-trx/scheduler: rx_rach_fn(): provide actual C/I ratio to L1SAP Change-Id: I8d86dec7ebc039cbfd038c4342ff328b11281865 --- M src/osmo-bts-trx/scheduler_trx.c 1 file changed, 7 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/88/14688/1 diff --git a/src/osmo-bts-trx/scheduler_trx.c b/src/osmo-bts-trx/scheduler_trx.c index 56483f3..5a24dda 100644 --- a/src/osmo-bts-trx/scheduler_trx.c +++ b/src/osmo-bts-trx/scheduler_trx.c @@ -816,8 +816,13 @@ l1sap.u.rach_ind.rssi = bi->rssi; l1sap.u.rach_ind.fn = bi->fn; - /* FIXME: check for TRX_BI_F_CI_CB, and use the value from UL.ind */ - l1sap.u.rach_ind.lqual_cb = l1t->trx->bts->min_qual_rach; + /* Link quality defined by C/I (Carrier-to-Interference ratio), + * which has optional presence. If it's absent, simulate the + * minimum acceptable value to pass L1SAP checks. */ + if (bi->flags & TRX_BI_F_CI_CB) + l1sap.u.rach_ind.lqual_cb = bi->ci_cb; + else + l1sap.u.rach_ind.lqual_cb = l1t->trx->bts->min_qual_rach; /* Decode RACH depending on its synch. sequence */ switch (synch_seq) { -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14688 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I8d86dec7ebc039cbfd038c4342ff328b11281865 Gerrit-Change-Number: 14688 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <axilira...@gmail.com> Gerrit-MessageType: newchange