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.orgHarald Welte has submitted this change and it was merged. Change subject: Start Dynamic PDCH Initialization after RSL is up ...................................................................... Start Dynamic PDCH Initialization after RSL is up We used to start the dynamic PDCH initialization right after the respective OML initialization of the related timeslot. However, this is problematic as the RSL link may very well not even be up yet at that point. So let's do this at RSL link esetablishment time, not from OML. Change-Id: I8ac30ca075a1db4b624dc38a423d844d705fda7e Closes: OS#1841 --- M src/libbsc/abis_om2000.c M src/libbsc/bsc_init.c M src/libbsc/bts_ipaccess_nanobts.c 3 files changed, 3 insertions(+), 13 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/libbsc/abis_om2000.c b/src/libbsc/abis_om2000.c index 756cc89..6057bc6 100644 --- a/src/libbsc/abis_om2000.c +++ b/src/libbsc/abis_om2000.c @@ -1683,15 +1683,6 @@ static void om2k_mo_st_wait_opinfo_accept(struct osmo_fsm_inst *fi, uint32_t event, void *data) { - struct om2k_mo_fsm_priv *omfp = fi->priv; - - /* if we have just received opinfo accept for the timeslot, - * start dynamic TCH switching procedures */ - if (omfp->mo->addr.class == OM2K_MO_CLS_TS) { - struct gsm_bts_trx_ts *ts; - ts = mo2obj(omfp->trx->bts, &omfp->mo->addr); - dyn_ts_init(ts); - } osmo_fsm_inst_state_chg(fi, OM2K_ST_DONE, 0, 0); } diff --git a/src/libbsc/bsc_init.c b/src/libbsc/bsc_init.c index f164533..ba8b8e6 100644 --- a/src/libbsc/bsc_init.c +++ b/src/libbsc/bsc_init.c @@ -350,8 +350,10 @@ rsl_nokia_si_end(trx); } - for (i = 0; i < ARRAY_SIZE(trx->ts); i++) + for (i = 0; i < ARRAY_SIZE(trx->ts); i++) { generate_ma_for_ts(&trx->ts[i]); + dyn_ts_init(&trx->ts[i]); + } if (acc_ramp_is_enabled(&trx->bts->acc_ramp)) acc_ramp_start(&trx->bts->acc_ramp); diff --git a/src/libbsc/bts_ipaccess_nanobts.c b/src/libbsc/bts_ipaccess_nanobts.c index edfe417..4f1ac4b 100644 --- a/src/libbsc/bts_ipaccess_nanobts.c +++ b/src/libbsc/bts_ipaccess_nanobts.c @@ -175,9 +175,6 @@ abis_nm_opstart(trx->bts, obj_class, trx->bts->bts_nr, trx->nr, ts->nr); } - if (new_state->operational == NM_OPSTATE_ENABLED - && new_state->availability == NM_AVSTATE_OK) - dyn_ts_init(ts); break; case NM_OC_RADIO_CARRIER: trx = obj; -- To view, visit https://gerrit.osmocom.org/7661 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ac30ca075a1db4b624dc38a423d844d705fda7e Gerrit-PatchSet: 1 Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Owner: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Jenkins Builder