moved spi transact to usrp2 impl, and removed spi read

3 files changed, 52 insertions, 49 deletions

diff --git a/host/lib/usrp/usrp2/dboard_interface.cpp b/host/lib/usrp/usrp2/dboard_interface.cpp
index eee7c087a..d10cfa37e 100644
--- a/host/lib/usrp/usrp2/dboard_interface.cpp
+++ b/host/lib/usrp/usrp2/dboard_interface.cpp
@@ -47,32 +47,16 @@ public:
         const spi_config_t &config,
         boost::uint32_t data,
         size_t num_bits
-    ){
-        transact_spi(unit, config, data, num_bits, false /*no rb*/);
-    }
-
-    boost::uint32_t read_spi(
-        unit_type_t unit,
-        const spi_config_t &config,
-        size_t num_bits
-    ){
-        return transact_spi(unit, config, 0, num_bits, true /*rb*/);
-    }
+    );
 
     boost::uint32_t read_write_spi(
         unit_type_t unit,
         const spi_config_t &config,
         boost::uint32_t data,
         size_t num_bits
-    ){
-        return transact_spi(unit, config, data, num_bits, true /*rb*/);
-    }
-
-private:
-    boost::uint32_t transact_spi(
-        unit_type_t, const spi_config_t &, boost::uint32_t, size_t, bool
     );
 
+private:
     usrp2_impl *_impl;
     boost::uint32_t _ddr_shadow;
 };
@@ -175,42 +159,22 @@ static boost::uint8_t unit_to_otw_dev(dboard_interface::unit_type_t unit){
     throw std::invalid_argument("unknown unit type type");
 }
 
-/*!
- * Static function to convert a spi edge enum
- * to an over-the-wire value for the usrp2 control.
- * \param edge the dboard interface spi edge enum
- * \return an over the wire representation
- */
-static boost::uint8_t spi_edge_to_otw(dboard_interface::spi_config_t::edge_t edge){
-    switch(edge){
-    case dboard_interface::spi_config_t::EDGE_RISE: return USRP2_CLK_EDGE_RISE;
-    case dboard_interface::spi_config_t::EDGE_FALL: return USRP2_CLK_EDGE_FALL;
-    }
-    throw std::invalid_argument("unknown spi edge type");
+void usrp2_dboard_interface::write_spi(
+    unit_type_t unit,
+    const spi_config_t &config,
+    boost::uint32_t data,
+    size_t num_bits
+){
+    _impl->transact_spi(unit_to_otw_dev(unit), config, data, num_bits, false /*no rb*/);
 }
 
-boost::uint32_t usrp2_dboard_interface::transact_spi(
+boost::uint32_t usrp2_dboard_interface::read_write_spi(
     unit_type_t unit,
     const spi_config_t &config,
     boost::uint32_t data,
-    size_t num_bits,
-    bool readback
+    size_t num_bits
 ){
-    //setup the out data
-    usrp2_ctrl_data_t out_data;
-    out_data.id = htonl(USRP2_CTRL_ID_TRANSACT_ME_SOME_SPI_BRO);
-    out_data.data.spi_args.dev = unit_to_otw_dev(unit);
-    out_data.data.spi_args.miso_edge = spi_edge_to_otw(config.miso_edge);
-    out_data.data.spi_args.mosi_edge = spi_edge_to_otw(config.mosi_edge);
-    out_data.data.spi_args.readback = (readback)? 1 : 0;
-    out_data.data.spi_args.num_bits = num_bits;
-    out_data.data.spi_args.data = htonl(data);
-
-    //send and recv
-    usrp2_ctrl_data_t in_data = _impl->ctrl_send_and_recv(out_data);
-    ASSERT_THROW(htonl(in_data.id) == USRP2_CTRL_ID_OMG_TRANSACTED_SPI_DUDE);
-
-    return ntohl(out_data.data.spi_args.data);
+    return _impl->transact_spi(unit_to_otw_dev(unit), config, data, num_bits, true /*rb*/);
 }
 
 /***********************************************************************
diff --git a/host/lib/usrp/usrp2/usrp2_impl.cpp b/host/lib/usrp/usrp2/usrp2_impl.cpp
index a7be2da8c..6d8c1a86c 100644
--- a/host/lib/usrp/usrp2/usrp2_impl.cpp
+++ b/host/lib/usrp/usrp2/usrp2_impl.cpp
@@ -15,15 +15,16 @@
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
 //
 
+#include "usrp2_impl.hpp"
 #include <uhd/transport/if_addrs.hpp>
 #include <uhd/usrp/device_props.hpp>
 #include <uhd/utils/assert.hpp>
 #include <uhd/utils/static.hpp>
+#include <boost/assign/list_of.hpp>
 #include <boost/format.hpp>
 #include <boost/foreach.hpp>
 #include <boost/bind.hpp>
 #include <iostream>
-#include "usrp2_impl.hpp"
 
 using namespace uhd;
 using namespace uhd::usrp;
@@ -217,6 +218,35 @@ boost::uint16_t usrp2_impl::peek16(boost::uint32_t addr){
     return impl_peek<boost::uint16_t>(this, addr);
 }
 
+boost::uint32_t usrp2_impl::transact_spi(
+    int which_slave,
+    const spi_config_t &config,
+    boost::uint32_t data,
+    size_t num_bits,
+    bool readback
+){
+    static const uhd::dict<spi_config_t::edge_t, int> spi_edge_to_otw = boost::assign::map_list_of
+        (spi_config_t::EDGE_RISE, USRP2_CLK_EDGE_RISE)
+        (spi_config_t::EDGE_FALL, USRP2_CLK_EDGE_FALL)
+    ;
+
+    //setup the out data
+    usrp2_ctrl_data_t out_data;
+    out_data.id = htonl(USRP2_CTRL_ID_TRANSACT_ME_SOME_SPI_BRO);
+    out_data.data.spi_args.dev = which_slave;
+    out_data.data.spi_args.miso_edge = spi_edge_to_otw[config.miso_edge];
+    out_data.data.spi_args.mosi_edge = spi_edge_to_otw[config.mosi_edge];
+    out_data.data.spi_args.readback = (readback)? 1 : 0;
+    out_data.data.spi_args.num_bits = num_bits;
+    out_data.data.spi_args.data = htonl(data);
+
+    //send and recv
+    usrp2_ctrl_data_t in_data = this->ctrl_send_and_recv(out_data);
+    ASSERT_THROW(htonl(in_data.id) == USRP2_CTRL_ID_OMG_TRANSACTED_SPI_DUDE);
+
+    return ntohl(out_data.data.spi_args.data);
+}
+
 /***********************************************************************
  * Control Send/Recv
  **********************************************************************/
diff --git a/host/lib/usrp/usrp2/usrp2_impl.hpp b/host/lib/usrp/usrp2/usrp2_impl.hpp
index 4e7154afa..5a02b33dc 100644
--- a/host/lib/usrp/usrp2/usrp2_impl.hpp
+++ b/host/lib/usrp/usrp2/usrp2_impl.hpp
@@ -110,6 +110,15 @@ public:
     void poke16(boost::uint32_t addr, boost::uint16_t data);
     boost::uint16_t peek16(boost::uint32_t addr);
 
+    //spi read and write
+    boost::uint32_t transact_spi(
+        int which_slave,
+        const uhd::usrp::spi_config_t &config,
+        boost::uint32_t data,
+        size_t num_bits,
+        bool readback
+    );
+
     //misc access methods
     double get_master_clock_freq(void);