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? */