<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2: -Code-Review</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">As it is done now, osmo-trx expects to always receive DL bursts all the time, so we are breaking stuff if we simply stop sending them.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hmm, this is probably what Hoernchen was talking about. And it only applies to C0, right? Because in a multi-trx setup, all additional transceivers send no bursts most of the time, and it works. Now I am wondering why it is the case, given that osmo-trx is a clock master, and it can simply send zero-samples on C0 if there is nothing to send, just like it does for C1..N.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It applies to C0, and yes, it can be configured to fill in buffers  with whatever fill method is required. BUt my main point is that it makes more sense to apply RF mute in osmo-trx since it knows/controls the HW. This means for some HW it could for instance turn off specific baseband HW to avoid consuming/heating while locked, or avoid spending CPU time.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/19827">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/19827">change 19827</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/+/19827"/><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: I1423ddb390ef327ec7d4a27de2ac5dca663773a5 </div>
<div style="display:none"> Gerrit-Change-Number: 19827 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 31 Aug 2020 11:20:12 +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>