<p><a href="https://gerrit.osmocom.org/9666">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9666/3/include/osmocom/bsc/gsm_data.h">File include/osmocom/bsc/gsm_data.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/9666/3/include/osmocom/bsc/gsm_data.h@755">Patch Set #3, Line 755:</a> <code style="font-family:monospace,monospace">struct gsm_bts_ref {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">ah. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">not a request, just a reminder for future cases.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9666/3/include/osmocom/bsc/neighbor_ident.h">File include/osmocom/bsc/neighbor_ident.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/9666/3/include/osmocom/bsc/neighbor_ident.h@18">Patch Set #3, Line 18:</a> <code style="font-family:monospace,monospace">        BSIC_9BIT,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">23.003 "4.3.2 Base Station Identify Code (BSIC)" […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ah, I didn't follow EC-GSM-IoT yet in that level of detail.  We don't support it (and it seems like it's becoming one of the "born dead" inventions of the GSM/3GPP specs).  We don't deal with a 9bit BSIC in any other part of the code in BTS or BSC, so it might be offering options/choices to people that are more confusing than helping.  Your call.  Either keep it in, or #ifdef it out?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9666/3/src/osmo-bsc/neighbor_ident_vty.c">File src/osmo-bsc/neighbor_ident_vty.c:</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/9666/3/src/osmo-bsc/neighbor_ident_vty.c@560">Patch Set #3, Line 560:</a> <code style="font-family:monospace,monospace">   install_element(BTS_NODE, &cfg_neighbor_resolve_cmd);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">hm, so when I'm in the bts node and entered some neighbor entries, I can't just query on the same le […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">yes, this separation about VIEW/ENABLE and CONFIG node exists. prefixing it with "show" is of course always an option, as long as it's in the VIEW/ENABLE node.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9666/3/tests/handover/neighbor_ident_test.c">File tests/handover/neighbor_ident_test.c:</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/9666/3/tests/handover/neighbor_ident_test.c@33">Patch Set #3, Line 33:</a> <code style="font-family:monospace,monospace">arfcn</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">In the C API to neighbor_ident, the high bit should be handled transparently. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The User Interface and VTY is 0-1023.  There are no higher ARFCN number.  But the semantics of [some of that] number space differs whether the neighbor is in DCS or in PCS bands.  I don't know the specs well enough to understand how they differentiate neighbors between those two bands.  However, there are (supported) setups of our customers that either use 900+1800 (more standard) or (AFAIK) 900+1900 (!) MHz bands.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe this can be re-visited as a separate follow-up ticket,  unrelated to the specific test case: Ensure that our neighbor-handling works correctly with neighbors in either DCS or PCS bands.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9666">change 9666</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/9666"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I0153d7069817fba9146ddc11214de2757d7d37bf </div>
<div style="display:none"> Gerrit-Change-Number: 9666 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 24 Jul 2018 05:06:35 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>