<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/11441">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gsm_data: set meaningful default values for amr modes<br><br>The function gsm_bts_alloc() does set default values for the amr rates<br>for a newly allocated bts, but it does not populate the ms_mode and<br>bts_mode flags which contain hysteresis and threshold. Those values are<br>currently set to 0 by default, which does not make much sense. Lets<br>popluate some appropriate default values.<br><br>- Make sure that .mode .hysteresis and .threshold are populated for<br>  MS and BTS in full and halfrate<br><br>Change-Id: If14843feeeea6584e5991d5c0abb765611dfaa57<br>Related: OS#3529<br>---<br>M src/osmo-bsc/gsm_data.c<br>1 file changed, 30 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bsc/gsm_data.c b/src/osmo-bsc/gsm_data.c</span><br><span>index 8d0b831..08d5bcb 100644</span><br><span>--- a/src/osmo-bsc/gsm_data.c</span><br><span>+++ b/src/osmo-bsc/gsm_data.c</span><br><span>@@ -900,6 +900,21 @@</span><br><span>                 .m12_2 = 1</span><br><span>   };</span><br><span>   memcpy(bts->mr_full.gsm48_ie, &mr_cfg, sizeof(bts->mr_full.gsm48_ie));</span><br><span style="color: hsl(120, 100%, 40%);">+      bts->mr_full.ms_mode[0].mode = 1;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_full.ms_mode[1].mode = 2;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_full.ms_mode[2].mode = 6;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_full.ms_mode[3].mode = 7;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_full.bts_mode[0].mode = 1;</span><br><span style="color: hsl(120, 100%, 40%);">+ bts->mr_full.bts_mode[1].mode = 2;</span><br><span style="color: hsl(120, 100%, 40%);">+ bts->mr_full.bts_mode[2].mode = 6;</span><br><span style="color: hsl(120, 100%, 40%);">+ bts->mr_full.bts_mode[3].mode = 7;</span><br><span style="color: hsl(120, 100%, 40%);">+ for (i = 0; i < 3; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+          bts->mr_full.ms_mode[i].hysteresis = 8;</span><br><span style="color: hsl(120, 100%, 40%);">+            bts->mr_full.ms_mode[i].threshold = 32;</span><br><span style="color: hsl(120, 100%, 40%);">+            bts->mr_full.bts_mode[i].hysteresis = 8;</span><br><span style="color: hsl(120, 100%, 40%);">+           bts->mr_full.bts_mode[i].threshold = 32;</span><br><span style="color: hsl(120, 100%, 40%);">+   }</span><br><span style="color: hsl(120, 100%, 40%);">+     bts->mr_full.num_modes = 4;</span><br><span> </span><br><span>   mr_cfg = (struct gsm48_multi_rate_conf) {</span><br><span>            .m4_75 = 0,</span><br><span>@@ -912,6 +927,21 @@</span><br><span>           .m12_2 = 0</span><br><span>   };</span><br><span>   memcpy(bts->mr_half.gsm48_ie, &mr_cfg, sizeof(bts->mr_half.gsm48_ie));</span><br><span style="color: hsl(120, 100%, 40%);">+      bts->mr_half.ms_mode[0].mode = 1;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_half.ms_mode[1].mode = 2;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_half.ms_mode[2].mode = 4;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_half.ms_mode[3].mode = 5;</span><br><span style="color: hsl(120, 100%, 40%);">+  bts->mr_half.bts_mode[0].mode = 1;</span><br><span style="color: hsl(120, 100%, 40%);">+ bts->mr_half.bts_mode[1].mode = 2;</span><br><span style="color: hsl(120, 100%, 40%);">+ bts->mr_half.bts_mode[2].mode = 4;</span><br><span style="color: hsl(120, 100%, 40%);">+ bts->mr_half.bts_mode[3].mode = 5;</span><br><span style="color: hsl(120, 100%, 40%);">+ for (i = 0; i < 3; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+          bts->mr_half.ms_mode[i].hysteresis = 8;</span><br><span style="color: hsl(120, 100%, 40%);">+            bts->mr_half.ms_mode[i].threshold = 32;</span><br><span style="color: hsl(120, 100%, 40%);">+            bts->mr_half.bts_mode[i].hysteresis = 8;</span><br><span style="color: hsl(120, 100%, 40%);">+           bts->mr_half.bts_mode[i].threshold = 32;</span><br><span style="color: hsl(120, 100%, 40%);">+   }</span><br><span style="color: hsl(120, 100%, 40%);">+     bts->mr_half.num_modes = 4;</span><br><span> </span><br><span>   return bts;</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/11441">change 11441</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/11441"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: If14843feeeea6584e5991d5c0abb765611dfaa57 </div>
<div style="display:none"> Gerrit-Change-Number: 11441 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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>