fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/30453 )

Change subject: trxcon: print TCH mode using gsm48_chan_mode_name()
......................................................................

trxcon: print TCH mode using gsm48_chan_mode_name()

Change-Id: I6c54ad857b91ef1a32501912ba4c393e1c04eff0
---
M src/host/trxcon/src/l1ctl.c
1 file changed, 5 insertions(+), 4 deletions(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved



diff --git a/src/host/trxcon/src/l1ctl.c b/src/host/trxcon/src/l1ctl.c
index 2f4881b..f28bdc8 100644
--- a/src/host/trxcon/src/l1ctl.c
+++ b/src/host/trxcon/src/l1ctl.c
@@ -625,8 +625,9 @@

        LOGPFSMSL(fi, g_logc_l1c, LOGL_NOTICE,
                  "Received L1CTL_DM_EST_REQ "
-                 "(tn=%u, chan_nr=0x%02x, tsc=%u, tch_mode=0x%02x)\n",
-                 req.chan_nr & 0x07, req.chan_nr, req.tsc, req.tch_mode);
+                 "(tn=%u, chan_nr=0x%02x, tsc=%u, tch_mode=%s)\n",
+                 req.chan_nr & 0x07, req.chan_nr, req.tsc,
+                 gsm48_chan_mode_name(est_req->tch_mode));

        /* Frequency hopping? */
        if (est_req->h)
@@ -729,8 +730,8 @@
        mode_req = (const struct l1ctl_tch_mode_req *)msg->l1h;

        LOGPFSMSL(fi, g_logc_l1c, LOGL_NOTICE,
-                 "Received L1CTL_TCH_MODE_REQ (tch_mode=%u, audio_mode=%u)\n",
-                 mode_req->tch_mode, mode_req->audio_mode);
+                 "Received L1CTL_TCH_MODE_REQ (tch_mode=%s, audio_mode=%u)\n",
+                 gsm48_chan_mode_name(mode_req->tch_mode), 
mode_req->audio_mode);

        /* TODO: do we need to care about audio_mode? */


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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I6c54ad857b91ef1a32501912ba4c393e1c04eff0
Gerrit-Change-Number: 30453
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to