<p style="white-space: pre-wrap; word-wrap: break-word;">my +1 is already here, need someone else now.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It's still hard to navigate why exactly this patch is needed though.<br>I assume that you're doing the right thing, but it is not very clearly explained. Good enough for me, but that may be the reason why there is only one vote so far.</p><p><a href="https://gerrit.osmocom.org/12180">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12180/1/src/common/rsl.c">File src/common/rsl.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12180/1/src/common/rsl.c@2207">Patch Set #1, Line 2207:</a> <code style="font-family:monospace,monospace">             /* TCH takes preference over PDCH so allow forcing PDCH DEACT,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">From looking at BTS code, it looks like PDCH in general is expected to be !NONE too, but I didn't go […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">(TBF activity never has any impact on osmo-bts, it just hands off all control to the PCU)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12180">change 12180</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/12180"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I6d6d12aec10c801fe55012ca6e58d0bc8755b15d </div>
<div style="display:none"> Gerrit-Change-Number: 12180 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 17 Dec 2018 14:19:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>