<p>pespin has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/25474">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">scheduler: Apply BCCH Carrier Power Reduction attenuation to PDCH TS on C0<br><br>The commit implementing BCCH Carrier Power Reduction forgot to apply it<br>for bursts for which there's no lchan controlled by OsmoBTS, such as<br>those coming from PDCH timeslots.<br><br>For other TRX, BS Power Reduction is still always 0. The BS Power Control<br>Loop for those channels still needs to be implemented in OsmoPCU and the<br>resulting bs_powerbe passed somehow over PCUIF together with RLCMAC data<br>to OsmoBTS, so it can be applied there.<br><br>Fixes: 0e8d68437af4a99dc99d11d4e1f441246bd3ec00<br>Related: SYS#4919<br>Change-Id: I3c6985c2bc6779b15094a34f5d4a345c7d8d7b38<br>---<br>M src/common/scheduler.c<br>1 file changed, 5 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/74/25474/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/common/scheduler.c b/src/common/scheduler.c</span><br><span>index 6e1e4fb..e7eac96 100644</span><br><span>--- a/src/common/scheduler.c</span><br><span>+++ b/src/common/scheduler.c</span><br><span>@@ -1318,7 +1318,11 @@</span><br><span>      /* BS Power reduction (in dB) per logical channel */</span><br><span>         if (l1cs->lchan != NULL)</span><br><span>          br->att = l1cs->lchan->bs_power_ctrl.current;</span><br><span style="color: hsl(0, 100%, 40%);">-  else /* Ensure no attenuation in the absence of lchan (e.g. on PDCH) */</span><br><span style="color: hsl(120, 100%, 40%);">+       /* Ensure no/reduced attenuation in the absence of lchan (e.g. on PDCH).</span><br><span style="color: hsl(120, 100%, 40%);">+       * Use calculated value for TS on TRX0 as per BCCH Carrier Power Reduction: */</span><br><span style="color: hsl(120, 100%, 40%);">+        else if (l1ts->ts->trx == l1ts->ts->trx->bts->c0)</span><br><span style="color: hsl(120, 100%, 40%);">+           br->att = l1ts->ts->c0_power_red_db;</span><br><span style="color: hsl(120, 100%, 40%);">+ else</span><br><span>                 br->att = 0;</span><br><span> </span><br><span>  /* encrypt */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/25474">change 25474</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/+/25474"/><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: I3c6985c2bc6779b15094a34f5d4a345c7d8d7b38 </div>
<div style="display:none"> Gerrit-Change-Number: 25474 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>