fixeria has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/29878 )
Change subject: gsm0502: gsm0502_fn_remap(): use GSM_TDMA_FN_SUB() macro ......................................................................
gsm0502: gsm0502_fn_remap(): use GSM_TDMA_FN_SUB() macro
Change-Id: Id77644d945ad195733d4bf68c0d1c5b450b16e58 --- M src/gsm/gsm0502.c 1 file changed, 1 insertion(+), 4 deletions(-)
Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
diff --git a/src/gsm/gsm0502.c b/src/gsm/gsm0502.c index e34d3f5..94db94e 100644 --- a/src/gsm/gsm0502.c +++ b/src/gsm/gsm0502.c @@ -179,7 +179,6 @@ uint8_t fn_cycle; uint8_t i; int sub = -1; - uint32_t fn_map; struct fn_remap_table *table;
OSMO_ASSERT(channel < ARRAY_SIZE(fn_remap_table_ptr)); @@ -199,9 +198,7 @@ return fn; }
- fn_map = (fn + GSM_MAX_FN - sub) % GSM_MAX_FN; - - return fn_map; + return GSM_TDMA_FN_SUB(fn, sub); }
/* Magic numbers (RNTABLE) for pseudo-random hopping sequence generation. */