<p>pespin <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/16668">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">radioInterfaceMulti: Fail to tune on freq not following multi-arfcn restrictions<br><br>multi-arfcn feature uses a hardcoded disposition of logical channels on<br>a physical channel. Logical channels in the phisical channel are<br>separated by MCBTS_SPACING Hz, that is 4 GSM ARFCNs.<br><br>As a result, multi-arfcn restricts the TRX ARFCN setup to the following:<br>ARFCN(TRX0)=N, ARFCN(TRX1)=N+1*4, ARFCN(TRX2)=N+2*4, ...<br><br>Let's make sure radioInterfaceMulti verifies the requested Rx/Tx<br>frequencies for each logical channel over TRXC match the restriction<br>explained above. It will check freq going to be set is indeed separated<br>by MCBTS_SPACING from already set channels, making sure the ARFCN series<br>is consistent.<br><br>Otherwise, before this patch, one could set in osmo-bsc:<br>ARFCN(TRX0)=N, ARFCN(TRX1)=N+2<br><br>and osmo-trx would silently ack the related Rx/TxTUNE TRXC commands, but<br>actually still transmit on ARFCN N+4 instead. As a result, in this<br>scenario TRX!=0 were unusable with multi-arfcn.<br><br>Related: OS#4207<br>Change-Id: I2f3d66a611d3a489b3e4d9431994f4ec77b4460f<br>---<br>M Transceiver52M/radioInterface.h<br>M Transceiver52M/radioInterfaceMulti.cpp<br>2 files changed, 59 insertions(+), 22 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/68/16668/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/16668">change 16668</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/+/16668"/><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: I2f3d66a611d3a489b3e4d9431994f4ec77b4460f </div>
<div style="display:none"> Gerrit-Change-Number: 16668 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: tnt <tnt@246tNt.com> </div>
<div style="display:none"> Gerrit-Reviewer: ttsou <tom@tsou.cc> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>