<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/10701">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/10701/1//COMMIT_MSG">Commit Message:</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/10701/1//COMMIT_MSG@18">Patch Set #1, Line 18:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">For the power,TA levels, take over an old lchan's values only when it is<br>  assigning a new lchan in the same cell.<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">this doesn't make sense, sorry. The process of establishing the new radio channel in asynchronous hand-over with the RACH bursts exists exactly to determine the TA value of the new channel.  Copying TA from the old lchan only makes sense if the old lchan is in the same *BTS*.  A diferent BTS managed by the same BSC will normally be at a completely different geographical location and you cannot make any assumption on the TA.  Hence the async hand-over....</p><p style="white-space: pre-wrap; word-wrap: break-word;">regarding power levels, I would always start with the hghest power level possible/permitted. the power control loop will then ramp down to whatever is sufficient.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10701">change 10701</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/10701"/><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: I360b003398487fa6f934296ff03643c33ec61a35 </div>
<div style="display:none"> Gerrit-Change-Number: 10701 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@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-Comment-Date: Thu, 30 Aug 2018 12:49:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>