laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/32105 )


Change subject: lc15: fix compiler warning about unused variable cell_size
......................................................................

lc15: fix compiler warning about unused variable cell_size

oml.c: In function ‘bts_model_apply_oml’:
oml.c:1814:17: error: variable ‘cell_size’ set but not used 
[-Werror=unused-but-set-variable]
 1814 |         uint8_t cell_size;
      |                 ^~~~~~~~~

Change-Id: I0bf1542f613f613d03609d50836137ff440401af
---
M src/osmo-bts-lc15/oml.c
1 file changed, 16 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/05/32105/1

diff --git a/src/osmo-bts-lc15/oml.c b/src/osmo-bts-lc15/oml.c
index fa8aa9b..221e075 100644
--- a/src/osmo-bts-lc15/oml.c
+++ b/src/osmo-bts-lc15/oml.c
@@ -1811,7 +1811,6 @@
        int rc;
        struct gsm_bts_trx *trx;
        struct lc15l1_hdl *fl1h;
-       uint8_t cell_size;

        /* TODO: NM Object without FSM: */
        switch (foh->obj_class) {
@@ -1825,10 +1824,10 @@
        case NM_MT_SET_RADIO_ATTR:
                trx = obj;
                fl1h = trx_lc15l1_hdl(trx);
-               /* convert max TA to max cell size in qbits */
-               cell_size = bts->max_ta << 2;

 #if LITECELL15_API_VERSION >= LITECELL15_API(2,1,7)
+               /* convert max TA to max cell size in qbits */
+               uint8_t cell_size = bts->max_ta << 2;
                /* We do not need to check for L1 handle
                 * because the max cell size parameter can receive before 
MphInit */
                if (fl1h->phy_inst->u.lc15.max_cell_size != cell_size) {

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I0bf1542f613f613d03609d50836137ff440401af
Gerrit-Change-Number: 32105
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-MessageType: newchange

Reply via email to