<p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/24388">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/+/24388/3/src/osmo-bts-trx/l1_if.h">File src/osmo-bts-trx/l1_if.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/osmo-bts/+/24388/3/src/osmo-bts-trx/l1_if.h@108">Patch Set #3, Line 108:</a> <code style="font-family:monospace,monospace">   bool                    setslot_valid[TRX_NR_TS];</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why not merging too the valid and sent into the new struct array you created? Feels like you went ha […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Just wanted to keep it consistent with the other fields above, but I am also fine with merging it into the new structure array.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/24388/3/src/osmo-bts-trx/trx_provision_fsm.c">File src/osmo-bts-trx/trx_provision_fsm.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/+/24388/3/src/osmo-bts-trx/trx_provision_fsm.c@521">Patch Set #3, Line 521:</a> <code style="font-family:monospace,monospace">          trx_if_cmd_setslot(l1h, ts_data->tn, l1if_setslot_cb);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">shouldn't you send it only if it didn't change?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I could introduce additional checks for the TSC fields too, but does it really hurt if we sent it with the same parameters as before? So far this event is generated on receipt of the NM_MT_SET_CHAN_ATTR message, which is usually sent once for each timeslot during the OML bootstrapping.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/24388">change 24388</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/+/24388"/><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: Idc5796151e3e83f42d60c2d4cb7c35890d76a7f5 </div>
<div style="display:none"> Gerrit-Change-Number: 24388 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 25 May 2021 21:02:47 +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>