From 0168bff835371140c0d75cc381e3228f8093fe70 Mon Sep 17 00:00:00 2001
From: Josh Blum <josh@joshknows.com>
Date: Mon, 25 Oct 2010 19:22:51 -0700
Subject: usrp: rework lo offset logic to use bandwidth, add test case

---
 host/lib/usrp/tune_helper.cpp | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

(limited to 'host/lib/usrp')

diff --git a/host/lib/usrp/tune_helper.cpp b/host/lib/usrp/tune_helper.cpp
index a6b615926..fa40a8a26 100644
--- a/host/lib/usrp/tune_helper.cpp
+++ b/host/lib/usrp/tune_helper.cpp
@@ -19,6 +19,7 @@
 #include <uhd/usrp/subdev_props.hpp>
 #include <uhd/usrp/dsp_props.hpp>
 #include <uhd/usrp/dboard_iface.hpp> //unit_t
+#include <uhd/utils/algorithm.hpp>
 #include <boost/math/special_functions/sign.hpp>
 #include <cmath>
 
@@ -43,9 +44,11 @@ static tune_result_t tune_xx_subdev_and_dsp(
     //------------------------------------------------------------------
     double lo_offset = 0.0;
     if (subdev[SUBDEV_PROP_USE_LO_OFFSET].as<bool>()){
-        //if the local oscillator will be in the passband, use an offset
-        //TODO make this nicer, use bandwidth property to clip bounds
-        lo_offset = 2.0*dsp[DSP_PROP_HOST_RATE].as<double>();
+        //If the local oscillator will be in the passband, use an offset.
+        //But constrain the LO offset by the width of the filter bandwidth.
+        double rate = dsp[DSP_PROP_HOST_RATE].as<double>();
+        double bw = subdev[SUBDEV_PROP_BANDWIDTH].as<double>();
+        if (bw > rate) lo_offset = std::min((bw - rate)/2, rate/2);
     }
 
     //------------------------------------------------------------------
-- 
cgit v1.2.3