<p><a href="https://gerrit.osmocom.org/11949">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11949/1//COMMIT_MSG">Commit Message:</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/11949/1//COMMIT_MSG@7">Patch Set #1, Line 7:</a> <code style="font-family:monospace,monospace">change scaling factor from .3 to .7 for lms sdr devices to improve the signal slope</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prepend "lms:" for easy/quickly categorization of scope of the patch. Something like:</p><p style="white-space: pre-wrap; word-wrap: break-word;">lms: Change scaling factor from .3 to .7 to improve signal slope</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11949/1/Transceiver52M/device/lms/LMSDevice.h">File Transceiver52M/device/lms/LMSDevice.h:</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/11949/1/Transceiver52M/device/lms/LMSDevice.h@31">Patch Set #1, Line 31:</a> <code style="font-family:monospace,monospace">// this was 0.3 before - values above 0.8 give more phase noise, values below 0.6 give too much amplitude resolution/noise to pass gmsk validation on a E4406</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">We usually use /* */ type comments. No need to specify it was 0.3 before, we have git log for that ;)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Probably add word "experimentally found" or similar to understand it doesn't come from some spec sheets from producer.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11949">change 11949</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/11949"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I2d1ba5cbd4ba98f912eb2e3e9f5383b69b28ff5b </div>
<div style="display:none"> Gerrit-Change-Number: 11949 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: roh <jsteiger@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 27 Nov 2018 12:26:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>