<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24883">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24883/3/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/+/24883/3/src/osmo-bsc/abis_rsl.c@1489">Patch Set #3, Line 1489:</a> <code style="font-family:monospace,monospace">ts</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Cosmetic: I would make this pointer scoped, it's not used outside of the loop.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24883/3/src/osmo-bsc/abis_rsl.c@1498">Patch Set #3, Line 1498:</a> <code style="font-family:monospace,monospace">                LOGP(DRSL, LOGL_ERROR, "Rx Resource Indication: missing Resource Info IE\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Older versions of osmo-bts actually do send the RF RESource INDication without this IE (violating the specs). But this happens only once after the RSL connection establishment, after that no more RESource INDication messages are sent to the BSC. Just FYI.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24883/3/src/osmo-bsc/abis_rsl.c@1524">Patch Set #3, Line 1524:</a> <code style="font-family:monospace,monospace">lchan->ts->trx->bts</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You already have the 'trx' pointer, so you could do 'trx->bts' below.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24883">change 24883</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/+/24883"/><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: Iccc1391e8419604bb09e464db8455e053dfbc982 </div>
<div style="display:none"> Gerrit-Change-Number: 24883 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 11 Jul 2021 04:52:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>