2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.


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(-)

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 */

