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

Reply via email to