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 &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged