Pau Espin Pedrol has uploaded this change for review. ( 
https://gerrit.osmocom.org/11767


Change subject: common: Implement OML for trx_set_available(1)
......................................................................

common: Implement OML for trx_set_available(1)

Related: OS#3692
Change-Id: I1ba32c162a8a98ed0fdd4190227de66270679e5a
---
M src/common/bts.c
1 file changed, 4 insertions(+), 6 deletions(-)



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

diff --git a/src/common/bts.c b/src/common/bts.c
index c251fdd..16f35ef 100644
--- a/src/common/bts.c
+++ b/src/common/bts.c
@@ -313,13 +313,11 @@
        LOGP(DSUM, LOGL_INFO, "TRX(%d): Setting available = %d\n",
                trx->nr, avail);
        if (avail) {
-               /* FIXME: This needs to be sorted out */
-#if 0
-               oml_mo_state_chg(&trx->mo,  NM_OPSTATE_DISABLED, 
NM_AVSTATE_OFF_LINE);
-               oml_mo_state_chg(&trx->bb_transc.mo, -1, NM_AVSTATE_OFF_LINE);
+               int op_state = trx->rsl_link ?  NM_OPSTATE_ENABLED : 
NM_OPSTATE_DISABLED;
+               oml_mo_state_chg(&trx->mo, op_state, NM_AVSTATE_OK);
+               oml_mo_state_chg(&trx->bb_transc.mo, -1, NM_AVSTATE_OK);
                for (tn = 0; tn < ARRAY_SIZE(trx->ts); tn++)
-                       oml_mo_state_chg(&trx->ts[tn].mo, NM_OPSTATE_DISABLED, 
NM_AVSTATE_DEPENDENCY);
-#endif
+                       oml_mo_state_chg(&trx->ts[tn].mo, op_state, 
NM_AVSTATE_OK);
        } else {
                oml_mo_state_chg(&trx->mo,  NM_OPSTATE_DISABLED, 
NM_AVSTATE_NOT_INSTALLED);
                oml_mo_state_chg(&trx->bb_transc.mo, -1, 
NM_AVSTATE_NOT_INSTALLED);

--
To view, visit https://gerrit.osmocom.org/11767
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1ba32c162a8a98ed0fdd4190227de66270679e5a
Gerrit-Change-Number: 11767
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de>

Reply via email to