laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/32105 )
(
2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted
one.
)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, 18 insertions(+), 3 deletions(-)
Approvals:
fixeria: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/osmo-bts-lc15/oml.c b/src/osmo-bts-lc15/oml.c
index 4847c17..3709d8c 100644
--- a/src/osmo-bts-lc15/oml.c
+++ b/src/osmo-bts-lc15/oml.c
@@ -1844,7 +1844,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) {
@@ -1858,10 +1857,11 @@
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) {
@@ -1870,6 +1870,7 @@
/* update current max cell size */
fl1h->phy_inst->u.lc15.max_cell_size = cell_size;
}
+ }
#endif
/* Did we go through MphInit yet? If yes fire and forget */
--
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: 3
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
