<p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22801">View Change</a></p><p>3 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/+/22801/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/+/22801/1/doc/manuals/chapters/configuration.adoc@216">Patch Set #1, Line 216:</a> <code style="font-family:monospace,monospace">Parameter `ALPHA` is set on the BSC VTY configuration file on a per-BTS basis,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Please also clarify here that presence of 'alpha <0-10>' in the config file makes osmo-pcu ignore the value received from the BSC. Some users tend to ignore deprecation warnings...</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22801/1/src/pcu_vty.c">File src/pcu_vty.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-pcu/+/22801/1/src/pcu_vty.c@234">Patch Set #1, Line 234:</a> <code style="font-family:monospace,monospace">       vty_out(vty, " alpha %u%s", the_pcu->vty.force_alpha, VTY_NEWLINE);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">code formatting</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/+/22801/1/src/pcu_vty.c@746">Patch Set #1, Line 746:</a> <code style="font-family:monospace,monospace">             the_pcu->vty.force_alpha = atoi(argv[0]);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">How about printing a warning that presence of 'alpha <0-10>' in the configuration file makes osmo-pcu ignore the value received from the BSC?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22801">change 22801</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/+/22801"/><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: I46f2a955b157a409055fca7fb917dc4f75482426 </div>
<div style="display:none"> Gerrit-Change-Number: 22801 </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, 09 Feb 2021 19:07:49 +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>