<p>fixeria has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20288">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vty: make most struct pointers const in show/write commands<br><br>Change-Id: Iacc32f7982c150d84ea4df7affa1f9e07806928f<br>---<br>M include/osmo-bts/bts.h<br>M include/osmo-bts/bts_model.h<br>M include/osmo-bts/phy_link.h<br>M src/common/l1sap.c<br>M src/common/phy_link.c<br>M src/common/vty.c<br>M src/osmo-bts-litecell15/lc15bts_vty.c<br>M src/osmo-bts-oc2g/oc2gbts_vty.c<br>M src/osmo-bts-octphy/octphy_vty.c<br>M src/osmo-bts-sysmo/sysmobts_vty.c<br>M src/osmo-bts-trx/trx_vty.c<br>M src/osmo-bts-virtual/virtualbts_vty.c<br>12 files changed, 77 insertions(+), 77 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/88/20288/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmo-bts/bts.h b/include/osmo-bts/bts.h</span><br><span>index 7d3f8bf..ffaeb57 100644</span><br><span>--- a/include/osmo-bts/bts.h</span><br><span>+++ b/include/osmo-bts/bts.h</span><br><span>@@ -347,7 +347,7 @@</span><br><span> int bts_supports_cm(struct gsm_bts *bts, enum gsm_phys_chan_config pchan,</span><br><span>                 enum gsm48_chan_mode cm);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-int32_t bts_get_avg_fn_advance(struct gsm_bts *bts);</span><br><span style="color: hsl(120, 100%, 40%);">+int32_t bts_get_avg_fn_advance(const struct gsm_bts *bts);</span><br><span> </span><br><span> /* return the gsm_lchan for the CBCH (if it exists at all) */</span><br><span> struct gsm_lchan *gsm_bts_get_cbch(struct gsm_bts *bts);</span><br><span>diff --git a/include/osmo-bts/bts_model.h b/include/osmo-bts/bts_model.h</span><br><span>index 4fd0813..568ff00 100644</span><br><span>--- a/include/osmo-bts/bts_model.h</span><br><span>+++ b/include/osmo-bts/bts_model.h</span><br><span>@@ -36,10 +36,10 @@</span><br><span> </span><br><span> int bts_model_vty_init(struct gsm_bts *bts);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_bts(struct vty *vty, struct gsm_bts *bts);</span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_trx(struct vty *vty, struct gsm_bts_trx *trx);</span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy(struct vty *vty, struct phy_link *plink);</span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy_inst(struct vty *vty, struct phy_instance *pinst);</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_bts(struct vty *vty, const struct gsm_bts *bts);</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_trx(struct vty *vty, const struct gsm_bts_trx *trx);</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy(struct vty *vty, const struct phy_link *plink);</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy_inst(struct vty *vty, const struct phy_instance *pinst);</span><br><span> </span><br><span> int bts_model_oml_estab(struct gsm_bts *bts);</span><br><span> </span><br><span>diff --git a/include/osmo-bts/phy_link.h b/include/osmo-bts/phy_link.h</span><br><span>index ce4ea45..ef3df27 100644</span><br><span>--- a/include/osmo-bts/phy_link.h</span><br><span>+++ b/include/osmo-bts/phy_link.h</span><br><span>@@ -164,7 +164,7 @@</span><br><span> const char *phy_link_state_name(enum phy_link_state state);</span><br><span> int phy_links_open(void);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-struct phy_instance *phy_instance_by_num(struct phy_link *plink, int num);</span><br><span style="color: hsl(120, 100%, 40%);">+struct phy_instance *phy_instance_by_num(const struct phy_link *plink, int num);</span><br><span> struct phy_instance *phy_instance_create(struct phy_link *plink, int num);</span><br><span> void phy_instance_link_to_trx(struct phy_instance *pinst, struct gsm_bts_trx *trx);</span><br><span> void phy_instance_destroy(struct phy_instance *pinst);</span><br><span>diff --git a/src/common/l1sap.c b/src/common/l1sap.c</span><br><span>index eac5000..ca1949a 100644</span><br><span>--- a/src/common/l1sap.c</span><br><span>+++ b/src/common/l1sap.c</span><br><span>@@ -882,7 +882,7 @@</span><br><span> }</span><br><span> </span><br><span> /* return the measured average of frame numbers that the RTS clock is running in advance */</span><br><span style="color: hsl(0, 100%, 40%);">-int32_t bts_get_avg_fn_advance(struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+int32_t bts_get_avg_fn_advance(const struct gsm_bts *bts)</span><br><span> {</span><br><span>         if (bts->fn_stats.avg_count == 0)</span><br><span>                 return 0;</span><br><span>diff --git a/src/common/phy_link.c b/src/common/phy_link.c</span><br><span>index d13d3f1..48ba283 100644</span><br><span>--- a/src/common/phy_link.c</span><br><span>+++ b/src/common/phy_link.c</span><br><span>@@ -79,7 +79,7 @@</span><br><span>       return get_value_string(phy_link_state_vals, state);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-struct phy_instance *phy_instance_by_num(struct phy_link *plink, int num)</span><br><span style="color: hsl(120, 100%, 40%);">+struct phy_instance *phy_instance_by_num(const struct phy_link *plink, int num)</span><br><span> {</span><br><span>         struct phy_instance *pinst;</span><br><span> </span><br><span>diff --git a/src/common/vty.c b/src/common/vty.c</span><br><span>index dd83a11..b662320 100644</span><br><span>--- a/src/common/vty.c</span><br><span>+++ b/src/common/vty.c</span><br><span>@@ -216,9 +216,9 @@</span><br><span>   return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void config_write_bts_single(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+static void config_write_bts_single(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-     struct gsm_bts_trx *trx;</span><br><span style="color: hsl(120, 100%, 40%);">+      const struct gsm_bts_trx *trx;</span><br><span>       const char *sapi_buf;</span><br><span>        int i;</span><br><span> </span><br><span>@@ -281,8 +281,8 @@</span><br><span>     bts_model_config_write_bts(vty, bts);</span><br><span> </span><br><span>    llist_for_each_entry(trx, &bts->trx_list, list) {</span><br><span style="color: hsl(0, 100%, 40%);">-                struct trx_power_params *tpp = &trx->power_params;</span><br><span style="color: hsl(0, 100%, 40%);">-               struct phy_instance *pinst = trx_phy_instance(trx);</span><br><span style="color: hsl(120, 100%, 40%);">+           const struct trx_power_params *tpp = &trx->power_params;</span><br><span style="color: hsl(120, 100%, 40%);">+               const struct phy_instance *pinst = trx_phy_instance(trx);</span><br><span>            vty_out(vty, " trx %u%s", trx->nr, VTY_NEWLINE);</span><br><span> </span><br><span>            if (trx->power_params.user_gain_mdB)</span><br><span>@@ -307,7 +307,7 @@</span><br><span> static int config_write_bts(struct vty *vty)</span><br><span> {</span><br><span>   struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(0, 100%, 40%);">- struct gsm_bts *bts;</span><br><span style="color: hsl(120, 100%, 40%);">+  const struct gsm_bts *bts;</span><br><span> </span><br><span>       llist_for_each_entry(bts, &net->bts_list, list)</span><br><span>               config_write_bts_single(vty, bts);</span><br><span>@@ -315,7 +315,7 @@</span><br><span>     return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void config_write_phy_single(struct vty *vty, struct phy_link *plink)</span><br><span style="color: hsl(120, 100%, 40%);">+static void config_write_phy_single(struct vty *vty, const struct phy_link *plink)</span><br><span> {</span><br><span>    int i;</span><br><span> </span><br><span>@@ -323,7 +323,7 @@</span><br><span>     bts_model_config_write_phy(vty, plink);</span><br><span> </span><br><span>  for (i = 0; i < 255; i++) {</span><br><span style="color: hsl(0, 100%, 40%);">-          struct phy_instance *pinst = phy_instance_by_num(plink, i);</span><br><span style="color: hsl(120, 100%, 40%);">+           const struct phy_instance *pinst = phy_instance_by_num(plink, i);</span><br><span>            if (!pinst)</span><br><span>                  break;</span><br><span>               vty_out(vty, " instance %u%s", pinst->num, VTY_NEWLINE);</span><br><span>@@ -336,7 +336,7 @@</span><br><span>  int i;</span><br><span> </span><br><span>   for (i = 0; i < 255; i++) {</span><br><span style="color: hsl(0, 100%, 40%);">-          struct phy_link *plink = phy_link_by_num(i);</span><br><span style="color: hsl(120, 100%, 40%);">+          const struct phy_link *plink = phy_link_by_num(i);</span><br><span>           if (!plink)</span><br><span>                  break;</span><br><span>               config_write_phy_single(vty, plink);</span><br><span>@@ -815,7 +815,7 @@</span><br><span>  * SHOW</span><br><span>  * ======================================================================*/</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void net_dump_nmstate(struct vty *vty, struct gsm_nm_state *nms)</span><br><span style="color: hsl(120, 100%, 40%);">+static void net_dump_nmstate(struct vty *vty, const struct gsm_nm_state *nms)</span><br><span> {</span><br><span>  vty_out(vty,"Oper '%s', Admin '%s', Avail '%s'%s",</span><br><span>                 abis_nm_opstate_name(nms->operational),</span><br><span>@@ -823,7 +823,7 @@</span><br><span>             abis_nm_avail_name(nms->availability), VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void bts_dump_vty_features(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+static void bts_dump_vty_features(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span>  unsigned int i;</span><br><span>      bool no_features;</span><br><span>@@ -855,9 +855,9 @@</span><br><span>              vty_out(vty, "    (not available)%s", VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void bts_dump_vty(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+static void bts_dump_vty(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">- struct gsm_bts_trx *trx;</span><br><span style="color: hsl(120, 100%, 40%);">+      const struct gsm_bts_trx *trx;</span><br><span> </span><br><span>   vty_out(vty, "BTS %u is of %s type in band %s, has CI %u LAC %u, "</span><br><span>                 "BSIC %u and %u TRX%s",</span><br><span>@@ -904,7 +904,7 @@</span><br><span>              bts_get_avg_fn_advance(bts), bts->fn_stats.min, bts->fn_stats.max, VTY_NEWLINE);</span><br><span> </span><br><span>   llist_for_each_entry(trx, &bts->trx_list, list) {</span><br><span style="color: hsl(0, 100%, 40%);">-                struct phy_instance *pinst = trx_phy_instance(trx);</span><br><span style="color: hsl(120, 100%, 40%);">+           const struct phy_instance *pinst = trx_phy_instance(trx);</span><br><span>            vty_out(vty, "  TRX %u%s", trx->nr, VTY_NEWLINE);</span><br><span>               if (pinst) {</span><br><span>                         vty_out(vty, "    phy %d %s", pinst->num, pinst->version);</span><br><span>@@ -923,7 +923,7 @@</span><br><span>     SHOW_STR "Display information about a BTS\n"</span><br><span>               BTS_NR_STR)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-        struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(120, 100%, 40%);">+       const struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span>        int bts_nr;</span><br><span> </span><br><span>      if (argc != 0) {</span><br><span>@@ -948,9 +948,9 @@</span><br><span>       "Various testing commands\n"</span><br><span>       "Send a test OML failure event report to the BSC\n" BTS_NR_STR)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-  struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(120, 100%, 40%);">+       const struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span>        int bts_nr = atoi(argv[0]);</span><br><span style="color: hsl(0, 100%, 40%);">-     struct gsm_bts *bts;</span><br><span style="color: hsl(120, 100%, 40%);">+  const struct gsm_bts *bts;</span><br><span> </span><br><span>       if (bts_nr >= net->num_bts) {</span><br><span>          vty_out(vty, "%% can't find BTS '%s'%s", argv[0], VTY_NEWLINE);</span><br><span>@@ -963,7 +963,7 @@</span><br><span>  return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void trx_dump_vty(struct vty *vty, struct gsm_bts_trx *trx)</span><br><span style="color: hsl(120, 100%, 40%);">+static void trx_dump_vty(struct vty *vty, const struct gsm_bts_trx *trx)</span><br><span> {</span><br><span>        vty_out(vty, "TRX %u of BTS %u is on ARFCN %u%s",</span><br><span>          trx->nr, trx->bts->nr, trx->arfcn, VTY_NEWLINE);</span><br><span>@@ -994,8 +994,8 @@</span><br><span>   SHOW_STR "Display information about a TRX\n"</span><br><span>       BTS_TRX_STR)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-       struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(0, 100%, 40%);">- struct gsm_bts *bts = NULL;</span><br><span style="color: hsl(120, 100%, 40%);">+   const struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct gsm_bts *bts = NULL;</span><br><span>    int bts_nr, trx_nr;</span><br><span> </span><br><span>      if (argc >= 1) {</span><br><span>@@ -1031,7 +1031,7 @@</span><br><span> }</span><br><span> </span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void ts_dump_vty(struct vty *vty, struct gsm_bts_trx_ts *ts)</span><br><span style="color: hsl(120, 100%, 40%);">+static void ts_dump_vty(struct vty *vty, const struct gsm_bts_trx_ts *ts)</span><br><span> {</span><br><span>  vty_out(vty, "BTS %u, TRX %u, Timeslot %u, phys cfg %s, TSC %u",</span><br><span>           ts->trx->bts->nr, ts->trx->nr, ts->nr,</span><br><span>@@ -1050,10 +1050,10 @@</span><br><span>   SHOW_STR "Display information about a TS\n"</span><br><span>        BTS_TRX_TS_STR)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-    struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(0, 100%, 40%);">- struct gsm_bts *bts = NULL;</span><br><span style="color: hsl(0, 100%, 40%);">-     struct gsm_bts_trx *trx = NULL;</span><br><span style="color: hsl(0, 100%, 40%);">- struct gsm_bts_trx_ts *ts = NULL;</span><br><span style="color: hsl(120, 100%, 40%);">+     const struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct gsm_bts *bts = NULL;</span><br><span style="color: hsl(120, 100%, 40%);">+     const struct gsm_bts_trx *trx = NULL;</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct gsm_bts_trx_ts *ts = NULL;</span><br><span>      int bts_nr, trx_nr, ts_nr;</span><br><span> </span><br><span>       if (argc >= 1) {</span><br><span>@@ -1122,7 +1122,7 @@</span><br><span> </span><br><span> /* call vty_out() to print a string like " as TCH/H" for dynamic timeslots.</span><br><span>  * Don't do anything if the ts is not dynamic. */</span><br><span style="color: hsl(0, 100%, 40%);">-static void vty_out_dyn_ts_status(struct vty *vty, struct gsm_bts_trx_ts *ts)</span><br><span style="color: hsl(120, 100%, 40%);">+static void vty_out_dyn_ts_status(struct vty *vty, const struct gsm_bts_trx_ts *ts)</span><br><span> {</span><br><span>     switch (ts->pchan) {</span><br><span>      case GSM_PCHAN_TCH_F_TCH_H_PDCH:</span><br><span>@@ -1152,7 +1152,7 @@</span><br><span>     }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void lchan_dump_full_vty(struct vty *vty, struct gsm_lchan *lchan)</span><br><span style="color: hsl(120, 100%, 40%);">+static void lchan_dump_full_vty(struct vty *vty, const struct gsm_lchan *lchan)</span><br><span> {</span><br><span>    struct in_addr ia;</span><br><span> </span><br><span>@@ -1228,7 +1228,7 @@</span><br><span>       /* TODO: MS Power Control */</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void lchan_dump_short_vty(struct vty *vty, struct gsm_lchan *lchan)</span><br><span style="color: hsl(120, 100%, 40%);">+static void lchan_dump_short_vty(struct vty *vty, const struct gsm_lchan *lchan)</span><br><span> {</span><br><span>       const struct gsm_meas_rep_unidir *mru = &lchan->meas.ul_res;</span><br><span> </span><br><span>@@ -1244,12 +1244,12 @@</span><br><span>            VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static int dump_lchan_trx_ts(struct gsm_bts_trx_ts *ts, struct vty *vty,</span><br><span style="color: hsl(0, 100%, 40%);">-                       void (*dump_cb)(struct vty *, struct gsm_lchan *))</span><br><span style="color: hsl(120, 100%, 40%);">+static int dump_lchan_trx_ts(const struct gsm_bts_trx_ts *ts, struct vty *vty,</span><br><span style="color: hsl(120, 100%, 40%);">+                            void (*dump_cb)(struct vty *, const struct gsm_lchan *))</span><br><span> {</span><br><span>   int lchan_nr;</span><br><span>        for (lchan_nr = 0; lchan_nr < TS_MAX_LCHAN; lchan_nr++) {</span><br><span style="color: hsl(0, 100%, 40%);">-            struct gsm_lchan *lchan = &ts->lchan[lchan_nr];</span><br><span style="color: hsl(120, 100%, 40%);">+                const struct gsm_lchan *lchan = &ts->lchan[lchan_nr];</span><br><span>                 if (lchan->state == LCHAN_S_NONE)</span><br><span>                         continue;</span><br><span>            dump_cb(vty, lchan);</span><br><span>@@ -1258,26 +1258,26 @@</span><br><span>       return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static int dump_lchan_trx(struct gsm_bts_trx *trx, struct vty *vty,</span><br><span style="color: hsl(0, 100%, 40%);">-                   void (*dump_cb)(struct vty *, struct gsm_lchan *))</span><br><span style="color: hsl(120, 100%, 40%);">+static int dump_lchan_trx(const struct gsm_bts_trx *trx, struct vty *vty,</span><br><span style="color: hsl(120, 100%, 40%);">+                         void (*dump_cb)(struct vty *, const struct gsm_lchan *))</span><br><span> {</span><br><span>      int ts_nr;</span><br><span> </span><br><span>       for (ts_nr = 0; ts_nr < TRX_NR_TS; ts_nr++) {</span><br><span style="color: hsl(0, 100%, 40%);">-                struct gsm_bts_trx_ts *ts = &trx->ts[ts_nr];</span><br><span style="color: hsl(120, 100%, 40%);">+           const struct gsm_bts_trx_ts *ts = &trx->ts[ts_nr];</span><br><span>            dump_lchan_trx_ts(ts, vty, dump_cb);</span><br><span>         }</span><br><span> </span><br><span>        return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static int dump_lchan_bts(struct gsm_bts *bts, struct vty *vty,</span><br><span style="color: hsl(0, 100%, 40%);">-                       void (*dump_cb)(struct vty *, struct gsm_lchan *))</span><br><span style="color: hsl(120, 100%, 40%);">+static int dump_lchan_bts(const struct gsm_bts *bts, struct vty *vty,</span><br><span style="color: hsl(120, 100%, 40%);">+                     void (*dump_cb)(struct vty *, const struct gsm_lchan *))</span><br><span> {</span><br><span>      int trx_nr;</span><br><span> </span><br><span>      for (trx_nr = 0; trx_nr < bts->num_trx; trx_nr++) {</span><br><span style="color: hsl(0, 100%, 40%);">-               struct gsm_bts_trx *trx = gsm_bts_trx_num(bts, trx_nr);</span><br><span style="color: hsl(120, 100%, 40%);">+               const struct gsm_bts_trx *trx = gsm_bts_trx_num(bts, trx_nr);</span><br><span>                dump_lchan_trx(trx, vty, dump_cb);</span><br><span>   }</span><br><span> </span><br><span>@@ -1285,13 +1285,13 @@</span><br><span> }</span><br><span> </span><br><span> static int lchan_summary(struct vty *vty, int argc, const char **argv,</span><br><span style="color: hsl(0, 100%, 40%);">-                       void (*dump_cb)(struct vty *, struct gsm_lchan *))</span><br><span style="color: hsl(120, 100%, 40%);">+                    void (*dump_cb)(struct vty *, const struct gsm_lchan *))</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-  struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(0, 100%, 40%);">- struct gsm_bts *bts;</span><br><span style="color: hsl(0, 100%, 40%);">-    struct gsm_bts_trx *trx;</span><br><span style="color: hsl(0, 100%, 40%);">-        struct gsm_bts_trx_ts *ts;</span><br><span style="color: hsl(0, 100%, 40%);">-      struct gsm_lchan *lchan;</span><br><span style="color: hsl(120, 100%, 40%);">+      const struct gsm_network *net = gsmnet_from_vty(vty);</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct gsm_bts *bts;</span><br><span style="color: hsl(120, 100%, 40%);">+    const struct gsm_bts_trx *trx;</span><br><span style="color: hsl(120, 100%, 40%);">+        const struct gsm_bts_trx_ts *ts;</span><br><span style="color: hsl(120, 100%, 40%);">+      const struct gsm_lchan *lchan;</span><br><span>       int bts_nr, trx_nr, ts_nr, lchan_nr;</span><br><span> </span><br><span>     if (argc >= 1) {</span><br><span>@@ -1370,8 +1370,8 @@</span><br><span>  return lchan_summary(vty, argc, argv, lchan_dump_short_vty);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static struct gsm_lchan *resolve_lchan(struct gsm_network *net,</span><br><span style="color: hsl(0, 100%, 40%);">-                                    const char **argv, int idx)</span><br><span style="color: hsl(120, 100%, 40%);">+static struct gsm_lchan *resolve_lchan(const struct gsm_network *net,</span><br><span style="color: hsl(120, 100%, 40%);">+                                   const char **argv, int idx)</span><br><span> {</span><br><span>      int bts_nr = atoi(argv[idx+0]);</span><br><span>      int trx_nr = atoi(argv[idx+1]);</span><br><span>diff --git a/src/osmo-bts-litecell15/lc15bts_vty.c b/src/osmo-bts-litecell15/lc15bts_vty.c</span><br><span>index 2cc812c..5308d1a 100644</span><br><span>--- a/src/osmo-bts-litecell15/lc15bts_vty.c</span><br><span>+++ b/src/osmo-bts-litecell15/lc15bts_vty.c</span><br><span>@@ -505,9 +505,9 @@</span><br><span>       return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_bts(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_bts(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-     struct bts_lc15_priv *bts_lc15 = bts->model_priv;</span><br><span style="color: hsl(120, 100%, 40%);">+  const struct bts_lc15_priv *bts_lc15 = bts->model_priv;</span><br><span>   vty_out(vty, " led-control-mode %s%s",</span><br><span>                     get_value_string(lc15_led_mode_strs, bts_lc15->led_ctrl_mode), VTY_NEWLINE);</span><br><span> </span><br><span>@@ -516,16 +516,16 @@</span><br><span> </span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_trx(struct vty *vty, struct gsm_bts_trx *trx)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_trx(struct vty *vty, const struct gsm_bts_trx *trx)</span><br><span> {</span><br><span>      vty_out(vty, "  nominal-tx-power %d%s", trx->nominal_power,VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy(struct vty *vty, struct phy_link *plink)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy(struct vty *vty, const struct phy_link *plink)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy_inst(struct vty *vty, struct phy_instance *pinst)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy_inst(struct vty *vty, const struct phy_instance *pinst)</span><br><span> {</span><br><span>  int i;</span><br><span> </span><br><span>diff --git a/src/osmo-bts-oc2g/oc2gbts_vty.c b/src/osmo-bts-oc2g/oc2gbts_vty.c</span><br><span>index 8dc4efa..c12d0bf 100644</span><br><span>--- a/src/osmo-bts-oc2g/oc2gbts_vty.c</span><br><span>+++ b/src/osmo-bts-oc2g/oc2gbts_vty.c</span><br><span>@@ -541,7 +541,7 @@</span><br><span> }</span><br><span> */</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_bts(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_bts(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span>      /* TODO(oramadan) MERGE</span><br><span>      struct gsm_bts_role_bts *btsb = bts_role_bts(bts);</span><br><span>@@ -555,16 +555,16 @@</span><br><span> </span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_trx(struct vty *vty, struct gsm_bts_trx *trx)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_trx(struct vty *vty, const struct gsm_bts_trx *trx)</span><br><span> {</span><br><span>       vty_out(vty, "  nominal-tx-power %d%s", trx->nominal_power,VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy(struct vty *vty, struct phy_link *plink)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy(struct vty *vty, const struct phy_link *plink)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy_inst(struct vty *vty, struct phy_instance *pinst)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy_inst(struct vty *vty, const struct phy_instance *pinst)</span><br><span> {</span><br><span>  int i;</span><br><span> </span><br><span>diff --git a/src/osmo-bts-octphy/octphy_vty.c b/src/osmo-bts-octphy/octphy_vty.c</span><br><span>index afc2ce7..8b263fd 100644</span><br><span>--- a/src/osmo-bts-octphy/octphy_vty.c</span><br><span>+++ b/src/osmo-bts-octphy/octphy_vty.c</span><br><span>@@ -360,7 +360,7 @@</span><br><span>        return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy(struct vty *vty, struct phy_link *plink)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy(struct vty *vty, const struct phy_link *plink)</span><br><span> {</span><br><span>    if (plink->u.octphy.netdev_name)</span><br><span>          vty_out(vty, " octphy net-device %s%s",</span><br><span>@@ -399,15 +399,15 @@</span><br><span> #endif</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy_inst(struct vty *vty, struct phy_instance *pinst)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy_inst(struct vty *vty, const struct phy_instance *pinst)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_bts(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_bts(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_trx(struct vty *vty, struct gsm_bts_trx *trx)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_trx(struct vty *vty, const struct gsm_bts_trx *trx)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span>diff --git a/src/osmo-bts-sysmo/sysmobts_vty.c b/src/osmo-bts-sysmo/sysmobts_vty.c</span><br><span>index 7876612..b613031 100644</span><br><span>--- a/src/osmo-bts-sysmo/sysmobts_vty.c</span><br><span>+++ b/src/osmo-bts-sysmo/sysmobts_vty.c</span><br><span>@@ -433,22 +433,22 @@</span><br><span> }</span><br><span> </span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_bts(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_bts(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_trx(struct vty *vty, struct gsm_bts_trx *trx)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_trx(struct vty *vty, const struct gsm_bts_trx *trx)</span><br><span> {</span><br><span>    if (trx->nominal_power != get_p_max_out_mdBm(trx))</span><br><span>                vty_out(vty, "  nominal-tx-power %d%s", trx->nominal_power,</span><br><span>                     VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy(struct vty *vty, struct phy_link *plink)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy(struct vty *vty, const struct phy_link *plink)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy_inst(struct vty *vty, struct phy_instance *pinst)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy_inst(struct vty *vty, const struct phy_instance *pinst)</span><br><span> {</span><br><span>  int i;</span><br><span> </span><br><span>diff --git a/src/osmo-bts-trx/trx_vty.c b/src/osmo-bts-trx/trx_vty.c</span><br><span>index 6b8ca77..e5199ec 100644</span><br><span>--- a/src/osmo-bts-trx/trx_vty.c</span><br><span>+++ b/src/osmo-bts-trx/trx_vty.c</span><br><span>@@ -514,7 +514,7 @@</span><br><span>        return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy(struct vty *vty, struct phy_link *plink)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy(struct vty *vty, const struct phy_link *plink)</span><br><span> {</span><br><span>    if (plink->u.osmotrx.local_ip)</span><br><span>            vty_out(vty, " osmotrx ip local %s%s",</span><br><span>@@ -542,7 +542,7 @@</span><br><span>               vty_out(vty, " osmotrx trxd-max-version %d%s", plink->u.osmotrx.trxd_hdr_ver_max, VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy_inst(struct vty *vty, struct phy_instance *pinst)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy_inst(struct vty *vty, const struct phy_instance *pinst)</span><br><span> {</span><br><span>         struct trx_l1h *l1h = pinst->u.osmotrx.hdl;</span><br><span> </span><br><span>@@ -571,11 +571,11 @@</span><br><span>                   VTY_NEWLINE);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_bts(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_bts(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_trx(struct vty *vty, struct gsm_bts_trx *trx)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_trx(struct vty *vty, const struct gsm_bts_trx *trx)</span><br><span> {</span><br><span>        struct phy_instance *pinst = trx_phy_instance(trx);</span><br><span>  struct trx_l1h *l1h = pinst->u.osmotrx.hdl;</span><br><span>diff --git a/src/osmo-bts-virtual/virtualbts_vty.c b/src/osmo-bts-virtual/virtualbts_vty.c</span><br><span>index 875e682..9d63242 100644</span><br><span>--- a/src/osmo-bts-virtual/virtualbts_vty.c</span><br><span>+++ b/src/osmo-bts-virtual/virtualbts_vty.c</span><br><span>@@ -52,19 +52,19 @@</span><br><span> </span><br><span> static struct gsm_bts *vty_bts;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_bts(struct vty *vty, struct gsm_bts *bts)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_bts(struct vty *vty, const struct gsm_bts *bts)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_trx(struct vty *vty, struct gsm_bts_trx *trx)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_trx(struct vty *vty, const struct gsm_bts_trx *trx)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy_inst(struct vty *vty, struct phy_instance *pinst)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy_inst(struct vty *vty, const struct phy_instance *pinst)</span><br><span> {</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-void bts_model_config_write_phy(struct vty *vty, struct phy_link *plink)</span><br><span style="color: hsl(120, 100%, 40%);">+void bts_model_config_write_phy(struct vty *vty, const struct phy_link *plink)</span><br><span> {</span><br><span>        if (plink->u.virt.mcast_dev)</span><br><span>              vty_out(vty, " virtual-um net-device %s%s",</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20288">change 20288</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bts/+/20288"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iacc32f7982c150d84ea4df7affa1f9e07806928f </div>
<div style="display:none"> Gerrit-Change-Number: 20288 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>