<p><a href="https://gerrit.osmocom.org/13662">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/13662/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/13662/1//COMMIT_MSG@7">Patch Set #1, Line 7:</a> <code style="font-family:monospace,monospace"> move LMS_GetLPFBWRange and LMS_Calibrate calls from open to start (bw, freq, power need to be set)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same as before:</p><ul><li>prepend "lms:"</li><li>Move description on why it is done (your parenthesis) out of first line and extend it.</li></ul></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13662/1/Transceiver52M/device/lms/LMSDevice.cpp">File Transceiver52M/device/lms/LMSDevice.cpp:</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/13662/1/Transceiver52M/device/lms/LMSDevice.cpp@377">Patch Set #1, Line 377:</a> <code style="font-family:monospace,monospace">     LOGC(DDEV, INFO) << "Setting LPFBW chan " << chan;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">From line 361 up to here (at least it seems) to be totally independent from channel, so you shouldn't be calling it for each chan. Better Move that out and pass lpfbw_rx and lpfbw_tx to do_filters().</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13662">change 13662</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/13662"/><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: I1090effdf0f43e5183a402e4c1a1ffe5abdefd37 </div>
<div style="display:none"> Gerrit-Change-Number: 13662 </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, 16 Apr 2019 15:52:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>