<p>Patch set 6:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ddd;">-Code-Review</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21164">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-bsc/+/21164/6/include/osmocom/bsc/bts.h">File include/osmocom/bsc/bts.h:</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-bsc/+/21164/6/include/osmocom/bsc/bts.h@485">Patch Set #6, Line 485:</a> <code style="font-family:monospace,monospace">repeated_acch_capability_bts</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">what is a bit uncleaer to me is what exactly we are configuring here.  </p><p style="white-space: pre-wrap; word-wrap: break-word;">Is the user expected to confiure that his BTS is capable of supporting those features?  In that case, this should be automatically detected based on feature flags at OML startup.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Is the user expected to confgure a policy, i.e. that the BSC is permitted to use those feautres if a) the BTS is capable and b) the MS is capable?</p><p style="white-space: pre-wrap; word-wrap: break-word;">The fact that the variable is called "capability_bts" seems to indicate that you are configuring what the BTS is capable of doing.  And that's really something the user shouldn't have to know.  Capabilities of the BTS are reported at OML startup.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21164">change 21164</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-bsc/+/21164"/><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-Change-Id: I083eaa2c30478912426e9c24a506f0b88836e190 </div>
<div style="display:none"> Gerrit-Change-Number: 21164 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@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-CC: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 20 Nov 2020 09:02:56 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>