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

Change subject: osmo-bts-trx/scheduler: rx_rach_fn(): enrich debug message
......................................................................

osmo-bts-trx/scheduler: rx_rach_fn(): enrich debug message

Change-Id: I1cb28a9d6c98993705b73937409276994f375dc0
---
M src/osmo-bts-trx/scheduler_trx.c
1 file changed, 9 insertions(+), 3 deletions(-)

Approvals:
  laforge: Looks good to me, but someone else must approve
  ipse: 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 cd3dbf4..79e42e1 100644
--- a/src/osmo-bts-trx/scheduler_trx.c
+++ b/src/osmo-bts-trx/scheduler_trx.c
@@ -803,10 +803,16 @@
        }

        LOGL1S(DL1P, LOGL_DEBUG, l1t, bi->tn, chan, bi->fn,
-              "Received RACH (%s; match=%.1f%%) toa=%d\n",
+              "Received%s RACH (%s): rssi=%d toa256=%d",
+              (chan != TRXC_RACH) ? " handover" : "",
               get_value_string(rach_synch_seq_names, synch_seq),
-              best_score * 100.0 / (127 * RACH_SYNCH_SEQ_LEN),
-              bi->toa256);
+              bi->rssi, bi->toa256);
+       if (bi->flags & TRX_BI_F_CI_CB)
+               LOGPC(DL1P, LOGL_DEBUG, " C/I=%d cB", bi->ci_cb);
+       else
+               LOGPC(DL1P, LOGL_DEBUG, " match=%.1f%%",
+                     best_score * 100.0 / (127 * RACH_SYNCH_SEQ_LEN));
+       LOGPC(DL1P, LOGL_DEBUG, "\n");

        /* Compose a new L1SAP primitive */
        memset(&l1sap, 0x00, sizeof(l1sap));

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I1cb28a9d6c98993705b73937409276994f375dc0
Gerrit-Change-Number: 14685
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: ipse <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged

Reply via email to