<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not a big fan in this, as the idea that only one logical "BTS" lives within osmo-bts is (as the TODO states) a hack and nothing that we want to perpetuate even further.</p><p style="white-space: pre-wrap; word-wrap: break-word;">There is hardware out there (e.g. osmo-bts-octphy) where the PHY handles multiple TRX in one "instance" of the PHY API.  You can use those multiple TRX as different TRX of the same BTS, but there may also be configurations where you actually want to operate multiple logical BTS on those TRX.  Running two instances of osmo-bts is not possible as you only have one PHY interface/API/... to attach to.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Neither I am a fan of the current 'single & global BTS' approach, and it's not like my code introducing this hack. I am simply moving things around, because my first attempt to un-globalize the 'struct gsm_bts' instance took me too much time: it's relatively easy for osmo-bts-trx, but osmo-trx-sysmo and the copy-pasted variants (sorry, but there actually is quite a lot of duplicated code) register VTY commands, where the single BTS is assumed and you cannot select specific BTS. So I had to stash everything and proceed with this approach. How should I proceed?</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/21011">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/21011">change 21011</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-bts/+/21011"/><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-Change-Id: Iee7fee6747dd1e7c0af36f9b27326f651ae37aaf </div>
<div style="display:none"> Gerrit-Change-Number: 21011 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </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-Comment-Date: Mon, 02 Nov 2020 09:33:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>