Change in osmo-pcu[master]: tbf_{dl, ul}_ass_fsm.c: use proper macro to log tbf

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Fri Nov 12 12:40:53 UTC 2021


pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/26214 )

Change subject: tbf_{dl,ul}_ass_fsm.c: use proper macro to log tbf
......................................................................

tbf_{dl,ul}_ass_fsm.c: use proper macro to log tbf

TBF can be either UL or DL in that FSM.

Change-Id: Ief0cd5298e062f11b0f39716162a67b87c9ff35f
---
M src/tbf_dl_ass_fsm.c
M src/tbf_ul_ass_fsm.c
2 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Jenkins Builder: Verified
  daniel: Looks good to me, approved



diff --git a/src/tbf_dl_ass_fsm.c b/src/tbf_dl_ass_fsm.c
index 555380d..1d44c91 100644
--- a/src/tbf_dl_ass_fsm.c
+++ b/src/tbf_dl_ass_fsm.c
@@ -97,10 +97,10 @@
 	bitvec_unhex(&bv, DUMMY_VEC);
 
 	if (ctx->tbf != (struct gprs_rlcmac_tbf *)new_dl_tbf)
-		LOGPTBFDL(ctx->tbf, LOGL_INFO, "start Packet Downlink Assignment (PACCH) for %s\n",
+		LOGPTBF(ctx->tbf, LOGL_INFO, "start Packet Downlink Assignment (PACCH) for %s\n",
 			  tbf_name((const struct gprs_rlcmac_tbf *)new_dl_tbf));
 	else
-		LOGPTBFDL(ctx->tbf, LOGL_INFO, "start Packet Downlink Assignment (PACCH)\n");
+		LOGPTBF(ctx->tbf, LOGL_INFO, "start Packet Downlink Assignment (PACCH)\n");
 
 	mac_control_block = (RlcMacDownlink_t *)talloc_zero(ctx->tbf, RlcMacDownlink_t);
 	write_packet_downlink_assignment(mac_control_block, old_tfi_is_valid,
diff --git a/src/tbf_ul_ass_fsm.c b/src/tbf_ul_ass_fsm.c
index d2dea46..ef75360 100644
--- a/src/tbf_ul_ass_fsm.c
+++ b/src/tbf_ul_ass_fsm.c
@@ -111,10 +111,10 @@
 	bitvec_unhex(&bv, DUMMY_VEC);
 
 	if (ctx->tbf != (struct gprs_rlcmac_tbf *)new_tbf)
-		LOGPTBFUL(ctx->tbf, LOGL_INFO, "start Packet Uplink Assignment (PACCH) for %s\n",
+		LOGPTBF(ctx->tbf, LOGL_INFO, "start Packet Uplink Assignment (PACCH) for %s\n",
 			  tbf_name((const struct gprs_rlcmac_tbf *)new_tbf));
 	else
-		LOGPTBFUL(ctx->tbf, LOGL_INFO, "start Packet Uplink Assignment (PACCH)\n");
+		LOGPTBF(ctx->tbf, LOGL_INFO, "start Packet Uplink Assignment (PACCH)\n");
 
 	mac_control_block = (RlcMacDownlink_t *)talloc_zero(ctx->tbf, RlcMacDownlink_t);
 	tlli = ms_tlli(ms);

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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ief0cd5298e062f11b0f39716162a67b87c9ff35f
Gerrit-Change-Number: 26214
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211112/c3cc696a/attachment.htm>


More information about the gerrit-log mailing list