<p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/17855">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/17855/4/src/osmo-bts-litecell15/l1_if.c">File src/osmo-bts-litecell15/l1_if.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-bts/+/17855/4/src/osmo-bts-litecell15/l1_if.c@1608">Patch Set #4, Line 1608:</a> <code style="font-family:monospace,monospace">static void dsp_alive_compl_cb(struct gsm_bts_trx *trx, struct msgb *resp, void *data)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Could you be explicit on which change do you want to be made?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The one that doesn't introduce a compiler warning.</p><p style="white-space: pre-wrap; word-wrap: break-word;">At this point, I'd rather defer to somebody else, as I don't know enough about coding standards and style to say if the correct thing here is void because nothing to return and so what if everything else says it should be int.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In terms of following the oc2g code, i would just point out that was 99% bulk merged in https://gerrit.osmocom.org/#/c/osmo-bts/+/11447/ and possibly (probably) contains the same non compliance with osmo standards?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/17855/4/src/osmo-bts-litecell15/l1_if.c@1722">Patch Set #4, Line 1722:</a> <code style="font-family:monospace,monospace">     fl1h->hw_alive.dsp_alive_timer.cb = dsp_alive_timer_cb;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Which logic error you see?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I read it wrong; that -EIO won't happen due to the L1 not responding. </p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">I still feel however that we're introducing bad code here. Maybe that's fine if it'll all get cleaned up in subsequent unification of oc2g/lc15<br> <br>I'll just defer.. see if somebody else gives +1</pre><p style="white-space: pre-wrap; word-wrap: break-word;">That's all from me.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/17855">change 17855</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-bts/+/17855"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ib16e7d423fc7030573acd86fbd356ae96697ed5d </div>
<div style="display:none"> Gerrit-Change-Number: 17855 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: rafael2k <rafael@rhizomatica.org> </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-Reviewer: rafael2k <rafael@rhizomatica.org> </div>
<div style="display:none"> Gerrit-CC: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 20 Apr 2020 19:29:20 +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: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Comment-In-Reply-To: rafael2k <rafael@rhizomatica.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>