Attention is currently required from: laforge. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/27778 )

Change subject: osmo-bts-trx: amr_loop: do not miss C/I samples ......................................................................

Patch Set 2:

(1 comment)

File src/osmo-bts-trx/amr_loop.c:

https://gerrit.osmocom.org/c/osmo-bts/+/27778/comment/fbb624fa_a65b29b1 PS1, Line 53: if (mi != chan_state->dl_cmr)

I would expect the C/I ratio to stay more or less the same, no matter which codec mode is used. […]

This is a perfectly valid point. I'll add this to the commit message. Indeed, C/I is calculated by checking the training sequence bits, i.e. the burst midamble, which remains the same regardless of the current codec rate.