<p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/18721">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/osmo-trx/+/18721/2/Transceiver52M/device/uhd/UHDDevice.cpp">File Transceiver52M/device/uhd/UHDDevice.cpp:</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/osmo-trx/+/18721/2/Transceiver52M/device/uhd/UHDDevice.cpp@1029">Patch Set #2, Line 1029:</a> <code style="font-family:monospace,monospace">if (band != 0 && req_band != band) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Pau,</p><p style="white-space: pre-wrap; word-wrap: break-word;">this patch, in particular this (req_band != band) check, makes it impossible to change an ARFCN from one band to an ARFCN from another band in osmo-bsc/osmo-bts-trx without restarting osmo-trx-uhd. I faced this a few minutes ago, and it took me a while to investigate/realize this.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Why do we need this check? I could not find any explanation in the commit message, nor any comments in the code why the new freq. band shall be the same as was used before? To me it looks like we don't need this check at all. If I am wrong, please add some comments and update documentation, so another user facing this error message would not need to read the source code and bisect git history.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/18721">change 18721</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/c/osmo-trx/+/18721"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: If7ef5bf95ffe4afe5864c0f051853aa38b9639eb </div>
<div style="display:none"> Gerrit-Change-Number: 18721 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: roh <jsteiger@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: tnt <tnt@246tNt.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 27 Jun 2020 11:58:14 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>