Harald Welte has submitted this change and it was merged.

Change subject: "show timeslot": Show dynamic PDCH state also for Osmocom-style 
dyn PDCH
......................................................................


"show timeslot": Show dynamic PDCH state also for Osmocom-style dyn PDCH

So far we only used to show the current mode of the timeslot for IPA
style dynamic PDCH, but not for osmocom-style.

Old output for osmocom-style dyn PDCH:

BTS 0, TRX 0, Timeslot 6, phys cfg TCH/F_TCH/H_PDCH, TSC 7
  NM State: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'

new output with this patch:

BTS 0, TRX 0, Timeslot 6, phys cfg TCH/F_TCH/H_PDCH, TSC 7 (PDCH mode)
  NM State: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'

where "PDCH mode" changes from "NONE mode" to "TCH/F mode" to "TCH/H mode"
depending on the current state.

Change-Id: If5044d6bec76b869bd6a7c0260a4c77e229dcd0d
Related: OS#3099
---
M src/libbsc/bsc_vty.c
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/libbsc/bsc_vty.c b/src/libbsc/bsc_vty.c
index 6daeb98..13e123b 100644
--- a/src/libbsc/bsc_vty.c
+++ b/src/libbsc/bsc_vty.c
@@ -1099,9 +1099,12 @@
        vty_out(vty, "BTS %u, TRX %u, Timeslot %u, phys cfg %s, TSC %u",
                ts->trx->bts->nr, ts->trx->nr, ts->nr,
                gsm_pchan_name(ts->pchan), gsm_ts_tsc(ts));
-       if (ts->pchan == GSM_PCHAN_TCH_F_PDCH)
+       if (ts->pchan == GSM_PCHAN_TCH_F_PDCH) {
                vty_out(vty, " (%s mode)",
                        ts->flags & TS_F_PDCH_ACTIVE ? "PDCH" : "TCH/F");
+       } else if (ts->pchan == GSM_PCHAN_TCH_F_TCH_H_PDCH) {
+               vty_out(vty, " (%s mode)", gsm_pchan_name(ts->dyn.pchan_is));
+       }
        vty_out(vty, "%s", VTY_NEWLINE);
        vty_out(vty, "  NM State: ");
        net_dump_nmstate(vty, &ts->mo.nm_state);

-- 
To view, visit https://gerrit.osmocom.org/7663
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If5044d6bec76b869bd6a7c0260a4c77e229dcd0d
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder

Reply via email to