fixeria submitted this change.

View Change


Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve
mobile: fix AUDIO_IOH_LOOPBACK mode: pull DL info header

The loopback mode is currently broken because the DL info header
remains present, thus becoming a bogus "part" of the speech frame.

Change-Id: I1af187b4bc5f5a99bc7f7634d90bf14ad3db0e49
Related: OS#4396
---
M src/host/layer23/src/mobile/voice.c
1 file changed, 15 insertions(+), 0 deletions(-)

diff --git a/src/host/layer23/src/mobile/voice.c b/src/host/layer23/src/mobile/voice.c
index 7d6c2e0..acc0dce 100644
--- a/src/host/layer23/src/mobile/voice.c
+++ b/src/host/layer23/src/mobile/voice.c
@@ -78,6 +78,8 @@
{
switch (ms->settings.audio.io_handler) {
case AUDIO_IOH_LOOPBACK:
+ /* Remove the DL info header */
+ msgb_pull_to_l2(msg);
/* Send voice frame back */
return gsm_send_voice_msg(ms, msg);
case AUDIO_IOH_MNCC_SOCK:

To view, visit change 34757. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I1af187b4bc5f5a99bc7f7634d90bf14ad3db0e49
Gerrit-Change-Number: 34757
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged