fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/34321?usp=email )


Change subject: bts_siemens_bs11: remove ip.access nanoBTS specific code
......................................................................

bts_siemens_bs11: remove ip.access nanoBTS specific code

This code is unreachable, because this file is all about Siemens BS11.

Change-Id: I2db92fe448b1f4cd25c1e5c6e1bb9593795c9cb2
---
M src/osmo-bsc/bts_siemens_bs11.c
1 file changed, 34 insertions(+), 45 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/21/34321/1

diff --git a/src/osmo-bsc/bts_siemens_bs11.c b/src/osmo-bsc/bts_siemens_bs11.c
index d1fbeaa..7aa437c 100644
--- a/src/osmo-bsc/bts_siemens_bs11.c
+++ b/src/osmo-bsc/bts_siemens_bs11.c
@@ -423,57 +423,35 @@

        patch_nm_tables(trx->bts);

-       switch (trx->bts->type) {
-       case GSM_BTS_TYPE_BS11:
-               /* FIXME: discover this by fetching an attribute */
+       /* FIXME: discover this by fetching an attribute */
 #if 0
-               trx->nominal_power = 15; /* 15dBm == 30mW PA configuration */
+       trx->nominal_power = 15; /* 15dBm == 30mW PA configuration */
 #else
-               trx->nominal_power = 24; /* 24dBm == 250mW PA configuration */
+       trx->nominal_power = 24; /* 24dBm == 250mW PA configuration */
 #endif
-               abis_nm_conn_terr_sign(trx, e1l->e1_nr, e1l->e1_ts,
-                                       e1l->e1_ts_ss);
-               abis_nm_establish_tei(trx->bts, trx->nr, e1l->e1_nr,
-                                     e1l->e1_ts, e1l->e1_ts_ss, 
trx->rsl_tei_primary);
+       abis_nm_conn_terr_sign(trx, e1l->e1_nr, e1l->e1_ts,
+                               e1l->e1_ts_ss);
+       abis_nm_establish_tei(trx->bts, trx->nr, e1l->e1_nr,
+                             e1l->e1_ts, e1l->e1_ts_ss, trx->rsl_tei_primary);

-               /* Set Radio Attributes */
-               if (trx == trx->bts->c0)
-                       abis_nm_set_radio_attr(trx, bs11_attr_radio,
-                                              sizeof(bs11_attr_radio));
-               else {
-                       uint8_t trx1_attr_radio[sizeof(bs11_attr_radio)];
-                       uint8_t arfcn_low = trx->arfcn & 0xff;
-                       uint8_t arfcn_high = (trx->arfcn >> 8) & 0x0f;
-                       memcpy(trx1_attr_radio, bs11_attr_radio,
-                               sizeof(trx1_attr_radio));
+       /* Set Radio Attributes */
+       if (trx == trx->bts->c0)
+               abis_nm_set_radio_attr(trx, bs11_attr_radio,
+                                      sizeof(bs11_attr_radio));
+       else {
+               uint8_t trx1_attr_radio[sizeof(bs11_attr_radio)];
+               uint8_t arfcn_low = trx->arfcn & 0xff;
+               uint8_t arfcn_high = (trx->arfcn >> 8) & 0x0f;
+               memcpy(trx1_attr_radio, bs11_attr_radio,
+                       sizeof(trx1_attr_radio));

-                       /* patch ARFCN into TRX Attributes */
-                       trx1_attr_radio[2] &= 0xf0;
-                       trx1_attr_radio[2] |= arfcn_high;
-                       trx1_attr_radio[3] = arfcn_low;
+               /* patch ARFCN into TRX Attributes */
+               trx1_attr_radio[2] &= 0xf0;
+               trx1_attr_radio[2] |= arfcn_high;
+               trx1_attr_radio[3] = arfcn_low;

-                       abis_nm_set_radio_attr(trx, trx1_attr_radio,
-                                              sizeof(trx1_attr_radio));
-               }
-               break;
-       case GSM_BTS_TYPE_NANOBTS:
-               switch (trx->bts->band) {
-               case GSM_BAND_850:
-               case GSM_BAND_900:
-                       trx->nominal_power = 20;
-                       break;
-               case GSM_BAND_1800:
-               case GSM_BAND_1900:
-                       trx->nominal_power = 23;
-                       break;
-               default:
-                       LOGP(DNM, LOGL_ERROR, "Unsupported nanoBTS GSM band 
%s\n",
-                               gsm_band_name(trx->bts->band));
-                       break;
-               }
-               break;
-       default:
-               break;
+               abis_nm_set_radio_attr(trx, trx1_attr_radio,
+                                      sizeof(trx1_attr_radio));
        }

        for (i = 0; i < TRX_NR_TS; i++)

--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/34321?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I2db92fe448b1f4cd25c1e5c6e1bb9593795c9cb2
Gerrit-Change-Number: 34321
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-MessageType: newchange

Reply via email to