<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18821">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-bts/+/18821/1/src/osmo-bts-trx/scheduler_trx.c">File src/osmo-bts-trx/scheduler_trx.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/+/18821/1/src/osmo-bts-trx/scheduler_trx.c@89">Patch Set #1, Line 89:</a> <code style="font-family:monospace,monospace">                    /* TODO: Tx attenuation is always 0? */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Right. Do you know if osmo-trx supports per-burst power attenuation? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">osmo-trx does use the value, see Transceiver::addRadioVector() https://git.osmocom.org/osmo-trx/tree/Transceiver52M/Transceiver.cpp#n373 (the variable is wrongly named RSSI there):</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br>"""<br>addRadioVector(chan, newBurst, dl->tx_att, currTime);<br>// which uses it like this: (dl->tx_att is RSSI)<br>  scaleVector(*burst, txFullScale * pow(10, -RSSI / 10));<br>"""</pre><p style="white-space: pre-wrap; word-wrap: break-word;">According to Harald the attenuation value is to be applied on top of already global one, so iiuc it's doing the correct thing here (further attenuating it if the value is !=0).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/18821">change 18821</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/+/18821"/><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: If3014e69746559963569b77561dbf7b163c68ffa </div>
<div style="display:none"> Gerrit-Change-Number: 18821 </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-Reviewer: fixeria <vyanitskiy@sysmocom.de> </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-Comment-Date: Mon, 15 Jun 2020 10:20:21 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>