Attention is currently required from: fixeria.

Hello osmith, Jenkins Builder, laforge, pespin,

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/c/osmo-bts/+/33934

to look at the new patch set (#2).

Change subject: csd_v110_rtp_encode(): properly set E1/E2/E3 bits
......................................................................

csd_v110_rtp_encode(): properly set E1/E2/E3 bits

The E1/E2/E3 bits are set based on out-of-band knowledge of the
current user data rate.  The actual bit values are defined in
3GPP TS 44.021, Figure 4 "Coding of data rates".

TODO: this is only valid for transparent services,
      for non-transparent services see 3GPP TS 48.020.
TODO: lchan->csd_mode is never set to the actual CSD mode...

Change-Id: I1a14597dff746cf975140b294400a2cc05badccd
Related: OS#1572
---
M include/osmo-bts/lchan.h
M src/common/csd_v110.c
M tests/csd/csd_test.c
M tests/csd/csd_test.err
4 files changed, 64 insertions(+), 19 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/34/33934/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/33934
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I1a14597dff746cf975140b294400a2cc05badccd
Gerrit-Change-Number: 33934
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-MessageType: newpatchset

Reply via email to