laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/28109 )
Change subject: firmware/layer1: clarify L1CTL_DM_EST_REQ related logging ......................................................................
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(-)
Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
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 */
-
laforge