[MERGED] osmo-bts[master]: Fix Downlink AMR FSM name to avoid illegal space character

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/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Tue Oct 24 17:01:20 UTC 2017


Harald Welte has submitted this change and it was merged.

Change subject: Fix Downlink AMR FSM name to avoid illegal space character
......................................................................


Fix Downlink AMR FSM name to avoid illegal space character

Since libosmocore Change-Id I9ef59432f43a3cdb94e4cbb0c44ac3f9b2aac0f2
we enforce that FSM names do not contain illegal characteers such as
spaces.  Let's change the DL TX FSM for AMR to comply to this.  Also,
actually do check the result of the FSM registration.

Change-Id: Ieccd1dc32c0faf5e544d17daca4a417d1d168c21
---
M src/common/bts.c
M src/common/dtx_dl_amr_fsm.c
2 files changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/common/bts.c b/src/common/bts.c
index cb5284d..6747f50 100644
--- a/src/common/bts.c
+++ b/src/common/bts.c
@@ -180,7 +180,9 @@
 	INIT_LLIST_HEAD(&btsb->oml_queue);
 
 	/* register DTX DL FSM */
-	osmo_fsm_register(&dtx_dl_amr_fsm);
+	rc = osmo_fsm_register(&dtx_dl_amr_fsm);
+	OSMO_ASSERT(rc == 0);
+
 	return rc;
 }
 
diff --git a/src/common/dtx_dl_amr_fsm.c b/src/common/dtx_dl_amr_fsm.c
index d599048..38e22c9 100644
--- a/src/common/dtx_dl_amr_fsm.c
+++ b/src/common/dtx_dl_amr_fsm.c
@@ -469,7 +469,7 @@
 };
 
 struct osmo_fsm dtx_dl_amr_fsm = {
-	.name = "DTX DL AMR FSM",
+	.name = "DTX_DL_AMR_FSM",
 	.states = dtx_dl_amr_fsm_states,
 	.num_states = ARRAY_SIZE(dtx_dl_amr_fsm_states),
 	.event_names = dtx_dl_amr_fsm_event_names,

-- 
To view, visit https://gerrit.osmocom.org/4405
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieccd1dc32c0faf5e544d17daca4a417d1d168c21
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list