path: root/src/common/vty.c
diff options
authorPau Espin Pedrol <pespin@sysmocom.de>2018-11-21 19:49:12 +0100
committerPau Espin Pedrol <pespin@sysmocom.de>2018-11-21 19:59:21 +0100
commit6dacc35c751452f0e94acf15e6fd42ac774c8e53 (patch)
tree1b0ffdbd4ffa49ab4a66520051cf19c6a757a1d7 /src/common/vty.c
parent36a61df7a6f4a644eb5feecf9981d65cb355ff12 (diff)
bts: Fix TRX0 param initialization
TRX0 is allocated in a different place than other TRX. Commit that introduced the bug added an initialization step (bts_trx_init) applied in VTY cmd "trx <0-254>", where TRX are allocated since same commit, but doesn't initialize TRX0 for reasons explained in first paragraph. As a result, some parameters were not applied to TRX0, like power ramping increment step, which prevented MS from finding the network in osmo-gsm-tester tests. Fixes: eebb6a4216f136132400831ca35ed70e7502e92a ("bts: Allocate TRX for BTS dynamically, deprecate -t") Change-Id: I5afd66548634ff53e2af24dc7055d408ca8ca4e3
Diffstat (limited to 'src/common/vty.c')
1 files changed, 4 insertions, 1 deletions
diff --git a/src/common/vty.c b/src/common/vty.c
index f3054139..7c2900f2 100644
--- a/src/common/vty.c
+++ b/src/common/vty.c
@@ -233,7 +233,10 @@ DEFUN(cfg_bts_trx, cfg_bts_trx_cmd,
bts->num_trx, VTY_NEWLINE);
} else if (trx_nr == bts->num_trx) {
- /* allocate a new one */
+ /* Allocate a new TRX
+ * Remark: TRX0 was already created during gsm_bts_alloc() and
+ * initialized in bts_init(), not here.
+ */
trx = gsm_bts_trx_alloc(bts);
if (trx)