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

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">scheduler_trx: Add reminders to use libosmocore functions<br><br>Let' wait until the just-merged osmo_timerfd_* functions<br>of Change-Id Ibeffba7c997252c003723bcd5d14122c4ded2fe7 have made it<br>into the next tagged release, and then replace the implementation<br>here.<br><br>Change-Id: Ic0f0a7437b6acb535177e5ad6ac7a6d336654c66<br>---<br>M src/osmo-bts-trx/scheduler_trx.c<br>1 file changed, 3 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bts-trx/scheduler_trx.c b/src/osmo-bts-trx/scheduler_trx.c</span><br><span>index 4c4ae6c..11c5c4e 100644</span><br><span>--- a/src/osmo-bts-trx/scheduler_trx.c</span><br><span>+++ b/src/osmo-bts-trx/scheduler_trx.c</span><br><span>@@ -1427,6 +1427,7 @@</span><br><span> }</span><br><span> </span><br><span> /*! disable the osmocom-wrapped timerfd */</span><br><span style="color: hsl(120, 100%, 40%);">+/* FIXME: Use libosmocore after release with Ibeffba7c997252c003723bcd5d14122c4ded2fe7 was made */</span><br><span> static int timer_ofd_disable(struct osmo_fd *ofd)</span><br><span> {</span><br><span>        const struct itimerspec its_null = {</span><br><span>@@ -1441,6 +1442,7 @@</span><br><span>  *  \param[in] first Relative time at which the timer should first execute (NULL = \a interval)</span><br><span>  *  \param[in] interval Time interval at which subsequent timer shall fire</span><br><span>  *  \returns 0 on success; negative on error */</span><br><span style="color: hsl(120, 100%, 40%);">+/* FIXME: Use libosmocore after release with Ibeffba7c997252c003723bcd5d14122c4ded2fe7 was made */</span><br><span> static int timer_ofd_schedule(struct osmo_fd *ofd, const struct timespec *first,</span><br><span>                           const struct timespec *interval)</span><br><span> {</span><br><span>@@ -1469,6 +1471,7 @@</span><br><span>  *  We simply initialize the data structures here, but do not yet</span><br><span>  *  schedule the timer.</span><br><span>  */</span><br><span style="color: hsl(120, 100%, 40%);">+/* FIXME: Use libosmocore after release with Ibeffba7c997252c003723bcd5d14122c4ded2fe7 was made */</span><br><span> static int timer_ofd_setup(struct osmo_fd *ofd, int (*cb)(struct osmo_fd *, unsigned int), void *data)</span><br><span> {</span><br><span>        ofd->cb = cb;</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/8103">change 8103</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/8103"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ic0f0a7437b6acb535177e5ad6ac7a6d336654c66 </div>
<div style="display:none"> Gerrit-Change-Number: 8103 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>