Change in osmo-bts[master]: sched_lchan_tch_x: use functions to determine AMR tranmssion phase
gerrit-no-reply at lists.osmocom.org
Tue Aug 31 19:35:02 UTC 2021
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/25296 )
Change subject: sched_lchan_tch_x: use functions to determine AMR tranmssion phase
Patch Set 1:
PS1, Line 470: LOGL1SB(DL1P, LOGL_NOTICE, l1ts, br, "Codec (FT = %d) "
> Shouldn't we keep sending with the older type until we can change it? I'd expect the received to acc […]
I don't understand this code in general, even the current one. We as an RTP capable BTS cannot influence at all what codec mode is transmitted in downlink. We may receive a CMR from the MS, and we pass that on in the CMR header of the uplink RTP frames. It is up to the sender/encoder of that RTP to then start generating the proper requested codec mode and start sending that.
The BTS is not generating/encoding the AMR, it merely relays AMR frames from Abis/RTP to Um and vice-versa. On the Um interface we only have either CMR or CMI [depending on FN] and we can only signal a subset of up to four codec modes from the set. On the RTP side every frame has CMI (== FT) and CMR, and we can express any of the AMR modes.
Can anyone explain to me what is happening here in this code and why?
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/25296
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Tue, 31 Aug 2021 19:35:02 +0000
Comment-In-Reply-To: pespin <pespin at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log