<p>dexter has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/20249">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp_e1: do not expose function  mgcp_e1_init()<br><br>The initialization of the E1 line in mgcp_e1.c is controlled internally,<br>there is no need to expose the function mgcp_e1_init(), lets make it<br>static and remove the prefix mgcp_<br><br>Change-Id: I6aba1c55c9b1d729709ee1fba2994c77bd848a9b<br>---<br>M include/osmocom/mgcp/mgcp_e1.h<br>M src/libosmo-mgcp/mgcp_e1.c<br>2 files changed, 2 insertions(+), 7 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/49/20249/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/mgcp/mgcp_e1.h b/include/osmocom/mgcp/mgcp_e1.h</span><br><span>index a4ae854..f0cf3ec 100644</span><br><span>--- a/include/osmocom/mgcp/mgcp_e1.h</span><br><span>+++ b/include/osmocom/mgcp/mgcp_e1.h</span><br><span>@@ -21,7 +21,6 @@</span><br><span> static const uint8_t e1_rates[] = { 64, 32, 32, 16, 16, 16, 16, 8, 8, 8, 8, 8, 8, 8, 8 };</span><br><span> static const uint8_t e1_offsets[] = { 0, 0, 4, 0, 2, 4, 6, 0, 1, 2, 3, 4, 5, 6, 7 };</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-int mgcp_e1_init(struct mgcp_trunk *trunk, uint8_t ts_nr);</span><br><span> int mgcp_e1_endp_equip(struct mgcp_endpoint *endp, uint8_t ts, uint8_t ss, uint8_t offs);</span><br><span> void mgcp_e1_endp_update(struct mgcp_endpoint *endp);</span><br><span> void mgcp_e1_endp_release(struct mgcp_endpoint *endp);</span><br><span>diff --git a/src/libosmo-mgcp/mgcp_e1.c b/src/libosmo-mgcp/mgcp_e1.c</span><br><span>index 50a2d85..7d4a29e 100644</span><br><span>--- a/src/libosmo-mgcp/mgcp_e1.c</span><br><span>+++ b/src/libosmo-mgcp/mgcp_e1.c</span><br><span>@@ -361,11 +361,7 @@</span><br><span>       e1_send(ts, trunk);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-/*! Find an endpoint by its name on a specified trunk.</span><br><span style="color: hsl(0, 100%, 40%);">- *  \param[in] trunk trunk configuration.</span><br><span style="color: hsl(0, 100%, 40%);">- *  \param[in] ts_nr E1 timeslot number.</span><br><span style="color: hsl(0, 100%, 40%);">- *  \returns -EINVAL on failure, 0 on success. */</span><br><span style="color: hsl(0, 100%, 40%);">-int mgcp_e1_init(struct mgcp_trunk *trunk, uint8_t ts_nr)</span><br><span style="color: hsl(120, 100%, 40%);">+static int e1_init(struct mgcp_trunk *trunk, uint8_t ts_nr)</span><br><span> {</span><br><span>      /*! Each timeslot needs only to be configured once. The Timeslot then</span><br><span>         *  stays open and permanently receives data. It is then up to the</span><br><span>@@ -517,7 +513,7 @@</span><br><span>     endp->e1.last_amr_ft = AMR_4_75;</span><br><span> </span><br><span>      /* Set up E1 line / timeslot */</span><br><span style="color: hsl(0, 100%, 40%);">- rc = mgcp_e1_init(endp->trunk, ts);</span><br><span style="color: hsl(120, 100%, 40%);">+        rc = e1_init(endp->trunk, ts);</span><br><span>    if (rc != 0)</span><br><span>                 return -EINVAL;</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/20249">change 20249</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/+/20249"/><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: I6aba1c55c9b1d729709ee1fba2994c77bd848a9b </div>
<div style="display:none"> Gerrit-Change-Number: 20249 </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-MessageType: newchange </div>