<p style="white-space: pre-wrap; word-wrap: break-word;">There seems to be a problem...</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/osmocom-bb/+/24016">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/osmocom-bb/+/24016/2/src/target/trx_toolkit/data_msg.py">File src/target/trx_toolkit/data_msg.py:</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/osmocom-bb/+/24016/2/src/target/trx_toolkit/data_msg.py@40">Patch Set #2, Line 40:</a> <code style="font-family:monospace,monospace">       ModAQPSK        = (0b1100, 2 * GMSK_BURST_LEN)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I have re-calculated the factors for the burst length. They seem to be correct, but for ModAQPSK the left side has 0b0110 and the right side has 0b1100 as index (?). To me this looks like an error.</p><p style="white-space: pre-wrap; word-wrap: break-word;">ModGMSK  = (0b0000, 148) => 1 * GMSK_BURST_LEN<br>Mod8PSK  = (0b0100, 444) => 3 * GMSK_BURST_LEN<br>ModAQPSK = (0b0110, 296) => 2 * GMSK_BURST_LEN (0b0110 != 0b1100)<br>Mod16QAM = (0b1000, 592) => 4 * GMSK_BURST_LEN<br>Mod32QAM = (0b1010, 740) => 5 * GMSK_BURST_LEN</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmocom-bb/+/24016">change 24016</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/osmocom-bb/+/24016"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmocom-bb </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ia96b263bbb162b6c40f2cf81616118cc297299a5 </div>
<div style="display:none"> Gerrit-Change-Number: 24016 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 04 May 2021 10:03:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>