Pau Espin Pedrol has submitted this change and it was merged. (
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(-)
Approvals:
Jenkins Builder: Verified
Harald Welte: Looks good to me, approved
diff --git a/src/common/bts.c b/src/common/bts.c
index 68cb167..2b17ec6 100644
--- a/src/common/bts.c
+++ b/src/common/bts.c
@@ -322,13 +322,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: merged
Gerrit-Change-Id: I1ba32c162a8a98ed0fdd4190227de66270679e5a
Gerrit-Change-Number: 11767
Gerrit-PatchSet: 4
Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de>
