<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">what I don't like here is that this implements "blind" change<br>from<br>one type to another type, on a global scope for the entire MGW.<br>I'd much rather see this more fine-grained: Only on the<br>AoIP-facing<br>connections, and only if the BSC actually asks us to. See my<br>other<br>comment.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I see the problem. I will check the specs what we can do here.<br>Maybe we should even be more restrictive and configure at the BSC<br>vty which exact packet format is spoken on the AoIP side. Maybe we<br>can apply this to the HRv1 problem then as well.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">There is indeed an SDP specified, which can be used to submit format specific parameters. The SDP string would look like this:</p><p style="white-space: pre-wrap; word-wrap: break-word;">a=fmtp:97 octet-align=1</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this is how it should be done. osmo-msc should get a VTY option and depending on this option the MGW gets instructed to apply the conversion or not.</p><p><a href="https://gerrit.osmocom.org/13082">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13082">change 13082</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/13082"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Id46a3622a6b1995d5fce4bc611d409bcb4aef13b </div>
<div style="display:none"> Gerrit-Change-Number: 13082 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 05 Mar 2019 10:29:57 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>