aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosh Blum <josh@joshknows.com>2011-12-15 18:26:43 -0800
committerJosh Blum <josh@joshknows.com>2011-12-15 18:26:43 -0800
commitd595f159674e507d6438174f53e0c8027239bcea (patch)
treeac1bcd4fd618647f95f2bcee74d33f1ad85593a7
parent35bd9dd1f29649a642264d628f4cdd0465a024a3 (diff)
downloaduhd-d595f159674e507d6438174f53e0c8027239bcea.tar.gz
uhd-d595f159674e507d6438174f53e0c8027239bcea.tar.bz2
uhd-d595f159674e507d6438174f53e0c8027239bcea.zip
usrp1: fixed swapped sign on rx cordic
We used to swap I and Q to share converter functions with the newer products. Because of this, the sign on the cordic also had to be swapped. Now that USRP1 has its own converter routines and I and Q are correct, so should be the sign on the cordic. This was a bug introduced in master. TX should not be an issue, because there is no cordic and is adjusted digitally by the codec, after any potential swapping.
-rw-r--r--host/lib/usrp/usrp1/io_impl.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/host/lib/usrp/usrp1/io_impl.cpp b/host/lib/usrp/usrp1/io_impl.cpp
index ff89b2e18..234bf1b25 100644
--- a/host/lib/usrp/usrp1/io_impl.cpp
+++ b/host/lib/usrp/usrp1/io_impl.cpp
@@ -543,7 +543,7 @@ double usrp1_impl::update_rx_dsp_freq(const size_t dspno, const double freq_){
static const boost::uint32_t dsp_index_to_reg_val[4] = {
FR_RX_FREQ_0, FR_RX_FREQ_1, FR_RX_FREQ_2, FR_RX_FREQ_3
};
- _iface->poke32(dsp_index_to_reg_val[dspno], ~freq_word + 1);
+ _iface->poke32(dsp_index_to_reg_val[dspno], freq_word);
return (double(freq_word) / scale_factor) * _master_clock_rate;
}