fixeria has submitted this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/34217 )
Change subject: csd_v110: fix comments in csd_v110_rtp_{en,de}code()
......................................................................
csd_v110: fix comments in csd_v110_rtp_{en,de}code()
Change-Id: I6fee665e587bfa76058187e13f0cdafaea991940
Related: OS#1572
---
M src/common/csd_v110.c
1 file changed, 14 insertions(+), 4 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/src/common/csd_v110.c b/src/common/csd_v110.c
index 9fc7ad3..8fa4229 100644
--- a/src/common/csd_v110.c
+++ b/src/common/csd_v110.c
@@ -91,11 +91,11 @@
if (OSMO_UNLIKELY(data_len < (desc->num_blocks * desc->num_bits)))
return -ENODATA;
- /* RA1'/RA1: convert to an intermediate data rate */
+ /* RA1'/RA1: convert from radio rate to an intermediate data rate */
for (unsigned int i = 0; i < desc->num_blocks; i++) {
struct osmo_v110_decoded_frame df;
- /* convert modified V.110 frames to normal V.110 frames */
+ /* convert a V.110 36-/60-bit frame to a V.110 80-bit frame */
if (desc->num_bits == 60)
osmo_csd_12k_6k_decode_frame(&df, &data[i * 60], 60);
else /* desc->num_bits == 36 */
@@ -158,11 +158,11 @@
ra_bits[i] = (rtp[i] >> 7);
}
- /* RA1'/RA1: convert to an intermediate data rate */
+ /* RA1'/RA1: convert from an intermediate rate to radio rate */
for (unsigned int i = 0; i < desc->num_blocks; i++) {
struct osmo_v110_decoded_frame df;
- /* convert modified V.110 frames to normal V.110 frames */
+ /* convert a V.110 80-bit frame to a V.110 36-/60-bit frame */
osmo_v110_decode_frame(&df, &ra_bits[i * 80], 80);
if (desc->num_bits == 60)
osmo_csd_12k_6k_encode_frame(&data[i * 60], 60, &df);
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bts/+/34217
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I6fee665e587bfa76058187e13f0cdafaea991940
Gerrit-Change-Number: 34217
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged