dexter has uploaded this change for review. ( https://gerrit.osmocom.org/10960


Change subject: gsm0808: fix wrong codec defaults for OFR_AMR_WB
......................................................................

gsm0808: fix wrong codec defaults for OFR_AMR_WB

3GPP TS 48.008 chapter 3.2.2.103 states that the configuration bits of
OFR_AMR_WB are coded as follows:

"S0, S2, S4 indicates the supported Codec Configurations. S1, S3, S5, S6,
S7 are reserved and coded with zeroes."

The current default setting of 0x3F violates this requirement. Lets set
the "forbidden" settings zu zero and keep only the allowed ones.

Change-Id: I4a481def59e9c98cfdcafc2b80c0ac7df0c14130
---
M include/osmocom/gsm/protocol/gsm_08_08.h
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/60/10960/1

diff --git a/include/osmocom/gsm/protocol/gsm_08_08.h 
b/include/osmocom/gsm/protocol/gsm_08_08.h
index 69007b5..9433817 100644
--- a/include/osmocom/gsm/protocol/gsm_08_08.h
+++ b/include/osmocom/gsm/protocol/gsm_08_08.h
@@ -499,7 +499,7 @@
        GSM0808_SC_CFG_DEFAULT_HR_AMR           = 0x073f,
        GSM0808_SC_CFG_DEFAULT_OHR_AMR          = 0x57ff,
        GSM0808_SC_CFG_DEFAULT_FR_AMR_WB        = 0x01,
-       GSM0808_SC_CFG_DEFAULT_OFR_AMR_WB       = 0x3f,
+       GSM0808_SC_CFG_DEFAULT_OFR_AMR_WB       = 0x15,
        GSM0808_SC_CFG_DEFAULT_OHR_AMR_WB       = 0x01,
 };


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4a481def59e9c98cfdcafc2b80c0ac7df0c14130
Gerrit-Change-Number: 10960
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>

Reply via email to