Attention is currently required from: fixeria, pespin.

fixeria uploaded patch set #3 to this change.

View Change

The following approvals got outdated and were removed: Code-Review+1 by pespin, Verified+1 by Jenkins Builder

mobile: set TRAFFIC.{ind,req} mode during call establishment

Now that we support data (CSD) calls in addition to voice calls,
we can no longer initialize the TRAFFIC.{ind,req} routing mode
in gsm48_rr_init(). We need to apply the appropriate TCH routing
mode *during call establishment* based on its type and the
configured I/O handler type.

After this patch, one can have the following configuration:

tch-voice
io-handler l1phy
tch-data
io-handler unix-sock
io-tch-format ti

so that the io-handler setting for voice would not affect data calls.
Before this patch, the L1 PHY (specifically, Calypso firmware) would
not route TRAFFIC.{ind,req} during data calls at all.

Thanks to this patch, it's also no longer required to restart the
mobile application after changing voice or data I/O handler.

Change-Id: Iab68cb47c28380a9c1efc149c6196ea54f75fdb8
Related: OS#4396
---
M src/host/layer23/src/mobile/gsm48_rr.c
M src/host/layer23/src/mobile/tch.c
M src/host/layer23/src/mobile/tch_data.c
M src/host/layer23/src/mobile/tch_voice.c
M src/host/layer23/src/mobile/vty_interface.c
5 files changed, 82 insertions(+), 65 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/40/35740/3

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Iab68cb47c28380a9c1efc149c6196ea54f75fdb8
Gerrit-Change-Number: 35740
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-CC: jolly <andreas@eversberg.eu>
Gerrit-Attention: pespin <pespin@sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de>
Gerrit-MessageType: newpatchset