Neels Hofmeyr has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/8100 )

Change subject: fix default fallbacks in audio_support_to_gsm88()
......................................................................

fix default fallbacks in audio_support_to_gsm88()

For audio->hr == true, use HR1, and for hr == false use FR1; not vice versa.

Change-Id: Ifb4dba7c8e9c1d0a22a007355fbd2eda57e789d3
---
M src/osmo-bsc/osmo_bsc_bssap.c
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/osmo-bsc/osmo_bsc_bssap.c b/src/osmo-bsc/osmo_bsc_bssap.c
index 3f7e4f8..dcc6b9f 100644
--- a/src/osmo-bsc/osmo_bsc_bssap.c
+++ b/src/osmo-bsc/osmo_bsc_bssap.c
@@ -61,7 +61,7 @@
                default:
                        LOGP(DMSC, LOGL_ERROR, "Wrong speech mode: %d\n",
                             audio->ver);
-                       return GSM0808_PERM_FR1;
+                       return GSM0808_PERM_HR1;
                }
        } else {
                switch (audio->ver) {
@@ -77,7 +77,7 @@
                default:
                        LOGP(DMSC, LOGL_ERROR, "Wrong speech mode: %d\n",
                             audio->ver);
-                       return GSM0808_PERM_HR1;
+                       return GSM0808_PERM_FR1;
                }
        }
 }

--
To view, visit https://gerrit.osmocom.org/8100
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifb4dba7c8e9c1d0a22a007355fbd2eda57e789d3
Gerrit-Change-Number: 8100
Gerrit-PatchSet: 2
Gerrit-Owner: Neels Hofmeyr <nhofm...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofm...@sysmocom.de>

Reply via email to