<p style="white-space: pre-wrap; word-wrap: break-word;">I still think the one part fixes how it works, and the other just clarifies comments, without changing the meaning of the comments. To me clearly separate changes...</p><p><a href="https://gerrit.osmocom.org/11954">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11954/6/src/osmo-bsc/timeslot_fsm.c">File src/osmo-bsc/timeslot_fsm.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/11954/6/src/osmo-bsc/timeslot_fsm.c@890">Patch Set #6, Line 890:</a> <code style="font-family:monospace,monospace">         /* If we were switching to PDCH, there's no PDCH lchan managed</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">That's exactly what I'm saying here, for people reading this code which may have not gone through al […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">the way I read it, it implies that "there is a PDCH lchan, but it is not managed here".</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also when doing PDCH_ACT, we simply and plainly are switching to PDCH, there isn't even an lchan involved. We're switching *because* all lchans are gone. And always to PDCH, as the code aptly shows. Nothing to do with "manual" or "PDCH lchans" :P</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11954/6/src/osmo-bsc/timeslot_fsm.c@891">Patch Set #6, Line 891:</a> <code style="font-family:monospace,monospace">              * in BSC, So we need to set it manually. */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">yes, here, in BSC. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">can we rephrase it so it still  helps future readers while it doesn't make me twitch? </p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  When switching to PDCH, there are no lchans and we are telling the PCU to take over the timeslot.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11954">change 11954</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/11954"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib03e5a91438a5b74a04e69f81fab565842b02b66 </div>
<div style="display:none"> Gerrit-Change-Number: 11954 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </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-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-CC: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 04 Dec 2018 17:19:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>