Change in osmo-bts[master]: vty: fix dump_lchan_trx_ts(): dump dedicated channels only

[laforge]

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

Change subject: vty: fix dump_lchan_trx_ts(): dump dedicated channels only
......................................................................

vty: fix dump_lchan_trx_ts(): dump dedicated channels only

It does not make sense to dump CCCH/CBCH as dedicated channels:

  OsmoBTS# show lchan
  BTS 0, TRX 0, Timeslot 0, Lchan 4: Type CCCH
    State: ACTIVE
    BS (Downlink) Power Control (autonomous):
      Channel reduction: 0 dB (max 0 dB)
      TRX reduction: 0 dB
      Actual / Nominal power: 13 dBm / 13 dBm
    MS (Uplink) Power Control (autonomous):
      Current power level: 0, -39 dBm (max 0, -39 dBm)
    Channel Mode / Codec: SIGNALLING
    LAPDm SAPIs: DCCH --, SACCH --
    Valid System Information: 0x00000060
    MS Timing Offset: 0, propagation delay: 0 symbols
    Radio Link Failure Counter 'S': 0

so let's only dump SDCCH, TCH/F, and TCH/H.

Change-Id: Id7880de56a93cc9fa4ca576b094cef35ee269822
---
M src/common/vty.c
1 file changed, 9 insertions(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/common/vty.c b/src/common/vty.c
index ada871a..7284bfa 100644
--- a/src/common/vty.c
+++ b/src/common/vty.c
@@ -1449,7 +1449,15 @@
 		const struct gsm_lchan *lchan = &ts->lchan[lchan_nr];
 		if (lchan->state == LCHAN_S_NONE)
 			continue;
-		dump_cb(vty, lchan);
+		switch (lchan->type) {
+		case GSM_LCHAN_SDCCH:
+		case GSM_LCHAN_TCH_F:
+		case GSM_LCHAN_TCH_H:
+			dump_cb(vty, lchan);
+			break;
+		default:
+			continue;
+		}
 	}
 
 	return CMD_SUCCESS;

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/21450
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Id7880de56a93cc9fa4ca576b094cef35ee269822
Gerrit-Change-Number: 21450
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201210/1b69538b/attachment.htm>