pespin has uploaded this change for review. (
https://gerrit.osmocom.org/c/osmocom-bb/+/33189 )
Change subject: layer23: modem: Add helper function to get FN from gsm48_req_ref
......................................................................
layer23: modem: Add helper function to get FN from gsm48_req_ref
Change-Id: I1265a580f55473f52e77af6c5f53c1573f8f3e70
---
M src/host/layer23/src/modem/grr.c
1 file changed, 25 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/89/33189/1
diff --git a/src/host/layer23/src/modem/grr.c b/src/host/layer23/src/modem/grr.c
index c4e8537..b290c8b 100644
--- a/src/host/layer23/src/modem/grr.c
+++ b/src/host/layer23/src/modem/grr.c
@@ -47,6 +47,16 @@
#include <l1ctl_proto.h>
+static uint32_t _gsm48_req_ref2fn(const struct gsm48_req_ref *ref)
+{
+ const struct gsm_time time = {
+ .t3 = ref->t3_high << 3 | ref->t3_low,
+ .t2 = ref->t2,
+ .t1 = ref->t1,
+ };
+ return gsm_gsmtime2fn(&time);
+}
+
/* Generate an 8-bit CHANNEL REQUEST message as per 3GPP TS 44.018, 9.1.8 */
uint8_t modem_grr_gen_chan_req(bool single_block)
{
@@ -254,9 +264,10 @@
return 0;
}
if (!grr_match_req_ref(ms, &ia->req_ref)) {
- LOGP(DRR, LOGL_INFO, "%s(): req_ref mismatch (RA=0x%02x, T1=%u, T3=%u,
T2=%u)\n",
+ LOGP(DRR, LOGL_INFO, "%s(): req_ref mismatch (RA=0x%02x, T1=%u, T3=%u, T2=%u,
FN=%u)\n",
__func__, ia->req_ref.ra, ia->req_ref.t1,
- ia->req_ref.t3_high << 3 | ia->req_ref.t3_low, ia->req_ref.t2);
+ ia->req_ref.t3_high << 3 | ia->req_ref.t3_low, ia->req_ref.t2,
+ _gsm48_req_ref2fn(&ia->req_ref));
return 0;
}
@@ -460,8 +471,9 @@
if (rr->state == GSM48_RR_ST_CONN_PEND) {
const struct gsm48_req_ref *ref = (void *)&ch->data[1];
LOGP(DRSL, LOGL_NOTICE,
- "Rx RACH.conf (RA=0x%02x, T1=%u, T3=%u, T2=%u)\n",
- rr->cr_ra, ref->t1, ref->t3_high << 3 | ref->t3_low,
ref->t2);
+ "Rx RACH.conf (RA=0x%02x, T1=%u, T3=%u, T2=%u, FN=%u)\n",
+ rr->cr_ra, ref->t1, ref->t3_high << 3 | ref->t3_low,
ref->t2,
+ _gsm48_req_ref2fn(ref));
/* shift the CHANNEL REQUEST history buffer */
memmove(&rr->cr_hist[1], &rr->cr_hist[0], ARRAY_SIZE(rr->cr_hist) -
1);
/* store the new entry */
--
To view, visit
https://gerrit.osmocom.org/c/osmocom-bb/+/33189
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I1265a580f55473f52e77af6c5f53c1573f8f3e70
Gerrit-Change-Number: 33189
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newchange