diff options
| author | Harald Welte <laforge@gnumonks.org> | 2019-05-21 01:14:24 +0200 |
|---|---|---|
| committer | Harald Welte <laforge@gnumonks.org> | 2019-05-24 00:02:34 +0200 |
| commit | b07e271994f91c410930e09b8c504ae1a8df531f (patch) | |
| tree | fdc9b9ba827f19caa8a211ef950c29a78bfed5b5 /src/common/msg_utils.c | |
| parent | 482564b422096115aa9eaa2c6599fea9168b49e2 (diff) | |
Use LOGPLCHAN whenever possible
There's no point in open-coding what LOGPLCHAN was created to do:
Log some event while stating the name of the logical channel.
Change-Id: I6913ac8fb543811126b85a54118333155c03bc03
Diffstat (limited to 'src/common/msg_utils.c')
| -rw-r--r-- | src/common/msg_utils.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/common/msg_utils.c b/src/common/msg_utils.c index f936c983..52b05668 100644 --- a/src/common/msg_utils.c +++ b/src/common/msg_utils.c @@ -203,8 +203,8 @@ int dtx_dl_amr_fsm_step(struct gsm_lchan *lchan, const uint8_t *rtp_pl, rc = osmo_amr_rtp_dec(rtp_pl, rtp_pl_len, &cmr, &cmi, &ft, &bfi, &sti); if (rc < 0) { - LOGP(DRTP, LOGL_ERROR, "failed to decode AMR RTP (length %zu, " - "%p)\n", rtp_pl_len, rtp_pl); + LOGPLCHAN(lchan, DRTP, LOGL_ERROR, "failed to decode AMR RTP (length %zu, " + "%p)\n", rtp_pl_len, rtp_pl); return rc; } @@ -274,7 +274,7 @@ int dtx_dl_amr_fsm_step(struct gsm_lchan *lchan, const uint8_t *rtp_pl, } if (ft != AMR_NO_DATA) { - LOGP(DRTP, LOGL_ERROR, "unsupported AMR FT 0x%02x\n", ft); + LOGPLCHAN(lchan, DRTP, LOGL_ERROR, "unsupported AMR FT 0x%02x\n", ft); return -ENOTSUP; } @@ -505,9 +505,9 @@ uint8_t repeat_last_sid(struct gsm_lchan *lchan, uint8_t *dst, uint32_t fn) return lchan->tch.dtx.len + 1; } - LOGP(DL1C, LOGL_DEBUG, "Have to send %s frame on TCH but SID buffer " - "is empty - sent nothing\n", - get_value_string(gsm48_chan_mode_names, lchan->tch_mode)); + LOGPLCHAN(lchan, DL1C, LOGL_DEBUG, "Have to send %s frame on TCH but SID buffer " + "is empty - sent nothing\n", + get_value_string(gsm48_chan_mode_names, lchan->tch_mode)); return 0; } |