<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24374">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/+/24374/20/src/osmo-bsc/abis_rsl.c">File src/osmo-bsc/abis_rsl.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/osmo-bsc/+/24374/20/src/osmo-bsc/abis_rsl.c@506">Patch Set #20, Line 506:</a> <code style="font-family:monospace,monospace">struct rsl_osmo_training_sequence_ie {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Looks like this belongs to libosmogsm or libosmoabis, because somo-bts also requires it.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">that's the weird thing about our RSL implementation: we have the protocol TLV list and IE defines and so on in libosmocore, but every program implements its own message encoding and decoding functions all over.<br>So I agree that this encoding and decoding of the IE should have lived in libosmocore, sort of like we do with the BSSAP messaging, but so far it kind of belongs here instead.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So now I asked Vadim whether he's going to use this struct in osmo-bts source, but he says no.<br>Since it's only two octets he rather reads them directly from the data.<br>That would have been a reason to move to libosmocore.</p><p style="white-space: pre-wrap; word-wrap: break-word;">(I could also just msg_put_u8() twice here...)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24374">change 24374</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/+/24374"/><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: Ibf53f4797d7491b17a33946fd7d920f038362b4c </div>
<div style="display:none"> Gerrit-Change-Number: 24374 </div>
<div style="display:none"> Gerrit-PatchSet: 20 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <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-Comment-Date: Wed, 09 Jun 2021 15:00:35 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>