<p style="white-space: pre-wrap; word-wrap: break-word;">So I find the way you chose to structure abstraction a bit weird tbh. That being said, I'm not going to block this, but I think it'd be less confusing if for instance, you added a function pointer for "initial" setup and one for "channel_act" setup of the IE, and implement that in each bts type.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Or for instance put the initial IE setup in the the ipacc_tx_..._def you are adding in the last commit, and use the function pointer you add in this commit as the "channel activ" one.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It's all even more confusing because you spread all this through different commits. Just sharing my opinion in case you want to give it another thought.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21787">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21787">change 21787</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/+/21787"/><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: I586b1c9e16390757b015c2871a36abc3975e0fc4 </div>
<div style="display:none"> Gerrit-Change-Number: 21787 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 18 Dec 2020 17:51:09 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>