fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33830 ) Change subject: lchan_select: fix lchan selection for GSM48_CMODE_DATA_{14k5,12k0} ...................................................................... lchan_select: fix lchan selection for GSM48_CMODE_DATA_{14k5,12k0} Change-Id: I271ebf7e64e3d0861586e9bc63bec113cb5fb410 Fixes: d5eb0f1b5 "lchan_select: chan_mode_to_chan_type: support CSD" Related: OS#4393 --- M src/osmo-bsc/lchan_select.c 1 file changed, 14 insertions(+), 3 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/osmo-bsc/lchan_select.c b/src/osmo-bsc/lchan_select.c index 1e12d96..ad5c984 100644 --- a/src/osmo-bsc/lchan_select.c +++ b/src/osmo-bsc/lchan_select.c @@ -178,14 +178,14 @@ return GSM_LCHAN_NONE; } case GSM48_CMODE_SPEECH_EFR: - /* EFR works over FR channels only */ + case GSM48_CMODE_DATA_14k5: + case GSM48_CMODE_DATA_12k0: + /* these rates work over full-rate channels only */ if (chan_rate != CH_RATE_FULL) return GSM_LCHAN_NONE; /* fall through */ case GSM48_CMODE_SPEECH_V1: case GSM48_CMODE_SPEECH_AMR: - case GSM48_CMODE_DATA_14k5: - case GSM48_CMODE_DATA_12k0: case GSM48_CMODE_DATA_6k0: case GSM48_CMODE_DATA_3k6: switch (chan_rate) { -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33830 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I271ebf7e64e3d0861586e9bc63bec113cb5fb410 Gerrit-Change-Number: 33830 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: osmith <osmith@sysmocom.de> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged