fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34757?usp=email ) Change subject: mobile: fix AUDIO_IOH_LOOPBACK mode: pull DL info header ...................................................................... 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(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve 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 https://gerrit.osmocom.org/c/osmocom-bb/+/34757?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/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