<p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21943">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21943/1/doc/manuals/chapters/configuration.adoc">File doc/manuals/chapters/configuration.adoc:</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-pcu/+/21943/1/doc/manuals/chapters/configuration.adoc@246">Patch Set #1, Line 246:</a> <code style="font-family:monospace,monospace">f</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">typo</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21943/1/doc/manuals/chapters/configuration.adoc@250">Patch Set #1, Line 250:</a> <code style="font-family:monospace,monospace">encoding and decoding of 8-PSK modulation</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This looks confusing, modulation is not something you encode and decode. I would say:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  "... the BTS must support 8-PSK modulation"</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21943/1/doc/manuals/chapters/configuration.adoc@250">Patch Set #1, Line 250:</a> <code style="font-family:monospace,monospace">in the event 8-PSK</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I may be wrong, but using "in the event" looks inappropriate in this context. It's not something that happens sporadically or unexpectedly, so it's not an event. Maybe "in case if 8-PSK is not supported"?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21943/1/doc/manuals/chapters/configuration.adoc@266">Patch Set #1, Line 266:</a> <code style="font-family:monospace,monospace">- It is possible to improve EGPRS performance (in particular, the TBF</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Copyright! :D</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21943/1/doc/manuals/chapters/configuration.adoc@302">Patch Set #1, Line 302:</a> <code style="font-family:monospace,monospace">encoded in GMSK</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"modulated with GMSK" or "GMSK modulated"</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21943">change 21943</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-pcu/+/21943"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I7b205f5cab5862058a408f628925beb9f0f60a92 </div>
<div style="display:none"> Gerrit-Change-Number: 21943 </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-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 05 Jan 2021 13:06:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>