laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
firmware/layer1: clarify L1CTL_DM_EST_REQ related logging

Change-Id: I7a7fb32eab0ab20672a47861c3b66da92bd83015
---
M src/target/firmware/layer1/l23_api.c
1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/target/firmware/layer1/l23_api.c b/src/target/firmware/layer1/l23_api.c
index 275043b..a133c62 100644
--- a/src/target/firmware/layer1/l23_api.c
+++ b/src/target/firmware/layer1/l23_api.c
@@ -257,8 +257,8 @@
struct l1ctl_info_ul *ul = (struct l1ctl_info_ul *) l1h->data;
struct l1ctl_dm_est_req *est_req = (struct l1ctl_dm_est_req *) ul->payload;

- printd("L1CTL_DM_EST_REQ (arfcn=%u, chan_nr=0x%02x, tsc=%u)\n",
- ntohs(est_req->h0.band_arfcn), ul->chan_nr, est_req->tsc);
+ printd("L1CTL_DM_EST_REQ (chan_nr=0x%02x, tsc=%u)\n",
+ ul->chan_nr, est_req->tsc);

/* disable neighbour cell measurement of C0 TS 0 */
mframe_disable(MF_TASK_NEIGH_PM51_C0T0);
@@ -276,8 +276,12 @@
l1s.dedicated.h1.n = est_req->h1.n;
for (i=0; i<est_req->h1.n; i++)
l1s.dedicated.h1.ma[i] = ntohs(est_req->h1.ma[i]);
+ printd("L1CTL_DM_EST_REQ indicates H1 (HSN=%u, MAIO=%u, chans=%u)\n",
+ est_req->h1.hsn, est_req->h1.maio, est_req->h1.n);
} else {
l1s.dedicated.h0.arfcn = ntohs(est_req->h0.band_arfcn);
+ printd("L1CTL_DM_EST_REQ indicates H0 (ARFCN=%u)\n",
+ l1s.dedicated.h0.arfcn & ~ARFCN_FLAG_MASK);
}

/* TCH config */

To view, visit change 28109. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I7a7fb32eab0ab20672a47861c3b66da92bd83015
Gerrit-Change-Number: 28109
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged