Change in osmo-bts[master]: bts: Clean up TS selection in sign_link_up

[pespin]

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/24319 )


Change subject: bts: Clean up TS selection in sign_link_up
......................................................................

bts: Clean up TS selection in sign_link_up

Change-Id: I0c92dfd05bf2ae40887980ef10b7e4c679213b6a
---
M src/common/abis.c
M tests/handover/handover_test.c
2 files changed, 11 insertions(+), 7 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/19/24319/1

diff --git a/src/common/abis.c b/src/common/abis.c
index be7e906..3dc7650 100644
--- a/src/common/abis.c
+++ b/src/common/abis.c
@@ -100,15 +100,17 @@
 					    enum e1inp_sign_type type)
 {
 	struct e1inp_sign_link *sign_link = NULL;
+	struct e1inp_ts *sign_ts;
 	struct gsm_bts_trx *trx;
 	int trx_nr;
 
 	switch (type) {
 	case E1INP_SIGN_OML:
+		sign_ts = e1inp_line_ipa_oml_ts(line);
 		LOGP(DABIS, LOGL_INFO, "OML Signalling link up\n");
-		e1inp_ts_config_sign(&line->ts[E1INP_SIGN_OML-1], line);
+		e1inp_ts_config_sign(sign_ts, line);
 		sign_link = g_bts->oml_link =
-			e1inp_sign_link_create(&line->ts[E1INP_SIGN_OML-1],
+			e1inp_sign_link_create(sign_ts,
 						E1INP_SIGN_OML, g_bts->c0, 255, 0);
 		if (clock_gettime(CLOCK_MONOTONIC, &g_bts->oml_conn_established_timestamp) != 0)
 			memset(&g_bts->oml_conn_established_timestamp, 0,
@@ -118,6 +120,7 @@
 		break;
 	default:
 		trx_nr = type - E1INP_SIGN_RSL;
+		sign_ts = e1inp_line_ipa_rsl_ts(line, trx_nr);
 		LOGP(DABIS, LOGL_INFO, "RSL Signalling link for TRX%d up\n",
 			trx_nr);
 		trx = gsm_bts_trx_num(g_bts, trx_nr);
@@ -126,9 +129,9 @@
 				trx_nr);
 			break;
 		}
-		e1inp_ts_config_sign(&line->ts[type-1], line);
+		e1inp_ts_config_sign(sign_ts, line);
 		sign_link = trx->rsl_link =
-			e1inp_sign_link_create(&line->ts[type-1],
+			e1inp_sign_link_create(sign_ts,
 						E1INP_SIGN_RSL, trx,
 						trx->rsl_tei, 0);
 		trx_link_estab(trx);
diff --git a/tests/handover/handover_test.c b/tests/handover/handover_test.c
index 6af8fac..9c9b020 100644
--- a/tests/handover/handover_test.c
+++ b/tests/handover/handover_test.c
@@ -58,6 +58,7 @@
 {
 	void *tall_bts_ctx;
 	struct e1inp_line *line;
+	struct e1inp_ts *sign_ts;
 	struct gsm_lchan *lchan;
 	struct osmo_phsap_prim nl1sap;
 	struct msgb *msg;
@@ -90,9 +91,9 @@
 
 	line = e1inp_line_create(0, "ipa");
 	OSMO_ASSERT(line);
-
-	e1inp_ts_config_sign(&line->ts[E1INP_SIGN_RSL-1], line);
-	trx->rsl_link = e1inp_sign_link_create(&line->ts[E1INP_SIGN_RSL-1], E1INP_SIGN_RSL, NULL, 0, 0);
+	sign_ts = e1inp_line_ipa_rsl_ts(line, 0);
+	e1inp_ts_config_sign(sign_ts, line);
+	trx->rsl_link = e1inp_sign_link_create(sign_ts, E1INP_SIGN_RSL, NULL, 0, 0);
 	OSMO_ASSERT(trx->rsl_link);
 	trx->rsl_link->trx = trx;
 

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I0c92dfd05bf2ae40887980ef10b7e4c679213b6a
Gerrit-Change-Number: 24319
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210520/16dc2f8b/attachment.htm>