This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Max gerrit-no-reply at lists.osmocom.orgReview at https://gerrit.osmocom.org/1374 DTX: add explicit check if DTX enabled Check explicitly if DTX is enabled for AMR before checking if SID frame repetition is optional. Change-Id: I660688d56505798cade1495c30338fd6806a3259 Related: OS#1801 --- M src/common/msg_utils.c 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/74/1374/1 diff --git a/src/common/msg_utils.c b/src/common/msg_utils.c index c5081b4..b844eec 100644 --- a/src/common/msg_utils.c +++ b/src/common/msg_utils.c @@ -257,6 +257,9 @@ */ static inline bool dtx_amr_sid_optional(struct gsm_lchan *lchan, uint32_t fn) { + if (!dtx_dl_amr_enabled(lchan)) + return true; + /* Compute approx. time delta x26 based on Fn duration */ uint32_t dx26 = 120 * (fn - lchan->tch.dtx.fn); -- To view, visit https://gerrit.osmocom.org/1374 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I660688d56505798cade1495c30338fd6806a3259 Gerrit-PatchSet: 1 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Max <msuraev at sysmocom.de>