<p>Pau Espin Pedrol <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/11681">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">abis_nm: Add support to parse some nanoBTS Attribute Response attributes<br><br>nanoBTS actually supports regular formatting. There are a few differences<br>with spec though:<br>* The attributes are listed directly in the message instead of being inside<br>  the Get Attributes Response Info after the unsupported attribute ID list.<br>* The Get Attributes Response Info can be at any position in the<br>  attribute list, and it only contains the unsupported attribute ID list.<br><br>As a result, parsing is currently split into 3 main parts or functions:<br>* Parsing regular (per spec) Get Attributes Response Info attr and get a<br>  pointer to the list of attributes.<br>* A function that parses the list of attributes, called directly in case<br>  of nanoBTS, and called by the former parser of Get Attributes Response<br>  Info for regular (per spec) OML endpoints.<br>* A function to parse the unsupported attribute ID list, also used in the<br>  first function to get a pointer to the list of attributes.<br><br>Related: OS#3624<br>Change-Id: I52e9f177c14fec1ec3f5c4ddb244594409008357<br>---<br>M src/osmo-bsc/abis_nm.c<br>1 file changed, 2 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bsc/abis_nm.c b/src/osmo-bsc/abis_nm.c</span><br><span>index 76fba59..005c417 100644</span><br><span>--- a/src/osmo-bsc/abis_nm.c</span><br><span>+++ b/src/osmo-bsc/abis_nm.c</span><br><span>@@ -569,7 +569,8 @@</span><br><span>   /* nanoBTS doesn't send Get Attribute Response Info, uses its own format */</span><br><span>      if (bts->type != GSM_BTS_TYPE_NANOBTS)</span><br><span>            return parse_attr_resp_info(bts, trx, foh, &tp);</span><br><span style="color: hsl(0, 100%, 40%);">-    return 0;</span><br><span style="color: hsl(120, 100%, 40%);">+     else</span><br><span style="color: hsl(120, 100%, 40%);">+          return parse_attr_resp_info_attr(bts, trx, foh, &tp);</span><br><span> }</span><br><span> </span><br><span> /* 3GPP TS 52.021 §6.2.5 */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/11681">change 11681</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/11681"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: I52e9f177c14fec1ec3f5c4ddb244594409008357 </div>
<div style="display:none"> Gerrit-Change-Number: 11681 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </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>