laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/28895 )

Change subject: bts_trx_vty: prefix bb_trxc mo with [Virtual] for ericsson BTS ......................................................................

bts_trx_vty: prefix bb_trxc mo with [Virtual] for ericsson BTS

The OM2000 model does not have a separate bb_transc MO, however for compatibilty reasons we have a virtual MO that just mirrors the state of the TRX mo. We should mark it as [Virtual] in show trx to reflect this to the user.

Change-Id: I0f5501f6fbc7ce6d5457676b16c7f93f70db5763 Related: OS#5101 --- M src/osmo-bsc/bts_trx_vty.c 1 file changed, 4 insertions(+), 2 deletions(-)

Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified

diff --git a/src/osmo-bsc/bts_trx_vty.c b/src/osmo-bsc/bts_trx_vty.c index bd13a05..ef4b96d 100644 --- a/src/osmo-bsc/bts_trx_vty.c +++ b/src/osmo-bsc/bts_trx_vty.c @@ -738,8 +738,10 @@ net_dump_nmstate(vty, &trx->mo.nm_state); if (print_rsl) vty_out(vty, " RSL State: %s%s", trx->rsl_link_primary? "connected" : "disconnected", VTY_NEWLINE); - vty_out(vty, " Baseband Transceiver NM State: "); - net_dump_nmstate(vty, &trx->bb_transc.mo.nm_state); + + vty_out(vty, " %sBaseband Transceiver NM State: ", is_ericsson_bts(trx->bts) ? "[Virtual] " : ""); + net_dump_nmstate(vty, &trx->bb_transc.mo.nm_state); + if (is_ipaccess_bts(trx->bts)) { vty_out(vty, " ip.access stream ID: 0x%02x ", trx->rsl_tei_primary); e1isl_dump_vty_tcp(vty, trx->rsl_link_primary);

3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.