<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/12922">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  Vadim Yanitskiy: Looks good to me, but someone else must approve
  Max: Looks good to me, but someone else must approve
  Harald Welte: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">scheduler_trx: use stored fn for pdtch data indications<br><br>When the ph-data indications for the PDTCH are passed up to l1sap,<br>then a forumla is used to calculate the frame number of the beginning of<br>the block that is just passed up. This is not necessary since the start<br>frame number of the block is stored in *first_fn when the block arrives.<br>We should use this frame number instead. (For the measurement indication<br>it is already done this way).<br><br>Change-Id: I6c01987be78203acfa689c6decb2c806f8fff3d6<br>Related: OS#2977<br>---<br>M src/osmo-bts-trx/scheduler_trx.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<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 b395479..32bdb98 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>@@ -974,7 +974,7 @@</span><br><span>            return 0;</span><br><span>    }</span><br><span>    ber10k = compute_ber10k(n_bits_total, n_errors);</span><br><span style="color: hsl(0, 100%, 40%);">-        return _sched_compose_ph_data_ind(l1t, tn, (fn + GSM_HYPERFRAME - 3) % GSM_HYPERFRAME, chan,</span><br><span style="color: hsl(120, 100%, 40%);">+  return _sched_compose_ph_data_ind(l1t, tn, *first_fn, chan,</span><br><span>          l2, rc, *rssi_sum / *rssi_num, *toa256_sum / *toa_num, 0,</span><br><span>                                      ber10k, PRES_INFO_BOTH);</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/12922">change 12922</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/12922"/><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: I6c01987be78203acfa689c6decb2c806f8fff3d6 </div>
<div style="display:none"> Gerrit-Change-Number: 12922 </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>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>