<p>Stefan Sperling has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/10280">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">fix conditions for sending fill frames during RTS IND<br><br>Rewrite an if-statement to better match the description given<br>in GSM 05.08, and quote the relevant paragraph in a comment.<br>Since this entire block of code only runs for SDCCH and TCH,<br>this new condition should provide the same result expect that<br>in accordance with the standard we now stop sending fill-frames<br>on a signalling TCH if DTX is in use.<br><br>Also note that this code should already cover parts of the<br>patch proposed at https://gerrit.osmocom.org/c/osmo-bts/+/5753<br>The changes to osmo-bts-litecell15/l1_if.c proposed there<br>should be equivalent to the fill-frame logic in this<br>existing common BTS code which is handling RTS IND.<br><br>Change-Id: Ibaf3ecbd0bde4f37e799d5e2f5d00dc695e0139a<br>Related: OS#1950<br>---<br>M src/common/l1sap.c<br>1 file changed, 10 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/80/10280/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/common/l1sap.c b/src/common/l1sap.c</span><br><span>index 8936674..8d735e5 100644</span><br><span>--- a/src/common/l1sap.c</span><br><span>+++ b/src/common/l1sap.c</span><br><span>@@ -758,9 +758,16 @@</span><br><span>                                       memcpy(p + 2, si, GSM_MACBLOCK_LEN - 2);</span><br><span>                             } else</span><br><span>                                       memcpy(p + 2, fill_frame, GSM_MACBLOCK_LEN - 2);</span><br><span style="color: hsl(0, 100%, 40%);">-                        } else if ((!L1SAP_IS_CHAN_TCHF(chan_nr) && !L1SAP_IS_CHAN_TCHH(chan_nr))</span><br><span style="color: hsl(0, 100%, 40%);">-                               || lchan->rsl_cmode == RSL_CMOD_SPD_SIGN) {</span><br><span style="color: hsl(0, 100%, 40%);">-                          /* send fill frame only, if not TCH/x != Signalling, otherwise send empty frame */</span><br><span style="color: hsl(120, 100%, 40%);">+                    } else if (L1SAP_IS_CHAN_SDCCH4(chan_nr) || L1SAP_IS_CHAN_SDCCH8(chan_nr) ||</span><br><span style="color: hsl(120, 100%, 40%);">+                             (lchan->rsl_cmode == RSL_CMOD_SPD_SIGN && !lchan->ts->trx->bts->dtxd)) {</span><br><span style="color: hsl(120, 100%, 40%);">+                            /*</span><br><span style="color: hsl(120, 100%, 40%);">+                             * SDCCH or TCH in signalling mode without DTX.</span><br><span style="color: hsl(120, 100%, 40%);">+                                *</span><br><span style="color: hsl(120, 100%, 40%);">+                             * Send fill frame according to GSM 05.08, section 8.3: "On the SDCCH and on the</span><br><span style="color: hsl(120, 100%, 40%);">+                          * half rate speech traffic channel in signalling only mode DTX is not allowed.</span><br><span style="color: hsl(120, 100%, 40%);">+                                * In these cases and during signalling on the TCH when DTX is not used, the same</span><br><span style="color: hsl(120, 100%, 40%);">+                              * L2 fill frame shall be transmitted in case there is nothing else to transmit."</span><br><span style="color: hsl(120, 100%, 40%);">+                                 */</span><br><span>                          p = msgb_put(msg, GSM_MACBLOCK_LEN);</span><br><span>                                 memcpy(p, fill_frame, GSM_MACBLOCK_LEN);</span><br><span>                     } /* else the message remains empty, so TCH frames are sent */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10280">change 10280</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/10280"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ibaf3ecbd0bde4f37e799d5e2f5d00dc695e0139a </div>
<div style="display:none"> Gerrit-Change-Number: 10280 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>