<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/19118">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp_endp: use define constant to define max number of E1 subslots<br><br>There are 15 possible subslots (not all at the same time) in one E1<br>timeslot. Lets use a define constant for that.<br><br>Change-Id: If7cb74e486946aff09e22abf8a8885bf0693f34e<br>Related: OS#2547<br>---<br>M include/osmocom/mgcp/mgcp_endp.h<br>M src/libosmo-mgcp/mgcp_endp.c<br>2 files changed, 5 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/mgcp/mgcp_endp.h b/include/osmocom/mgcp/mgcp_endp.h</span><br><span>index a58053a..879947b 100644</span><br><span>--- a/include/osmocom/mgcp/mgcp_endp.h</span><br><span>+++ b/include/osmocom/mgcp/mgcp_endp.h</span><br><span>@@ -27,6 +27,9 @@</span><br><span> struct mgcp_conn;</span><br><span> struct mgcp_endpoint;</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/* Number of E1 subslots (different variants, not all useable at the same time) */</span><br><span style="color: hsl(120, 100%, 40%);">+#define MGCP_ENDP_E1_SUBSLOTS 15</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> #define LOGPENDP(endp, cat, level, fmt, args...) \</span><br><span> LOGP(cat, level, "endpoint:%s " fmt, \</span><br><span>      endp ? endp->name : "none", \</span><br><span>diff --git a/src/libosmo-mgcp/mgcp_endp.c b/src/libosmo-mgcp/mgcp_endp.c</span><br><span>index 82f7732..3ac3c5d 100644</span><br><span>--- a/src/libosmo-mgcp/mgcp_endp.c</span><br><span>+++ b/src/libosmo-mgcp/mgcp_endp.c</span><br><span>@@ -119,7 +119,7 @@</span><br><span>           endp->type = &ep_typeset.rtp;</span><br><span>                 endp->name = gen_e1_epname(endp, trunk->cfg->domain,</span><br><span>                                           trunk->trunk_nr,</span><br><span style="color: hsl(0, 100%, 40%);">-                                     index / 15, index % 15);</span><br><span style="color: hsl(120, 100%, 40%);">+                                      index / MGCP_ENDP_E1_SUBSLOTS, index % MGCP_ENDP_E1_SUBSLOTS);</span><br><span>            break;</span><br><span>       default:</span><br><span>             osmo_panic("Cannot allocate unimplemented trunk type %d! %s:%d\n",</span><br><span>@@ -524,7 +524,7 @@</span><br><span>    * endpoint we need to verify, only the overlaps need to be checked. This is</span><br><span>          * also the reason why the related subslot number is missing from each each</span><br><span>   * line. */</span><br><span style="color: hsl(0, 100%, 40%);">-     const int8_t interlock_tab[15][16] =</span><br><span style="color: hsl(120, 100%, 40%);">+  const int8_t interlock_tab[MGCP_ENDP_E1_SUBSLOTS][15] =</span><br><span>              { { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, -1 },</span><br><span>             { 0, 3, 4, 7, 8, 9, 10, -1, -1, -1, -1, -1, -1, -1, -1 },</span><br><span>            { 0, 5, 6, 11, 12, 13, 14, -1, -1, -1, -1, -1, -1, -1, -1 },</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/19118">change 19118</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/c/osmo-mgw/+/19118"/><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-Change-Id: If7cb74e486946aff09e22abf8a8885bf0693f34e </div>
<div style="display:none"> Gerrit-Change-Number: 19118 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>