Change in osmo-bts[master]: bts-trx: Split part of bts_model_trx_close() steps into bts_model_dea...

[pespin]

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


Change subject: bts-trx: Split part of bts_model_trx_close() steps into bts_model_deact_rf
......................................................................

bts-trx: Split part of bts_model_trx_close() steps into bts_model_deact_rf

bts_model_trx_close is only called during bts_shutdown immediately after
bts_model_deact_rf, so its logic keeps being essentially the same after
this code movement.

On the other hand, bts_model_deact_rf is also called during RSL link
establishment if it failed for whatever reason in
bts.c:trx_link_estab(). In that case, we want to make sure the TRX is
not used so we need to implement bts_model_deact_rf.

Change-Id: Id4eae743da81773a04b82e7b454071b0cc66677a
---
M src/osmo-bts-trx/l1_if.c
1 file changed, 18 insertions(+), 7 deletions(-)



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

diff --git a/src/osmo-bts-trx/l1_if.c b/src/osmo-bts-trx/l1_if.c
index f068659..1848253 100644
--- a/src/osmo-bts-trx/l1_if.c
+++ b/src/osmo-bts-trx/l1_if.c
@@ -397,8 +397,8 @@
 	return oml_mo_opstart_ack(&trx->mo);
 }
 
-/* deactivate transceiver */
-int bts_model_trx_close(struct gsm_bts_trx *trx)
+/* Deact RF on transceiver */
+int bts_model_trx_deact_rf(struct gsm_bts_trx *trx)
 {
 	struct phy_instance *pinst = trx_phy_instance(trx);
 	struct trx_l1h *l1h = pinst->u.osmotrx.hdl;
@@ -412,6 +412,22 @@
 	    pchan == GSM_PCHAN_CCCH_SDCCH4_CBCH) {
 		lchan_set_state(&trx->ts[0].lchan[CCCH_LCHAN], LCHAN_S_INACTIVE);
 	}
+	/* FIXME: There's currently no way to communicate to osmo-trx through
+	 * TRXC that a specific TRX processing shall be paused. Let's simply
+	 * make sure that at least we don't transmit with power on it:
+	 */
+	power_ramp_start(trx, 0, 1, NULL);
+
+	/* Set to Operational State: Disabled */
+	check_transceiver_availability_trx(l1h, 0);
+	return 0;
+}
+
+/* deactivate transceiver */
+int bts_model_trx_close(struct gsm_bts_trx *trx)
+{
+	struct phy_instance *pinst = trx_phy_instance(trx);
+	struct trx_l1h *l1h = pinst->u.osmotrx.hdl;
 
 	/* power off transceiver, if not already */
 	if (l1h->config.enabled) {
@@ -847,11 +863,6 @@
 	return oml_mo_statechg_ack(mo);
 }
 
-int bts_model_trx_deact_rf(struct gsm_bts_trx *trx)
-{
-	return 0;
-}
-
 int bts_model_oml_estab(struct gsm_bts *bts)
 {
 	return 0;

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Id4eae743da81773a04b82e7b454071b0cc66677a
Gerrit-Change-Number: 18910
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/20200618/9b680132/attachment.htm>