<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21787">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/+/21787/1/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/+/21787/1/src/osmo-bsc/abis_rsl.c@516">Patch Set #1, Line 516:</a> <code style="font-family:monospace,monospace">               msgb_tlv_put(msg, tag, 0, NULL);</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">TThe 2 switch cases above (nanobts, osmobts), are afaiu the same as the default one when calling a enc_power_rams as a NOOP, am I correct?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, if bts->model->enc_power_params() would append no vendor-specific IEs, then we get an empty TLV (L=0) like in the two cases above. If bts->model->enc_power_params is NULL, then MS/BS Power Parameters IE is not included at all.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">So if we have an abstracted way to set the IE contents (enc_power_params func pointer), why do we want to have this switch case for osmobts and nanobts?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Because ip.access models allow us to send all parameters only once, in "Measurement Pre-processing Defaults" message, and then avoid repeating them again and again in CHANnel ACTIVation messages. So we use this possibility here. For other BTS models you still need to send those parameters in every CHANnel ACTIVation message.</p><p style="white-space: pre-wrap; word-wrap: break-word;">And I intentionally kept this decision logic (whether to send an empty IE) separate from bts->model->enc_power_params(), because this function does not (and is not supposed to) know in which cases we should skip parameters and in which we should add them.</p></li></ul></li></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:44:04 +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"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>