[PATCH] osmocom-bb[master]: host/trxcon: get rid of useless trxcon fsm events

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
Thu Feb 22 15:33:11 UTC 2018


Review at  https://gerrit.osmocom.org/6745

host/trxcon: get rid of useless trxcon fsm events

Both SCH_EVENT_CLCK_IND and SCH_EVENT_CLCK_LOSS were not handled,
moreover there is no purpose to keep them.

Change-Id: I8efac459a40f4287e3325890809991d5ef46e9b1
---
M src/host/trxcon/sched_clck.c
M src/host/trxcon/sched_trx.c
M src/host/trxcon/trxcon.c
M src/host/trxcon/trxcon.h
4 files changed, 4 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/45/6745/1

diff --git a/src/host/trxcon/sched_clck.c b/src/host/trxcon/sched_clck.c
index efb3f4c..9c1edc5 100644
--- a/src/host/trxcon/sched_clck.c
+++ b/src/host/trxcon/sched_clck.c
@@ -47,16 +47,12 @@
 static void sched_clck_tick(void *data)
 {
 	struct trx_sched *sched = (struct trx_sched *) data;
-	struct trx_instance *trx = (struct trx_instance *) sched->data;
-
 	struct timeval tv_now, *tv_clock;
 	int32_t elapsed;
 
 	/* Check if transceiver is still alive */
 	if (sched->fn_counter_lost++ == TRX_LOSS_FRAMES) {
 		LOGP(DSCH, LOGL_NOTICE, "No more clock from transceiver\n");
-
-		osmo_fsm_inst_dispatch(trxcon_fsm, SCH_EVENT_CLCK_LOSS, trx);
 		sched->state = SCH_CLCK_STATE_WAIT;
 
 		return;
@@ -74,7 +70,6 @@
 		LOGP(DSCH, LOGL_NOTICE, "PC clock skew: "
 			"elapsed uS %d\n", elapsed);
 
-		osmo_fsm_inst_dispatch(trxcon_fsm, SCH_EVENT_CLCK_LOSS, trx);
 		sched->state = SCH_CLCK_STATE_WAIT;
 
 		return;
@@ -122,7 +117,6 @@
 
 int sched_clck_handle(struct trx_sched *sched, uint32_t fn)
 {
-	struct trx_instance *trx = (struct trx_instance *) sched->data;
 	struct timeval tv_now, *tv_clock;
 	int32_t elapsed, elapsed_fn;
 
@@ -138,7 +132,6 @@
 		sched_clck_correct(sched, &tv_now, fn);
 
 		LOGP(DSCH, LOGL_NOTICE, "Initial clock received: fn=%u\n", fn);
-		osmo_fsm_inst_dispatch(trxcon_fsm, SCH_EVENT_CLCK_IND, trx);
 		sched->state = SCH_CLCK_STATE_OK;
 
 		return 0;
diff --git a/src/host/trxcon/sched_trx.c b/src/host/trxcon/sched_trx.c
index be7a480..cc9b1ce 100644
--- a/src/host/trxcon/sched_trx.c
+++ b/src/host/trxcon/sched_trx.c
@@ -152,6 +152,9 @@
 	sched->fn_counter_proc = 0;
 	sched->fn_counter_lost = 0;
 
+	/* Reset internal state */
+	sched->state = SCH_CLCK_STATE_WAIT;
+
 	return 0;
 }
 
diff --git a/src/host/trxcon/trxcon.c b/src/host/trxcon/trxcon.c
index c0d5da3..f0fca16 100644
--- a/src/host/trxcon/trxcon.c
+++ b/src/host/trxcon/trxcon.c
@@ -99,8 +99,6 @@
 		break;
 	case TRX_EVENT_RSP_ERROR:
 	case TRX_EVENT_OFFLINE:
-	case SCH_EVENT_CLCK_IND:
-	case SCH_EVENT_CLCK_LOSS:
 	case TRX_EVENT_RESET_IND:
 		/* TODO: notify L2 & L3 about that */
 		break;
@@ -121,9 +119,7 @@
 			GEN_MASK(L1CTL_EVENT_DISCONNECT) |
 			GEN_MASK(TRX_EVENT_RESET_IND) |
 			GEN_MASK(TRX_EVENT_RSP_ERROR) |
-			GEN_MASK(TRX_EVENT_OFFLINE) |
-			GEN_MASK(SCH_EVENT_CLCK_IND) |
-			GEN_MASK(SCH_EVENT_CLCK_LOSS)),
+			GEN_MASK(TRX_EVENT_OFFLINE)),
 		.out_state_mask = GEN_MASK(TRXCON_STATE_IDLE),
 		.name = "MANAGED",
 		.action = trxcon_fsm_managed_action,
diff --git a/src/host/trxcon/trxcon.h b/src/host/trxcon/trxcon.h
index 17d3408..1ab3a14 100644
--- a/src/host/trxcon/trxcon.h
+++ b/src/host/trxcon/trxcon.h
@@ -19,8 +19,4 @@
 	TRX_EVENT_RESET_IND,
 	TRX_EVENT_RSP_ERROR,
 	TRX_EVENT_OFFLINE,
-
-	/* Scheduler specific events */
-	SCH_EVENT_CLCK_IND,
-	SCH_EVENT_CLCK_LOSS,
 };

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8efac459a40f4287e3325890809991d5ef46e9b1
Gerrit-PatchSet: 1
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>



More information about the gerrit-log mailing list