pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/27363 )


Change subject: ipa oml: Fix encoding of T3105
......................................................................

ipa oml: Fix encoding of T3105

As the comment above the fix suggest, the encoding is in 10ms units.
osmo-bts is also doing the proper:
"""
uint8_t t3105 = *TLVP_VAL(&tp, NM_ATT_BTS_AIR_TIMER);
bts->t3105_ms = t3105 * 10;
"""

Related: SYS#5838
Change-Id: Ie190514ee35d1ca81b70e9180bf7393b973d3504
---
M src/osmo-bsc/bts_ipaccess_nanobts_omlattr.c
1 file changed, 1 insertion(+), 1 deletion(-)



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

diff --git a/src/osmo-bsc/bts_ipaccess_nanobts_omlattr.c 
b/src/osmo-bsc/bts_ipaccess_nanobts_omlattr.c
index aac0ddf..d5b4215 100644
--- a/src/osmo-bsc/bts_ipaccess_nanobts_omlattr.c
+++ b/src/osmo-bsc/bts_ipaccess_nanobts_omlattr.c
@@ -85,7 +85,7 @@
        msgb_tv_fixed_put(msgb, NM_ATT_LDAVG_SLOTS, 2, buf);

        /* 10 milliseconds */
-       msgb_tv_put(msgb, NM_ATT_BTS_AIR_TIMER, 
osmo_tdef_get(bts->network->T_defs, 3105, OSMO_TDEF_MS, -1));
+       msgb_tv_put(msgb, NM_ATT_BTS_AIR_TIMER, 
osmo_tdef_get(bts->network->T_defs, 3105, OSMO_TDEF_MS, -1)/10);

        /* 10 retransmissions of physical config */
        msgb_tv_put(msgb, NM_ATT_NY1, 10);

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ie190514ee35d1ca81b70e9180bf7393b973d3504
Gerrit-Change-Number: 27363
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-MessageType: newchange

Reply via email to