diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2020-06-18 17:43:54 +0200 |
|---|---|---|
| committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2020-06-23 14:53:37 +0200 |
| commit | ceb80fc6b6edcc701a423d6bfe8601b08ff3a10d (patch) | |
| tree | eb079b736d0c29e146c5ae5bd35786363d2b4abc | |
| parent | 0211bd6e79a9a20b25b429422f86b7ca9de742c5 (diff) | |
bts_shutdown: First deact RF on all TRX, finally close them
It makes more sense to first deactive RF on all TRX and finally close
them totally. This way we keep consistency between all TRXs and it's
easier for lower layers which may need to close them all at once. Also
in the event that we want to turn bts_model_trx_deact_rf to return
asynchronously.
Change-Id: Ib62358384e37a5cef692926439462042fab0138c
| -rw-r--r-- | src/common/bts_shutdown_fsm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/common/bts_shutdown_fsm.c b/src/common/bts_shutdown_fsm.c index faa92a0b..e65fff42 100644 --- a/src/common/bts_shutdown_fsm.c +++ b/src/common/bts_shutdown_fsm.c @@ -90,6 +90,8 @@ static void st_exit_on_enter(struct osmo_fsm_inst *fi, uint32_t prev_state) struct gsm_bts_trx *trx; llist_for_each_entry_reverse(trx, &bts->trx_list, list) { bts_model_trx_deact_rf(trx); + } + llist_for_each_entry_reverse(trx, &bts->trx_list, list) { bts_model_trx_close(trx); } /* There's yet no way to get confirmation from lower layers regarding |