<p>Vadim Yanitskiy <strong>uploaded patch set #6</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18591">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Do not mix public and private BTS features, use libosmocore's API<br><br>It was a very bad idea to mix "public" BTS features, that are<br>reported to the BSC via OML, and those features, that are used<br>locally (and exclusively) in osmo-bts.<br><br>Why? At least because we already have the BTS feature manipulation<br>API in libosmocore, that is used by osmo-bsc, but for some reason<br>not by osmo-bts. New features added to libosmocore would clash<br>with the existing "internal" ones like BTS_FEAT_MS_PWR_CTRL_DSP.<br><br>So what this change does can be described as follows:<br><br>  - remove duplicate definition of the "public" features,<br>  - use libosmocore's API for the "public" features,<br>  - separate both "internal" and "public" features:<br>    - the "public" features continue to live in bitvec,<br>    - the "internal" features become flags,<br>    - s/BTS_FEAT/BTS_IMPL_FLAG/g.<br><br>As a bonus, let's allocate the feature vector using libosmocore's<br>bitvec API instead of doing this manually in gsm_bts_alloc().<br><br>Change-Id: Icf792d02323bb73e3b8d46384c7890cb1eb4731e<br>---<br>M include/osmo-bts/gsm_data_shared.h<br>M src/common/bts.c<br>M src/common/gsm_data_shared.c<br>M src/common/l1sap.c<br>M src/common/oml.c<br>M src/common/vty.c<br>M src/osmo-bts-litecell15/main.c<br>M src/osmo-bts-oc2g/main.c<br>M src/osmo-bts-octphy/l1_if.c<br>M src/osmo-bts-sysmo/main.c<br>M src/osmo-bts-trx/main.c<br>M src/osmo-bts-virtual/main.c<br>M tests/misc/misc_test.c<br>13 files changed, 115 insertions(+), 136 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/91/18591/6</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/18591">change 18591</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/+/18591"/><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: Icf792d02323bb73e3b8d46384c7890cb1eb4731e </div>
<div style="display:none"> Gerrit-Change-Number: 18591 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <vyanitskiy@sysmocom.de> </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-MessageType: newpatchset </div>