From 9e2d669de18b40a69b212175bf1e6f44ad7cc5bc Mon Sep 17 00:00:00 2001 From: Josh Blum Date: Mon, 24 Jan 2011 10:58:34 -0800 Subject: usrp1: changes that make benchmark rx work initialize the time to something in soft time control use std::max(0, lost) because the time is emulated, lost can be negative which means huge positive unsigned... --- host/lib/usrp/usrp1/soft_time_ctrl.cpp | 3 +++ 1 file changed, 3 insertions(+) (limited to 'host/lib') diff --git a/host/lib/usrp/usrp1/soft_time_ctrl.cpp b/host/lib/usrp/usrp1/soft_time_ctrl.cpp index 856faf89d..c91ecc7ed 100644 --- a/host/lib/usrp/usrp1/soft_time_ctrl.cpp +++ b/host/lib/usrp/usrp1/soft_time_ctrl.cpp @@ -69,6 +69,9 @@ public: _thread_group.create_thread(boost::bind(&soft_time_ctrl_impl::recv_cmd_dispatcher, this)); _update_mutex.lock(); //lock blocks until spawned _update_mutex.unlock(); //unlock mutex before done + + //initialize the time to something + this->set_time(time_spec_t(0.0)); } ~soft_time_ctrl_impl(void){ -- cgit v1.2.3 From bf25ec24adc6e4d1aa563962452630628dad3215 Mon Sep 17 00:00:00 2001 From: Nick Foster Date: Tue, 25 Jan 2011 14:12:39 -0800 Subject: N210: longer GPS timeout, removed try/catch in safe_recv loop (since it can't throw any more) --- host/lib/usrp/gps_ctrl.cpp | 16 ++++------------ host/lib/usrp/usrp2/mboard_impl.cpp | 6 +++--- 2 files changed, 7 insertions(+), 15 deletions(-) (limited to 'host/lib') diff --git a/host/lib/usrp/gps_ctrl.cpp b/host/lib/usrp/gps_ctrl.cpp index 3c7c00134..b1062fa39 100644 --- a/host/lib/usrp/gps_ctrl.cpp +++ b/host/lib/usrp/gps_ctrl.cpp @@ -58,7 +58,7 @@ public: if(trim_right_copy(reply) == "Command Error") { gps_type = GPS_TYPE_JACKSON_LABS; break; - } + } else if(reply.substr(0, 3) == "$GP") i_heard_some_nmea = true; //but keep looking for that "Command Error" response else if(reply.length() != 0) i_heard_something_weird = true; //probably wrong baud rate boost::this_thread::sleep(boost::posix_time::milliseconds(200)); @@ -104,6 +104,7 @@ public: found_gprmc = true; break; } + boost::this_thread::sleep(boost::posix_time::milliseconds(200)); } if(!found_gprmc) { if(gps_type == GPS_TYPE_JACKSON_LABS) std::cout << "Firefly GPS not locked or warming up." << std::endl; @@ -127,16 +128,7 @@ public: //TODO: this isn't generalizeable to non-USRP2 USRPs. std::string safe_gps_read() { - std::string reply; - try { - reply = _recv(); - } catch (std::runtime_error err) { - if(err.what() != std::string("usrp2 no control response")) throw; //sorry can't cope with that - else { //we don't actually have a GPS installed - reply = std::string(); - } - } - return reply; + return _recv(); } ptime get_time(void) { @@ -196,7 +188,7 @@ private: GPS_TYPE_NONE } gps_type; - static const int GPS_TIMEOUT_TRIES = 5; + static const int GPS_TIMEOUT_TRIES = 10; static const int GPS_TIMEOUT_DELAY_MS = 200; static const int FIREFLY_STUPID_DELAY_MS = 200; diff --git a/host/lib/usrp/usrp2/mboard_impl.cpp b/host/lib/usrp/usrp2/mboard_impl.cpp index 95f7013e7..ee7ad99c5 100644 --- a/host/lib/usrp/usrp2/mboard_impl.cpp +++ b/host/lib/usrp/usrp2/mboard_impl.cpp @@ -66,9 +66,9 @@ usrp2_mboard_impl::usrp2_mboard_impl( //contruct the interfaces to mboard perifs _clock_ctrl = usrp2_clock_ctrl::make(_iface); _codec_ctrl = usrp2_codec_ctrl::make(_iface); - //_gps_ctrl = gps_ctrl::make( - // _iface->get_gps_write_fn(), - // _iface->get_gps_read_fn()); +// _gps_ctrl = gps_ctrl::make( +// _iface->get_gps_write_fn(), +// _iface->get_gps_read_fn()); //if(_gps_ctrl->gps_detected()) std::cout << "GPS time: " << _gps_ctrl->get_time() << std::endl; -- cgit v1.2.3 From db1fd30da6def1269eae62b5ccd3c81c28ae000f Mon Sep 17 00:00:00 2001 From: Josh Blum Date: Wed, 26 Jan 2011 11:27:25 -0800 Subject: uhd: fixed maxosx bug, was resizing the transport buffer but it cant be resized on macos and I messed up the code that blocked that behavior reimplemented block in usrp2_impl, seems more correct here --- host/lib/transport/udp_zero_copy_asio.cpp | 4 ---- host/lib/usrp/subdev_spec.cpp | 2 +- host/lib/usrp/usrp2/usrp2_impl.cpp | 7 +++++-- host/lib/utils/paths.cpp | 1 + 4 files changed, 7 insertions(+), 7 deletions(-) (limited to 'host/lib') diff --git a/host/lib/transport/udp_zero_copy_asio.cpp b/host/lib/transport/udp_zero_copy_asio.cpp index 5c049cfad..3826dcd79 100644 --- a/host/lib/transport/udp_zero_copy_asio.cpp +++ b/host/lib/transport/udp_zero_copy_asio.cpp @@ -353,14 +353,10 @@ template static void resize_buff_helper( "See the transport application notes on buffer resizing.\n%s" ) % name % min_sock_buff_size % help_message)); } - - //only enable on platforms that are happy with the large buffer resize - #if defined(UHD_PLATFORM_LINUX) || defined(UHD_PLATFORM_WIN32) //otherwise, ensure that the buffer is at least the minimum size else if (udp_trans->get_buff_size() < min_sock_buff_size){ resize_buff_helper(udp_trans, min_sock_buff_size, name); } - #endif /*defined(UHD_PLATFORM_LINUX) || defined(UHD_PLATFORM_WIN32)*/ } udp_zero_copy::sptr udp_zero_copy::make( diff --git a/host/lib/usrp/subdev_spec.cpp b/host/lib/usrp/subdev_spec.cpp index 51c88bda3..d5d950f1f 100644 --- a/host/lib/usrp/subdev_spec.cpp +++ b/host/lib/usrp/subdev_spec.cpp @@ -46,7 +46,7 @@ bool usrp::operator==(const subdev_spec_pair_t &lhs, const subdev_spec_pair_t &r subdev_spec_t::subdev_spec_t(const std::string &markup){ BOOST_FOREACH(const std::string &pair, pair_tokenizer(markup)){ - if (pair == "") continue; + if (pair.empty()) continue; std::vector db_sd; boost::split(db_sd, pair, boost::is_any_of(":")); switch(db_sd.size()){ case 1: this->push_back(subdev_spec_pair_t("", db_sd.front())); break; diff --git a/host/lib/usrp/usrp2/usrp2_impl.cpp b/host/lib/usrp/usrp2/usrp2_impl.cpp index 059ddf65f..9ce0f7359 100644 --- a/host/lib/usrp/usrp2/usrp2_impl.cpp +++ b/host/lib/usrp/usrp2/usrp2_impl.cpp @@ -202,8 +202,11 @@ static device::sptr usrp2_make(const device_addr_t &device_addr){ //setup the dsp transport hints (default to a large recv buff) device_addr_t dsp_xport_hints = device_addr; if (not dsp_xport_hints.has_key("recv_buff_size")){ - //set to half-a-second of buffering at max rate - dsp_xport_hints["recv_buff_size"] = "50e6"; + //only enable on platforms that are happy with the large buffer resize + #if defined(UHD_PLATFORM_LINUX) || defined(UHD_PLATFORM_WIN32) + //set to half-a-second of buffering at max rate + dsp_xport_hints["recv_buff_size"] = "50e6"; + #endif /*defined(UHD_PLATFORM_LINUX) || defined(UHD_PLATFORM_WIN32)*/ } //create a ctrl and data transport for each address diff --git a/host/lib/utils/paths.cpp b/host/lib/utils/paths.cpp index 93d15d290..8d604d849 100644 --- a/host/lib/utils/paths.cpp +++ b/host/lib/utils/paths.cpp @@ -64,6 +64,7 @@ static std::vector get_env_paths(const std::string &var_name){ //convert to filesystem path, filter blank paths std::vector paths; + if (var_value.empty()) return paths; //FIXME boost tokenizer throws w/ blank strings on some platforms BOOST_FOREACH(const std::string &path_string, path_tokenizer(var_value)){ if (path_string.empty()) continue; paths.push_back(fs::system_complete(path_string)); -- cgit v1.2.3 From 94f5b0471bb49952ab6de87ebf90707ebb4b5810 Mon Sep 17 00:00:00 2001 From: Josh Blum Date: Wed, 26 Jan 2011 12:38:17 -0800 Subject: uhd: if_addrs check that iter->ifa_addr is NULL before continuing Happened when building UHD within the OpenWRT environment where ifa_addr can be null it seems! --- host/lib/transport/if_addrs.cpp | 1 + 1 file changed, 1 insertion(+) (limited to 'host/lib') diff --git a/host/lib/transport/if_addrs.cpp b/host/lib/transport/if_addrs.cpp index ad9a2325b..17cf8455b 100644 --- a/host/lib/transport/if_addrs.cpp +++ b/host/lib/transport/if_addrs.cpp @@ -42,6 +42,7 @@ std::vector uhd::transport::get_if_addrs(void){ if (getifaddrs(&ifap) == 0){ for (struct ifaddrs *iter = ifap; iter != NULL; iter = iter->ifa_next){ //ensure that the entries are valid + if (iter->ifa_addr == NULL) continue; if (iter->ifa_addr->sa_family != AF_INET) continue; if (iter->ifa_netmask->sa_family != AF_INET) continue; if (iter->ifa_broadaddr->sa_family != AF_INET) continue; -- cgit v1.2.3 From 5cd32555638301244add316a3d3788a9d44d6598 Mon Sep 17 00:00:00 2001 From: Josh Blum Date: Wed, 26 Jan 2011 12:41:48 -0800 Subject: udp_zero_copy_asio: removed the #ifdefed num frames and min buff size stuff basically, everything can share a common number of frames, and there is no min buff size, only resize if a size was specified, this simplifies the warnings and resize code --- host/lib/transport/udp_zero_copy_asio.cpp | 54 ++++++------------------------- 1 file changed, 9 insertions(+), 45 deletions(-) (limited to 'host/lib') diff --git a/host/lib/transport/udp_zero_copy_asio.cpp b/host/lib/transport/udp_zero_copy_asio.cpp index 3826dcd79..697e172cd 100644 --- a/host/lib/transport/udp_zero_copy_asio.cpp +++ b/host/lib/transport/udp_zero_copy_asio.cpp @@ -24,7 +24,7 @@ #include #include #include -#include +#include #include #include @@ -32,9 +32,6 @@ using namespace uhd; using namespace uhd::transport; namespace asio = boost::asio; -/*********************************************************************** - * Constants - **********************************************************************/ //Define this to the the boost async io calls to perform receive. //Otherwise, get_recv_buff uses a blocking receive with timeout. #define USE_ASIO_ASYNC_RECV @@ -43,32 +40,6 @@ namespace asio = boost::asio; //Otherwise, the commit callback uses a blocking send. //#define USE_ASIO_ASYNC_SEND -//By default, this buffer is sized insufficiently small. -//For peformance, this buffer should be 10s of megabytes. -static const size_t MIN_RECV_SOCK_BUFF_SIZE = size_t(10e3); - -//Large buffers cause more underflow at high rates. -//Perhaps this is due to the kernel scheduling, -//but may change with host-based flow control. -static const size_t MIN_SEND_SOCK_BUFF_SIZE = size_t(10e3); - -//The number of async frames to allocate for each send and recv: -//The non-async recv can have a very large number of recv frames -//because the CPU overhead is independent of the number of frames. -#ifdef USE_ASIO_ASYNC_RECV -static const size_t DEFAULT_NUM_RECV_FRAMES = 32; -#else -static const size_t DEFAULT_NUM_RECV_FRAMES = MIN_RECV_SOCK_BUFF_SIZE/udp_simple::mtu; -#endif - -//The non-async send only ever requires a single frame -//because the buffer will be committed before a new get. -#ifdef USE_ASIO_ASYNC_SEND -static const size_t DEFAULT_NUM_SEND_FRAMES = 32; -#else -static const size_t DEFAULT_NUM_SEND_FRAMES = MIN_SEND_SOCK_BUFF_SIZE/udp_simple::mtu; -#endif - //The number of service threads to spawn for async ASIO: //A single concurrent thread for io_service seems to be the fastest. //Threads are disabled when no async implementations are enabled. @@ -78,6 +49,9 @@ static const size_t CONCURRENCY_HINT = 1; static const size_t CONCURRENCY_HINT = 0; #endif +//A reasonable number of frames for send/recv and async/sync +static const size_t DEFAULT_NUM_FRAMES = 32; + /*********************************************************************** * Zero Copy UDP implementation with ASIO: * This is the portable zero copy implementation for systems @@ -95,9 +69,9 @@ public: const device_addr_t &hints ): _recv_frame_size(size_t(hints.cast("recv_frame_size", udp_simple::mtu))), - _num_recv_frames(size_t(hints.cast("num_recv_frames", DEFAULT_NUM_RECV_FRAMES))), + _num_recv_frames(size_t(hints.cast("num_recv_frames", DEFAULT_NUM_FRAMES))), _send_frame_size(size_t(hints.cast("send_frame_size", udp_simple::mtu))), - _num_send_frames(size_t(hints.cast("num_send_frames", DEFAULT_NUM_SEND_FRAMES))), + _num_send_frames(size_t(hints.cast("num_send_frames", DEFAULT_NUM_FRAMES))), _concurrency_hint(hints.cast("concurrency_hint", CONCURRENCY_HINT)), _io_service(_concurrency_hint) { @@ -325,16 +299,11 @@ template static void resize_buff_helper( const size_t target_size, const std::string &name ){ - size_t min_sock_buff_size = 0; - if (name == "recv") min_sock_buff_size = MIN_RECV_SOCK_BUFF_SIZE; - if (name == "send") min_sock_buff_size = MIN_SEND_SOCK_BUFF_SIZE; - min_sock_buff_size = std::max(min_sock_buff_size, target_size); - std::string help_message; #if defined(UHD_PLATFORM_LINUX) help_message = str(boost::format( "Please run: sudo sysctl -w net.core.%smem_max=%d\n" - ) % ((name == "recv")?"r":"w") % min_sock_buff_size); + ) % ((name == "recv")?"r":"w") % target_size); #endif /*defined(UHD_PLATFORM_LINUX)*/ //resize the buffer if size was provided @@ -348,14 +317,9 @@ template static void resize_buff_helper( "Current %s sock buff size: %d bytes" ) % name % actual_size << std::endl; if (actual_size < target_size) uhd::warning::post(str(boost::format( - "The %s buffer is smaller than the requested size.\n" - "The minimum requested buffer size is %d bytes.\n" + "The %s buffer could not be resized sufficiently.\n" "See the transport application notes on buffer resizing.\n%s" - ) % name % min_sock_buff_size % help_message)); - } - //otherwise, ensure that the buffer is at least the minimum size - else if (udp_trans->get_buff_size() < min_sock_buff_size){ - resize_buff_helper(udp_trans, min_sock_buff_size, name); + ) % name % help_message)); } } -- cgit v1.2.3 From e7df80534244adf49d1df6efcb316b96f6af994d Mon Sep 17 00:00:00 2001 From: Josh Blum Date: Thu, 27 Jan 2011 10:55:59 -0800 Subject: usrp-e100: revert compat number until its ready --- host/lib/usrp/usrp_e100/usrp_e100_impl.hpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'host/lib') diff --git a/host/lib/usrp/usrp_e100/usrp_e100_impl.hpp b/host/lib/usrp/usrp_e100/usrp_e100_impl.hpp index 864e82099..df8e5dc9f 100644 --- a/host/lib/usrp/usrp_e100/usrp_e100_impl.hpp +++ b/host/lib/usrp/usrp_e100/usrp_e100_impl.hpp @@ -30,7 +30,7 @@ #ifndef INCLUDED_USRP_E100_IMPL_HPP #define INCLUDED_USRP_E100_IMPL_HPP -static const boost::uint16_t USRP_E_COMPAT_NUM = 0x03; +static const boost::uint16_t USRP_E_COMPAT_NUM = 0x02; //make this 3 then the mainline fpga image gets fixed for embedded //! load an fpga image from a bin file into the usrp-e fpga extern void usrp_e100_load_fpga(const std::string &bin_file); -- cgit v1.2.3 From 8a62a20041605238320eae1f1a69221ef00b6bf1 Mon Sep 17 00:00:00 2001 From: Josh Blum Date: Thu, 27 Jan 2011 12:32:11 -0800 Subject: usrp2: dont set the time for slave devices, they always take from mimo cable --- host/lib/usrp/usrp2/mboard_impl.cpp | 3 +++ 1 file changed, 3 insertions(+) (limited to 'host/lib') diff --git a/host/lib/usrp/usrp2/mboard_impl.cpp b/host/lib/usrp/usrp2/mboard_impl.cpp index ee7ad99c5..3e7e8b01f 100644 --- a/host/lib/usrp/usrp2/mboard_impl.cpp +++ b/host/lib/usrp/usrp2/mboard_impl.cpp @@ -243,6 +243,9 @@ void usrp2_mboard_impl::update_clock_config(void){ } void usrp2_mboard_impl::set_time_spec(const time_spec_t &time_spec, bool now){ + //dont set the time for slave devices, they always take from mimo cable + if (not _mimo_clocking_mode_is_master) return; + //set the ticks _iface->poke32(_iface->regs.time64_ticks, time_spec.get_tick_count(get_master_clock_freq())); -- cgit v1.2.3 From e16445483e1505942b7b1ddcd9fc575532fd93ba Mon Sep 17 00:00:00 2001 From: Josh Blum Date: Thu, 27 Jan 2011 15:24:47 -0800 Subject: uhd: remove single usrp (leave a typedef), multi-usrp is a superset now --- host/docs/coding.rst | 34 +-- host/include/uhd/usrp/multi_usrp.hpp | 123 ++++++---- host/include/uhd/usrp/single_usrp.hpp | 435 +--------------------------------- host/lib/usrp/CMakeLists.txt | 1 - host/lib/usrp/multi_usrp.cpp | 13 +- host/lib/usrp/single_usrp.cpp | 339 -------------------------- 6 files changed, 93 insertions(+), 852 deletions(-) delete mode 100644 host/lib/usrp/single_usrp.cpp (limited to 'host/lib') diff --git a/host/docs/coding.rst b/host/docs/coding.rst index 7533445ea..ecca4e8b8 100644 --- a/host/docs/coding.rst +++ b/host/docs/coding.rst @@ -22,41 +22,11 @@ The device API provides ways to: See the documentation in *device.hpp* for reference. -^^^^^^^^^^^^^^^^^^^^^^^^^^^ -High-Level: The single usrp -^^^^^^^^^^^^^^^^^^^^^^^^^^^ -The goal of the single usrp API is to wrap high level functions around the device properties. -The single usrp provides a fat interface to access the most common properties. -The single usrp provides ways to: - -* Set and get daughterboard gains. -* Set and get daughterboard antennas. -* Set and get the streaming rates. -* Tune the DSPs and daughterboards. -* Issue stream commands. -* Set the clock configuration. -* Set the usrp time registers. -* Get the underlying device (as discussed above). - -See the documentation in *usrp/single_usrp.hpp* for reference. - ^^^^^^^^^^^^^^^^^^^^^^^^^^^ High-Level: The multi usrp ^^^^^^^^^^^^^^^^^^^^^^^^^^^ -The multi usrp API provides a wrapper around a device that represents several motherboards. -This API provides convenience calls just like the single usrp, -however the calls either work across all channels in the configuration, -or take a channel argument to specify which channel to configure. -The multi usrp provides ways to: - -* Set and get the sample rate across all channels. -* Issue a stream command across all channels. -* Set the time registers across all channels. -* Set and get individual daughterboard gains. -* Set and get individual daughterboard antennas. -* Tune individual DSPs and daughterboards. -* Get the underlying device (as discussed above). - +The Multi-USRP class provides a FAT interface to a single USRP with +one or more channels, or multiple USRPs in a homogeneous setup. See the documentation in *usrp/multi_usrp.hpp* for reference. ------------------------------------------------------------------------ diff --git a/host/include/uhd/usrp/multi_usrp.hpp b/host/include/uhd/usrp/multi_usrp.hpp index b603d4324..6e17c0ea8 100644 --- a/host/include/uhd/usrp/multi_usrp.hpp +++ b/host/include/uhd/usrp/multi_usrp.hpp @@ -34,19 +34,30 @@ namespace uhd{ namespace usrp{ /*! - * The multi-USRP device class: - * A multi-USRP facilitates ease-of-use for multiple USRP scenarios. - * The wrapper provides convenience functions to control the group - * of underlying devices as if they consisted of a single device. + * The Multi-USRP device class: * - * A few notes about a multi-USRP configuration: + * This class facilitates ease-of-use for most use-case scenarios. + * The wrapper provides convenience functions to tune the devices, + * set the dboard gains, antennas, filters, and other properties. + * This class can be used to interface with a single USRP with + * one or more channels, or multiple USRPs in a homogeneous setup. + * All members take an optional parameter for board number or channel number. + * In the single device, single channel case, these parameters can be unspecified. + * + * When using a single device with multiple channels: + * - Channel mapping is determined by the subdevice specifications + * - All channels share a common RX sample rate + * - All channels share a common TX sample rate + * + * When using multiple devices in a configuration: + * - Channel mapping is determined by the device address arguments * - All boards share a common RX sample rate * - All boards share a common TX sample rate * - All boards share a common RX subdevice specification size * - All boards share a common TX subdevice specification size * - All boards must have synchronized times (see the set_time_*() calls) * - * Example to setup channel mapping: + * Example to setup channel mapping for multiple devices: *
  *
  * //create a multi_usrp with two boards in the configuration
@@ -106,7 +117,7 @@ public:
      * \param mboard which motherboard to query
      * \return a string representing the name
      */
-    virtual std::string get_mboard_name(size_t mboard) = 0;
+    virtual std::string get_mboard_name(size_t mboard = 0) = 0;
 
     /*!
      * Get the current time in the usrp time registers.
@@ -120,6 +131,19 @@ public:
      */
     virtual time_spec_t get_time_last_pps(void) = 0;
 
+    /*!
+     * Sets the time registers on the usrp immediately.
+     *
+     * If only one MIMO master is present in your configuration, set_time_now is
+     * safe to use because the slave's time automatically follows the master's time.
+     * Otherwise, this call cannot set the time synchronously across multiple devices.
+     * Please use the set_time_next_pps or set_time_unknown_pps calls with a PPS signal.
+     *
+     * \param time_spec the time to latch into the usrp device
+     * \param mboard the motherboard index 0 to M-1
+     */
+    virtual void set_time_now(const time_spec_t &time_spec, size_t mboard = ALL_MBOARDS) = 0;
+
     /*!
      * Set the time registers on the usrp at the next pps tick.
      * The values will not be latched in until the pulse occurs.
@@ -162,6 +186,11 @@ public:
      * Issue a stream command to the usrp device.
      * This tells the usrp to send samples into the host.
      * See the documentation for stream_cmd_t for more info.
+     *
+     * With multiple devices, the first stream command in a chain of commands
+     * should have a time spec in the near future and stream_now = false;
+     * to ensure that the packets can be aligned by their time specs.
+     *
      * \param stream_cmd the stream command to issue
      */
     virtual void issue_stream_cmd(const stream_cmd_t &stream_cmd) = 0;
@@ -173,7 +202,7 @@ public:
      * \param clock_config the clock configuration to set
      * \param mboard which motherboard to set the config
      */
-    virtual void set_clock_config(const clock_config_t &clock_config, size_t mboard) = 0;
+    virtual void set_clock_config(const clock_config_t &clock_config, size_t mboard = ALL_MBOARDS) = 0;
 
     /*!
      * Get the number of USRP motherboards in this configuration.
@@ -191,14 +220,14 @@ public:
      * \param spec the new subdevice specification
      * \param mboard the motherboard index 0 to M-1
      */
-    virtual void set_rx_subdev_spec(const uhd::usrp::subdev_spec_t &spec, size_t mboard) = 0;
+    virtual void set_rx_subdev_spec(const uhd::usrp::subdev_spec_t &spec, size_t mboard = ALL_MBOARDS) = 0;
 
     /*!
      * Get the RX subdevice specification.
      * \param mboard the motherboard index 0 to M-1
      * \return the subdevice specification in use
      */
-    virtual uhd::usrp::subdev_spec_t get_rx_subdev_spec(size_t mboard) = 0;
+    virtual uhd::usrp::subdev_spec_t get_rx_subdev_spec(size_t mboard = 0) = 0;
 
     /*!
      * Get the number of RX channels in this configuration.
@@ -212,7 +241,7 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the subdevice name
      */
-    virtual std::string get_rx_subdev_name(size_t chan) = 0;
+    virtual std::string get_rx_subdev_name(size_t chan = 0) = 0;
 
     /*!
      * Set the RX sample rate across all channels.
@@ -241,14 +270,14 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the frequency in Hz
      */
-    virtual double get_rx_freq(size_t chan) = 0;
+    virtual double get_rx_freq(size_t chan = 0) = 0;
 
     /*!
      * Get the RX center frequency range.
      * \param chan the channel index 0 to N-1
      * \return a frequency range object
      */
-    virtual freq_range_t get_rx_freq_range(size_t chan) = 0;
+    virtual freq_range_t get_rx_freq_range(size_t chan = 0) = 0;
 
     /*!
      * Set the RX gain value for the specified gain element.
@@ -257,10 +286,10 @@ public:
      * \param name the name of the gain element
      * \param chan the channel index 0 to N-1
      */
-    virtual void set_rx_gain(double gain, const std::string &name, size_t chan) = 0;
+    virtual void set_rx_gain(double gain, const std::string &name, size_t chan = 0) = 0;
 
     //! A convenience wrapper for setting overall RX gain
-    void set_rx_gain(double gain, size_t chan){
+    void set_rx_gain(double gain, size_t chan = 0){
         return this->set_rx_gain(gain, ALL_GAINS, chan);
     }
 
@@ -271,10 +300,10 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the gain in dB
      */
-    virtual double get_rx_gain(const std::string &name, size_t chan) = 0;
+    virtual double get_rx_gain(const std::string &name, size_t chan = 0) = 0;
 
     //! A convenience wrapper for getting overall RX gain
-    double get_rx_gain(size_t chan){
+    double get_rx_gain(size_t chan = 0){
         return this->get_rx_gain(ALL_GAINS, chan);
     }
 
@@ -285,10 +314,10 @@ public:
      * \param chan the channel index 0 to N-1
      * \return a gain range object
      */
-    virtual gain_range_t get_rx_gain_range(const std::string &name, size_t chan) = 0;
+    virtual gain_range_t get_rx_gain_range(const std::string &name, size_t chan = 0) = 0;
 
     //! A convenience wrapper for getting overall RX gain range
-    gain_range_t get_rx_gain_range(size_t chan){
+    gain_range_t get_rx_gain_range(size_t chan = 0){
         return this->get_rx_gain_range(ALL_GAINS, chan);
     }
 
@@ -298,49 +327,49 @@ public:
      * \param chan the channel index 0 to N-1
      * \return a vector of gain element names
      */
-    virtual std::vector get_rx_gain_names(size_t chan) = 0;
+    virtual std::vector get_rx_gain_names(size_t chan = 0) = 0;
 
     /*!
      * Select the RX antenna on the subdevice.
      * \param ant the antenna name
      * \param chan the channel index 0 to N-1
      */
-    virtual void set_rx_antenna(const std::string &ant, size_t chan) = 0;
+    virtual void set_rx_antenna(const std::string &ant, size_t chan = 0) = 0;
 
     /*!
      * Get the selected RX antenna on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return the antenna name
      */
-    virtual std::string get_rx_antenna(size_t chan) = 0;
+    virtual std::string get_rx_antenna(size_t chan = 0) = 0;
 
     /*!
      * Get a list of possible RX antennas on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return a vector of antenna names
      */
-    virtual std::vector get_rx_antennas(size_t chan) = 0;
+    virtual std::vector get_rx_antennas(size_t chan = 0) = 0;
 
     /*!
      * Get the locked status of the LO on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return true for locked
      */
-    virtual bool get_rx_lo_locked(size_t chan) = 0;
+    virtual bool get_rx_lo_locked(size_t chan = 0) = 0;
 
     /*!
      * Set the RX bandwidth on the subdevice.
      * \param bandwidth the bandwidth in Hz
      * \param chan the channel index 0 to N-1
      */
-    virtual void set_rx_bandwidth(double bandwidth, size_t chan) = 0;
+    virtual void set_rx_bandwidth(double bandwidth, size_t chan = 0) = 0;
 
     /*!
      * Get the RX bandwidth on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return the bandwidth in Hz
      */
-    virtual double get_rx_bandwidth(size_t chan) = 0;
+    virtual double get_rx_bandwidth(size_t chan = 0) = 0;
 
     /*!
      * Read the RSSI value on the RX subdevice.
@@ -348,7 +377,7 @@ public:
      * \return the rssi in dB
      * \throw exception if RSSI readback not supported
      */
-    virtual double read_rssi(size_t chan) = 0;
+    virtual double read_rssi(size_t chan = 0) = 0;
 
     /*!
      * Get the dboard interface object for the RX subdevice.
@@ -357,7 +386,7 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the dboard interface sptr
      */
-    virtual dboard_iface::sptr get_rx_dboard_iface(size_t chan) = 0;
+    virtual dboard_iface::sptr get_rx_dboard_iface(size_t chan = 0) = 0;
 
     /*******************************************************************
      * TX methods
@@ -370,14 +399,14 @@ public:
      * \param spec the new subdevice specification
      * \param mboard the motherboard index 0 to M-1
      */
-    virtual void set_tx_subdev_spec(const uhd::usrp::subdev_spec_t &spec, size_t mboard) = 0;
+    virtual void set_tx_subdev_spec(const uhd::usrp::subdev_spec_t &spec, size_t mboard = ALL_MBOARDS) = 0;
 
     /*!
      * Get the TX subdevice specification.
      * \param mboard the motherboard index 0 to M-1
      * \return the subdevice specification in use
      */
-    virtual uhd::usrp::subdev_spec_t get_tx_subdev_spec(size_t mboard) = 0;
+    virtual uhd::usrp::subdev_spec_t get_tx_subdev_spec(size_t mboard = 0) = 0;
 
     /*!
      * Get the number of TX channels in this configuration.
@@ -391,7 +420,7 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the subdevice name
      */
-    virtual std::string get_tx_subdev_name(size_t chan) = 0;
+    virtual std::string get_tx_subdev_name(size_t chan = 0) = 0;
 
     /*!
      * Set the TX sample rate across all channels.
@@ -420,14 +449,14 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the frequency in Hz
      */
-    virtual double get_tx_freq(size_t chan) = 0;
+    virtual double get_tx_freq(size_t chan = 0) = 0;
 
     /*!
      * Get the TX center frequency range.
      * \param chan the channel index 0 to N-1
      * \return a frequency range object
      */
-    virtual freq_range_t get_tx_freq_range(size_t chan) = 0;
+    virtual freq_range_t get_tx_freq_range(size_t chan = 0) = 0;
 
     /*!
      * Set the TX gain value for the specified gain element.
@@ -436,10 +465,10 @@ public:
      * \param name the name of the gain element
      * \param chan the channel index 0 to N-1
      */
-    virtual void set_tx_gain(double gain, const std::string &name, size_t chan) = 0;
+    virtual void set_tx_gain(double gain, const std::string &name, size_t chan = 0) = 0;
 
     //! A convenience wrapper for setting overall TX gain
-    void set_tx_gain(double gain, size_t chan){
+    void set_tx_gain(double gain, size_t chan = 0){
         return this->set_tx_gain(gain, ALL_GAINS, chan);
     }
 
@@ -450,10 +479,10 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the gain in dB
      */
-    virtual double get_tx_gain(const std::string &name, size_t chan) = 0;
+    virtual double get_tx_gain(const std::string &name, size_t chan = 0) = 0;
 
     //! A convenience wrapper for getting overall TX gain
-    double get_tx_gain(size_t chan){
+    double get_tx_gain(size_t chan = 0){
         return this->get_tx_gain(ALL_GAINS, chan);
     }
 
@@ -464,10 +493,10 @@ public:
      * \param chan the channel index 0 to N-1
      * \return a gain range object
      */
-    virtual gain_range_t get_tx_gain_range(const std::string &name, size_t chan) = 0;
+    virtual gain_range_t get_tx_gain_range(const std::string &name, size_t chan = 0) = 0;
 
     //! A convenience wrapper for getting overall TX gain range
-    gain_range_t get_tx_gain_range(size_t chan){
+    gain_range_t get_tx_gain_range(size_t chan = 0){
         return this->get_tx_gain_range(ALL_GAINS, chan);
     }
 
@@ -477,49 +506,49 @@ public:
      * \param chan the channel index 0 to N-1
      * \return a vector of gain element names
      */
-    virtual std::vector get_tx_gain_names(size_t chan) = 0;
+    virtual std::vector get_tx_gain_names(size_t chan = 0) = 0;
 
     /*!
      * Select the TX antenna on the subdevice.
      * \param ant the antenna name
      * \param chan the channel index 0 to N-1
      */
-    virtual void set_tx_antenna(const std::string &ant, size_t chan) = 0;
+    virtual void set_tx_antenna(const std::string &ant, size_t chan = 0) = 0;
 
     /*!
      * Get the selected TX antenna on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return the antenna name
      */
-    virtual std::string get_tx_antenna(size_t chan) = 0;
+    virtual std::string get_tx_antenna(size_t chan = 0) = 0;
 
     /*!
      * Get a list of possible TX antennas on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return a vector of antenna names
      */
-    virtual std::vector get_tx_antennas(size_t chan) = 0;
+    virtual std::vector get_tx_antennas(size_t chan = 0) = 0;
 
     /*!
      * Get the locked status of the LO on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return true for locked
      */
-    virtual bool get_tx_lo_locked(size_t chan) = 0;
+    virtual bool get_tx_lo_locked(size_t chan = 0) = 0;
 
     /*!
      * Set the TX bandwidth on the subdevice.
      * \param bandwidth the bandwidth in Hz
      * \param chan the channel index 0 to N-1
      */
-    virtual void set_tx_bandwidth(double bandwidth, size_t chan) = 0;
+    virtual void set_tx_bandwidth(double bandwidth, size_t chan = 0) = 0;
 
     /*!
      * Get the TX bandwidth on the subdevice.
      * \param chan the channel index 0 to N-1
      * \return the bandwidth in Hz
      */
-    virtual double get_tx_bandwidth(size_t chan) = 0;
+    virtual double get_tx_bandwidth(size_t chan = 0) = 0;
 
     /*!
      * Get the dboard interface object for the TX subdevice.
@@ -528,7 +557,7 @@ public:
      * \param chan the channel index 0 to N-1
      * \return the dboard interface sptr
      */
-    virtual dboard_iface::sptr get_tx_dboard_iface(size_t chan) = 0;
+    virtual dboard_iface::sptr get_tx_dboard_iface(size_t chan = 0) = 0;
 };
 
 }}
diff --git a/host/include/uhd/usrp/single_usrp.hpp b/host/include/uhd/usrp/single_usrp.hpp
index d80999300..0520db162 100644
--- a/host/include/uhd/usrp/single_usrp.hpp
+++ b/host/include/uhd/usrp/single_usrp.hpp
@@ -18,441 +18,12 @@
 #ifndef INCLUDED_UHD_USRP_SINGLE_USRP_HPP
 #define INCLUDED_UHD_USRP_SINGLE_USRP_HPP
 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
+#include 
 
 namespace uhd{ namespace usrp{
 
-/*!
- * The single-USRP device class:
- * A single-USRP facilitates ease-of-use for most use-case scenarios.
- * The wrapper provides convenience functions to tune the devices
- * as well as to set the dboard gains, antennas, and other properties.
- * This wrapper supports multi-channel configurations per motherboard.
- */
-class UHD_API single_usrp : boost::noncopyable{
-public:
-    typedef boost::shared_ptr sptr;
-
-    //! A wildcard gain element name
-    static const std::string ALL_GAINS;
-
-    /*!
-     * Make a new single usrp from the device address.
-     * \param dev_addr the device address
-     * \return a new single usrp object
-     */
-    static sptr make(const device_addr_t &dev_addr);
-
-    /*!
-     * Get the underlying device object.
-     * This is needed to get access to the streaming API and properties.
-     * \return the device object within this single usrp
-     */
-    virtual device::sptr get_device(void) = 0;
-
-    /*******************************************************************
-     * Mboard methods
-     ******************************************************************/
-    /*!
-     * Get a printable summary for this USRP configuration.
-     * \return a printable string
-     */
-    virtual std::string get_pp_string(void) = 0;
-
-    /*!
-     * Get canonical name for this USRP motherboard.
-     * \return a string representing the name
-     */
-    virtual std::string get_mboard_name(void) = 0;
-
-    /*!
-     * Get the current time in the usrp time registers.
-     * \return a timespec representing current usrp time
-     */
-    virtual time_spec_t get_time_now(void) = 0;
-
-    /*!
-     * Get the time when the last pps pulse occured.
-     * \return a timespec representing the last pps
-     */
-    virtual time_spec_t get_time_last_pps(void) = 0;
-
-    /*!
-     * Sets the time registers on the usrp immediately.
-     * \param time_spec the time to latch into the usrp device
-     */
-    virtual void set_time_now(const time_spec_t &time_spec) = 0;
-
-    /*!
-     * Set the time registers on the usrp at the next pps tick.
-     * The values will not be latched in until the pulse occurs.
-     * It is recommended that the user sleep(1) after calling to ensure
-     * that the time registers will be in a known state prior to use.
-     *
-     * Note: Because this call sets the time on the "next" pps,
-     * the seconds in the time spec should be current seconds + 1.
-     *
-     * \param time_spec the time to latch into the usrp device
-     */
-    virtual void set_time_next_pps(const time_spec_t &time_spec) = 0;
-
-    /*!
-     * Issue a stream command to the usrp device.
-     * This tells the usrp to send samples into the host.
-     * See the documentation for stream_cmd_t for more info.
-     * \param stream_cmd the stream command to issue
-     */
-    virtual void issue_stream_cmd(const stream_cmd_t &stream_cmd) = 0;
-
-    /*!
-     * Set the clock configuration for the usrp device.
-     * This tells the usrp how to get a 10Mhz reference and PPS clock.
-     * See the documentation for clock_config_t for more info.
-     * \param clock_config the clock configuration to set
-     */
-    virtual void set_clock_config(const clock_config_t &clock_config) = 0;
-
-    /*******************************************************************
-     * RX methods
-     ******************************************************************/
-    /*!
-     * Set the RX subdevice specification:
-     * The subdev spec maps a physical part of a daughter-board to a channel number.
-     * Set the subdev spec before calling into any methods with a channel number.
-     * \param spec the new subdevice specification
-     */
-    virtual void set_rx_subdev_spec(const uhd::usrp::subdev_spec_t &spec) = 0;
-
-    /*!
-     * Get the RX subdevice specification.
-     * \return the subdevice specification in use
-     */
-    virtual uhd::usrp::subdev_spec_t get_rx_subdev_spec(void) = 0;
-
-    /*!
-     * Get the name of the RX subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return the subdevice name
-     */
-    virtual std::string get_rx_subdev_name(size_t chan = 0) = 0;
-
-    /*!
-     * Set the RX sample rate across all channels.
-     * \param rate the rate in Sps
-     */
-    virtual void set_rx_rate(double rate) = 0;
-
-    /*!
-     * Gets the RX sample rate for all channels.
-     * \return the rate in Sps
-     */
-    virtual double get_rx_rate(void) = 0;
-
-    /*!
-     * Set the RX center frequency.
-     * \param tune_request tune request instructions
-     * \param chan the channel index 0 to N-1
-     * \return a tune result object
-     */
-    virtual tune_result_t set_rx_freq(
-        const tune_request_t &tune_request, size_t chan = 0
-    ) = 0;
-
-    /*!
-     * Get the RX center frequency.
-     * \param chan the channel index 0 to N-1
-     * \return the frequency in Hz
-     */
-    virtual double get_rx_freq(size_t chan = 0) = 0;
-
-    /*!
-     * Get the RX center frequency range.
-     * \param chan the channel index 0 to N-1
-     * \return a frequency range object
-     */
-    virtual freq_range_t get_rx_freq_range(size_t chan = 0) = 0;
-
-    /*!
-     * Set the RX gain value for the specified gain element.
-     * For an empty name, distribute across all gain elements.
-     * \param gain the gain in dB
-     * \param name the name of the gain element
-     * \param chan the channel index 0 to N-1
-     */
-    virtual void set_rx_gain(double gain, const std::string &name, size_t chan = 0) = 0;
-
-    //! A convenience wrapper for setting overall RX gain
-    void set_rx_gain(double gain, size_t chan = 0){
-        return this->set_rx_gain(gain, ALL_GAINS, chan);
-    }
-
-    /*!
-     * Get the RX gain value for the specified gain element.
-     * For an empty name, sum across all gain elements.
-     * \param name the name of the gain element
-     * \param chan the channel index 0 to N-1
-     * \return the gain in dB
-     */
-    virtual double get_rx_gain(const std::string &name, size_t chan = 0) = 0;
-
-    //! A convenience wrapper for getting overall RX gain
-    double get_rx_gain(size_t chan = 0){
-        return this->get_rx_gain(ALL_GAINS, chan);
-    }
-
-    /*!
-     * Get the RX gain range for the specified gain element.
-     * For an empty name, calculate the overall gain range.
-     * \param name the name of the gain element
-     * \param chan the channel index 0 to N-1
-     * \return a gain range object
-     */
-    virtual gain_range_t get_rx_gain_range(const std::string &name, size_t chan = 0) = 0;
-
-    //! A convenience wrapper for getting overall RX gain range
-    gain_range_t get_rx_gain_range(size_t chan = 0){
-        return this->get_rx_gain_range(ALL_GAINS, chan);
-    }
-
-    /*!
-     * Get the names of the gain elements in the RX chain.
-     * Gain elements are ordered from antenna to FPGA.
-     * \param chan the channel index 0 to N-1
-     * \return a vector of gain element names
-     */
-    virtual std::vector get_rx_gain_names(size_t chan = 0) = 0;
-
-    /*!
-     * Select the RX antenna on the subdevice.
-     * \param ant the antenna name
-     * \param chan the channel index 0 to N-1
-     */
-    virtual void set_rx_antenna(const std::string &ant, size_t chan = 0) = 0;
-
-    /*!
-     * Get the selected RX antenna on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return the antenna name
-     */
-    virtual std::string get_rx_antenna(size_t chan = 0) = 0;
-
-    /*!
-     * Get a list of possible RX antennas on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return a vector of antenna names
-     */
-    virtual std::vector get_rx_antennas(size_t chan = 0) = 0;
-
-    /*!
-     * Get the locked status of the LO on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return true for locked
-     */
-    virtual bool get_rx_lo_locked(size_t chan = 0) = 0;
-
-    /*!
-     * Set the RX bandwidth on the subdevice.
-     * \param bandwidth the bandwidth in Hz
-     * \param chan the channel index 0 to N-1
-     */
-    virtual void set_rx_bandwidth(double bandwidth, size_t chan = 0) = 0;
-
-    /*!
-     * Get the RX bandwidth on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return the bandwidth in Hz
-     */
-    virtual double get_rx_bandwidth(size_t chan = 0) = 0;
-
-    /*!
-     * Read the RSSI value on the RX subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return the rssi in dB
-     * \throw exception if RSSI readback not supported
-     */
-    virtual double read_rssi(size_t chan = 0) = 0;
-
-    /*!
-     * Get the dboard interface object for the RX subdevice.
-     * The dboard interface gives access to GPIOs, SPI, I2C, low-speed ADC and DAC.
-     * Use at your own risk!
-     * \param chan the channel index 0 to N-1
-     * \return the dboard interface sptr
-     */
-    virtual dboard_iface::sptr get_rx_dboard_iface(size_t chan = 0) = 0;
-
-    /*******************************************************************
-     * TX methods
-     ******************************************************************/
-    /*!
-     * Set the TX subdevice specification:
-     * The subdev spec maps a physical part of a daughter-board to a channel number.
-     * Set the subdev spec before calling into any methods with a channel number.
-     * \param spec the new subdevice specification
-     */
-    virtual void set_tx_subdev_spec(const uhd::usrp::subdev_spec_t &spec) = 0;
-
-    /*!
-     * Get the TX subdevice specification.
-     * \return the subdevice specification in use
-     */
-    virtual uhd::usrp::subdev_spec_t get_tx_subdev_spec(void) = 0;
-
-    /*!
-     * Get the name of the TX subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return the subdevice name
-     */
-    virtual std::string get_tx_subdev_name(size_t chan = 0) = 0;
-
-    /*!
-     * Set the TX sample rate across all channels.
-     * \param rate the rate in Sps
-     */
-    virtual void set_tx_rate(double rate) = 0;
-
-    /*!
-     * Gets the TX sample rate for all channels.
-     * \return the rate in Sps
-     */
-    virtual double get_tx_rate(void) = 0;
-
-    /*!
-     * Set the TX center frequency.
-     * \param tune_request tune request instructions
-     * \param chan the channel index 0 to N-1
-     * \return a tune result object
-     */
-    virtual tune_result_t set_tx_freq(
-        const tune_request_t &tune_request, size_t chan = 0
-    ) = 0;
-
-    /*!
-     * Get the TX center frequency.
-     * \param chan the channel index 0 to N-1
-     * \return the frequency in Hz
-     */
-    virtual double get_tx_freq(size_t chan = 0) = 0;
-
-    /*!
-     * Get the TX center frequency range.
-     * \param chan the channel index 0 to N-1
-     * \return a frequency range object
-     */
-    virtual freq_range_t get_tx_freq_range(size_t chan = 0) = 0;
-
-    /*!
-     * Set the TX gain value for the specified gain element.
-     * For an empty name, distribute across all gain elements.
-     * \param gain the gain in dB
-     * \param name the name of the gain element
-     * \param chan the channel index 0 to N-1
-     */
-    virtual void set_tx_gain(double gain, const std::string &name, size_t chan = 0) = 0;
-
-    //! A convenience wrapper for setting overall TX gain
-    void set_tx_gain(double gain, size_t chan = 0){
-        return this->set_tx_gain(gain, ALL_GAINS, chan);
-    }
-
-    /*!
-     * Get the TX gain value for the specified gain element.
-     * For an empty name, sum across all gain elements.
-     * \param name the name of the gain element
-     * \param chan the channel index 0 to N-1
-     * \return the gain in dB
-     */
-    virtual double get_tx_gain(const std::string &name, size_t chan = 0) = 0;
-
-    //! A convenience wrapper for getting overall TX gain
-    double get_tx_gain(size_t chan = 0){
-        return this->get_tx_gain(ALL_GAINS, chan);
-    }
-
-    /*!
-     * Get the TX gain range for the specified gain element.
-     * For an empty name, calculate the overall gain range.
-     * \param name the name of the gain element
-     * \param chan the channel index 0 to N-1
-     * \return a gain range object
-     */
-    virtual gain_range_t get_tx_gain_range(const std::string &name, size_t chan = 0) = 0;
-
-    //! A convenience wrapper for getting overall TX gain range
-    gain_range_t get_tx_gain_range(size_t chan = 0){
-        return this->get_tx_gain_range(ALL_GAINS, chan);
-    }
-
-    /*!
-     * Get the names of the gain elements in the TX chain.
-     * Gain elements are ordered from antenna to FPGA.
-     * \param chan the channel index 0 to N-1
-     * \return a vector of gain element names
-     */
-    virtual std::vector get_tx_gain_names(size_t chan = 0) = 0;
-
-    /*!
-     * Select the TX antenna on the subdevice.
-     * \param ant the antenna name
-     * \param chan the channel index 0 to N-1
-     */
-    virtual void set_tx_antenna(const std::string &ant, size_t chan = 0) = 0;
-
-    /*!
-     * Get the selected TX antenna on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return the antenna name
-     */
-    virtual std::string get_tx_antenna(size_t chan = 0) = 0;
-
-    /*!
-     * Get a list of possible TX antennas on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return a vector of antenna names
-     */
-    virtual std::vector get_tx_antennas(size_t chan = 0) = 0;
-
-    /*!
-     * Get the locked status of the LO on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return true for locked
-     */
-    virtual bool get_tx_lo_locked(size_t chan = 0) = 0;
-
-    /*!
-     * Set the TX bandwidth on the subdevice.
-     * \param bandwidth the bandwidth in Hz
-     * \param chan the channel index 0 to N-1
-     */
-    virtual void set_tx_bandwidth(double bandwidth, size_t chan = 0) = 0;
-
-    /*!
-     * Get the TX bandwidth on the subdevice.
-     * \param chan the channel index 0 to N-1
-     * \return the bandwidth in Hz
-     */
-    virtual double get_tx_bandwidth(size_t chan = 0) = 0;
-
-    /*!
-     * Get the dboard interface object for the TX subdevice.
-     * The dboard interface gives access to GPIOs, SPI, I2C, low-speed ADC and DAC.
-     * Use at your own risk!
-     * \param chan the channel index 0 to N-1
-     * \return the dboard interface sptr
-     */
-    virtual dboard_iface::sptr get_tx_dboard_iface(size_t chan = 0) = 0;
-};
+    //! Multi-USRP is a superset of Single-USRP
+    typedef multi_usrp single_usrp;
 
 }}
 
diff --git a/host/lib/usrp/CMakeLists.txt b/host/lib/usrp/CMakeLists.txt
index bd25aec2b..97a54a798 100644
--- a/host/lib/usrp/CMakeLists.txt
+++ b/host/lib/usrp/CMakeLists.txt
@@ -29,7 +29,6 @@ LIBUHD_APPEND_SOURCES(
     ${CMAKE_CURRENT_SOURCE_DIR}/mboard_eeprom.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/misc_utils.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/multi_usrp.cpp
-    ${CMAKE_CURRENT_SOURCE_DIR}/single_usrp.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/subdev_spec.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/tune_helper.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/wrapper_utils.hpp
diff --git a/host/lib/usrp/multi_usrp.cpp b/host/lib/usrp/multi_usrp.cpp
index 48eec28c1..17458496b 100644
--- a/host/lib/usrp/multi_usrp.cpp
+++ b/host/lib/usrp/multi_usrp.cpp
@@ -55,9 +55,10 @@ public:
      ******************************************************************/
     std::string get_pp_string(void){
         std::string buff = str(boost::format(
-            "Multi USRP:\n"
+            "%s USRP:\n"
             "  Device: %s\n"
         )
+            % ((get_num_mboards() > 1)? "Multi" : "Single")
             % (*_dev)[DEVICE_PROP_NAME].as()
         );
         for (size_t m = 0; m < get_num_mboards(); m++){
@@ -121,6 +122,16 @@ public:
         return _mboard(0)[MBOARD_PROP_TIME_PPS].as();
     }
 
+    void set_time_now(const time_spec_t &time_spec, size_t mboard){
+        if (mboard != ALL_MBOARDS){
+            _mboard(mboard)[MBOARD_PROP_TIME_NOW] = time_spec;
+            return;
+        }
+        for (size_t m = 0; m < get_num_mboards(); m++){
+            set_time_now(time_spec, m);
+        }
+    }
+
     void set_time_next_pps(const time_spec_t &time_spec){
         for (size_t m = 0; m < get_num_mboards(); m++){
             _mboard(m)[MBOARD_PROP_TIME_PPS] = time_spec;
diff --git a/host/lib/usrp/single_usrp.cpp b/host/lib/usrp/single_usrp.cpp
deleted file mode 100644
index c37449c5f..000000000
--- a/host/lib/usrp/single_usrp.cpp
+++ /dev/null
@@ -1,339 +0,0 @@
-//
-// Copyright 2010-2011 Ettus Research LLC
-//
-// This program is free software: you can redistribute it and/or modify
-// it under the terms of the GNU General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// This program is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-// GNU General Public License for more details.
-//
-// You should have received a copy of the GNU General Public License
-// along with this program.  If not, see .
-//
-
-#include "wrapper_utils.hpp"
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-
-using namespace uhd;
-using namespace uhd::usrp;
-
-const std::string single_usrp::ALL_GAINS = "";
-
-/***********************************************************************
- * Simple USRP Implementation
- **********************************************************************/
-class single_usrp_impl : public single_usrp{
-public:
-    single_usrp_impl(const device_addr_t &addr){
-        _dev = device::make(addr);
-    }
-
-    device::sptr get_device(void){
-        return _dev;
-    }
-
-    /*******************************************************************
-     * Mboard methods
-     ******************************************************************/
-    std::string get_pp_string(void){
-        std::string buff = str(boost::format(
-            "Single USRP:\n"
-            "  Device: %s\n"
-            "  Mboard: %s\n"
-        )
-            % (*_dev)[DEVICE_PROP_NAME].as()
-            % _mboard()[MBOARD_PROP_NAME].as()
-        );
-
-        //----------- rx side of life ----------------------------------
-        buff += str(boost::format(
-            "  RX DSP: %s\n"
-        )
-            % _rx_dsp()[DSP_PROP_NAME].as()
-        );
-        for (size_t chan = 0; chan < this->get_rx_subdev_spec().size(); chan++){
-            buff += str(boost::format(
-                "  RX Channel: %u\n"
-                "    RX Dboard: %s\n"
-                "    RX Subdev: %s\n"
-            ) % chan
-                % _rx_dboard(chan)[DBOARD_PROP_NAME].as()
-                % _rx_subdev(chan)[SUBDEV_PROP_NAME].as()
-            );
-        }
-
-        //----------- tx side of life ----------------------------------
-        buff += str(boost::format(
-            "  TX DSP: %s\n"
-        )
-            % _tx_dsp()[DSP_PROP_NAME].as()
-        );
-        for (size_t chan = 0; chan < this->get_tx_subdev_spec().size(); chan++){
-            buff += str(boost::format(
-                "  TX Channel: %u\n"
-                "    TX Dboard: %s\n"
-                "    TX Subdev: %s\n"
-            ) % chan
-                % _tx_dboard(chan)[DBOARD_PROP_NAME].as()
-                % _tx_subdev(chan)[SUBDEV_PROP_NAME].as()
-            );
-        }
-
-        return buff;
-    }
-
-    std::string get_mboard_name(void){
-        return _mboard()[MBOARD_PROP_NAME].as();
-    }
-
-    time_spec_t get_time_now(void){
-        return _mboard()[MBOARD_PROP_TIME_NOW].as();
-    }
-
-    time_spec_t get_time_last_pps(void){
-        return _mboard()[MBOARD_PROP_TIME_PPS].as();
-    }
-
-    void set_time_now(const time_spec_t &time_spec){
-        _mboard()[MBOARD_PROP_TIME_NOW] = time_spec;
-    }
-
-    void set_time_next_pps(const time_spec_t &time_spec){
-        _mboard()[MBOARD_PROP_TIME_PPS] = time_spec;
-    }
-
-    void issue_stream_cmd(const stream_cmd_t &stream_cmd){
-        _mboard()[MBOARD_PROP_STREAM_CMD] = stream_cmd;
-    }
-
-    void set_clock_config(const clock_config_t &clock_config){
-        _mboard()[MBOARD_PROP_CLOCK_CONFIG] = clock_config;
-    }
-
-    /*******************************************************************
-     * RX methods
-     ******************************************************************/
-    void set_rx_subdev_spec(const subdev_spec_t &spec){
-        _mboard()[MBOARD_PROP_RX_SUBDEV_SPEC] = spec;
-    }
-
-    subdev_spec_t get_rx_subdev_spec(void){
-        return _mboard()[MBOARD_PROP_RX_SUBDEV_SPEC].as();
-    }
-
-    std::string get_rx_subdev_name(size_t chan){
-        return _rx_subdev(chan)[SUBDEV_PROP_NAME].as();
-    }
-
-    void set_rx_rate(double rate){
-        _rx_dsp()[DSP_PROP_HOST_RATE] = rate;
-        do_samp_rate_warning_message(rate, get_rx_rate(), "RX");
-    }
-
-    double get_rx_rate(void){
-        return _rx_dsp()[DSP_PROP_HOST_RATE].as();
-    }
-
-    tune_result_t set_rx_freq(const tune_request_t &tune_request, size_t chan){
-        tune_result_t r = tune_rx_subdev_and_dsp(_rx_subdev(chan), _rx_dsp(), chan, tune_request);
-        do_tune_freq_warning_message(tune_request.target_freq, get_rx_freq(chan), "RX");
-        return r;
-    }
-
-    double get_rx_freq(size_t chan){
-        return derive_freq_from_rx_subdev_and_dsp(_rx_subdev(chan), _rx_dsp(), chan);
-    }
-
-    freq_range_t get_rx_freq_range(size_t chan){
-        return add_dsp_shift(_rx_subdev(chan)[SUBDEV_PROP_FREQ_RANGE].as(), _rx_dsp());
-    }
-
-    void set_rx_gain(double gain, const std::string &name, size_t chan){
-        return _rx_gain_group(chan)->set_value(gain, name);
-    }
-
-    double get_rx_gain(const std::string &name, size_t chan){
-        return _rx_gain_group(chan)->get_value(name);
-    }
-
-    gain_range_t get_rx_gain_range(const std::string &name, size_t chan){
-        return _rx_gain_group(chan)->get_range(name);
-    }
-
-    std::vector get_rx_gain_names(size_t chan){
-        return _rx_gain_group(chan)->get_names();
-    }
-
-    void set_rx_antenna(const std::string &ant, size_t chan){
-        _rx_subdev(chan)[SUBDEV_PROP_ANTENNA] = ant;
-    }
-
-    std::string get_rx_antenna(size_t chan){
-        return _rx_subdev(chan)[SUBDEV_PROP_ANTENNA].as();
-    }
-
-    std::vector get_rx_antennas(size_t chan){
-        return _rx_subdev(chan)[SUBDEV_PROP_ANTENNA_NAMES].as();
-    }
-
-    bool get_rx_lo_locked(size_t chan){
-        return _rx_subdev(chan)[SUBDEV_PROP_LO_LOCKED].as();
-    }
-
-    void set_rx_bandwidth(double bandwidth, size_t chan){
-        _rx_subdev(chan)[SUBDEV_PROP_BANDWIDTH] = bandwidth;
-    }
-
-    double get_rx_bandwidth(size_t chan){
-        return _rx_subdev(chan)[SUBDEV_PROP_BANDWIDTH].as();
-    }
-
-    double read_rssi(size_t chan){
-        return _rx_subdev(chan)[SUBDEV_PROP_RSSI].as();
-    }
-
-    dboard_iface::sptr get_rx_dboard_iface(size_t chan){
-        return _rx_dboard(chan)[DBOARD_PROP_DBOARD_IFACE].as();
-    }
-
-    /*******************************************************************
-     * TX methods
-     ******************************************************************/
-    void set_tx_subdev_spec(const subdev_spec_t &spec){
-        _mboard()[MBOARD_PROP_TX_SUBDEV_SPEC] = spec;
-    }
-
-    subdev_spec_t get_tx_subdev_spec(void){
-        return _mboard()[MBOARD_PROP_TX_SUBDEV_SPEC].as();
-    }
-
-    std::string get_tx_subdev_name(size_t chan){
-        return _tx_subdev(chan)[SUBDEV_PROP_NAME].as();
-    }
-
-    void set_tx_rate(double rate){
-        _tx_dsp()[DSP_PROP_HOST_RATE] = rate;
-        do_samp_rate_warning_message(rate, get_tx_rate(), "TX");
-    }
-
-    double get_tx_rate(void){
-        return _tx_dsp()[DSP_PROP_HOST_RATE].as();
-    }
-
-    tune_result_t set_tx_freq(const tune_request_t &tune_request, size_t chan){
-        tune_result_t r = tune_tx_subdev_and_dsp(_tx_subdev(chan), _tx_dsp(), chan, tune_request);
-        do_tune_freq_warning_message(tune_request.target_freq, get_tx_freq(chan), "TX");
-        return r;
-    }
-
-    double get_tx_freq(size_t chan){
-        return derive_freq_from_tx_subdev_and_dsp(_tx_subdev(chan), _tx_dsp(), chan);
-    }
-
-    freq_range_t get_tx_freq_range(size_t chan){
-        return add_dsp_shift(_tx_subdev(chan)[SUBDEV_PROP_FREQ_RANGE].as(), _tx_dsp());
-    }
-
-    void set_tx_gain(double gain, const std::string &name, size_t chan){
-        return _tx_gain_group(chan)->set_value(gain, name);
-    }
-
-    double get_tx_gain(const std::string &name, size_t chan){
-        return _tx_gain_group(chan)->get_value(name);
-    }
-
-    gain_range_t get_tx_gain_range(const std::string &name, size_t chan){
-        return _tx_gain_group(chan)->get_range(name);
-    }
-
-    std::vector get_tx_gain_names(size_t chan){
-        return _tx_gain_group(chan)->get_names();
-    }
-
-    void set_tx_antenna(const std::string &ant, size_t chan){
-        _tx_subdev(chan)[SUBDEV_PROP_ANTENNA] = ant;
-    }
-
-    std::string get_tx_antenna(size_t chan){
-        return _tx_subdev(chan)[SUBDEV_PROP_ANTENNA].as();
-    }
-
-    std::vector get_tx_antennas(size_t chan){
-        return _tx_subdev(chan)[SUBDEV_PROP_ANTENNA_NAMES].as();
-    }
-
-    bool get_tx_lo_locked(size_t chan){
-        return _tx_subdev(chan)[SUBDEV_PROP_LO_LOCKED].as();
-    }
-
-    void set_tx_bandwidth(double bandwidth, size_t chan){
-        _tx_subdev(chan)[SUBDEV_PROP_BANDWIDTH] = bandwidth;
-    }
-
-    double get_tx_bandwidth(size_t chan){
-        return _tx_subdev(chan)[SUBDEV_PROP_BANDWIDTH].as();
-    }
-
-    dboard_iface::sptr get_tx_dboard_iface(size_t chan){
-        return _tx_dboard(chan)[DBOARD_PROP_DBOARD_IFACE].as();
-    }
-
-private:
-    device::sptr _dev;
-    wax::obj _mboard(void){
-        return (*_dev)[DEVICE_PROP_MBOARD];
-    }
-    wax::obj _rx_dsp(void){
-        return _mboard()[MBOARD_PROP_RX_DSP];
-    }
-    wax::obj _tx_dsp(void){
-        return _mboard()[MBOARD_PROP_TX_DSP];
-    }
-    wax::obj _rx_dboard(size_t chan){
-        std::string db_name = this->get_rx_subdev_spec().at(chan).db_name;
-        return _mboard()[named_prop_t(MBOARD_PROP_RX_DBOARD, db_name)];
-    }
-    wax::obj _tx_dboard(size_t chan){
-        std::string db_name = this->get_tx_subdev_spec().at(chan).db_name;
-        return _mboard()[named_prop_t(MBOARD_PROP_TX_DBOARD, db_name)];
-    }
-    wax::obj _rx_subdev(size_t chan){
-        std::string sd_name = this->get_rx_subdev_spec().at(chan).sd_name;
-        return _rx_dboard(chan)[named_prop_t(DBOARD_PROP_SUBDEV, sd_name)];
-    }
-    wax::obj _tx_subdev(size_t chan){
-        std::string sd_name = this->get_tx_subdev_spec().at(chan).sd_name;
-        return _tx_dboard(chan)[named_prop_t(DBOARD_PROP_SUBDEV, sd_name)];
-    }
-    gain_group::sptr _rx_gain_group(size_t chan){
-        std::string sd_name = this->get_rx_subdev_spec().at(chan).sd_name;
-        return _rx_dboard(chan)[named_prop_t(DBOARD_PROP_GAIN_GROUP, sd_name)].as();
-    }
-    gain_group::sptr _tx_gain_group(size_t chan){
-        std::string sd_name = this->get_tx_subdev_spec().at(chan).sd_name;
-        return _tx_dboard(chan)[named_prop_t(DBOARD_PROP_GAIN_GROUP, sd_name)].as();
-    }
-};
-
-/***********************************************************************
- * The Make Function
- **********************************************************************/
-single_usrp::sptr single_usrp::make(const device_addr_t &dev_addr){
-    return sptr(new single_usrp_impl(dev_addr));
-}
-- 
cgit v1.2.3


From 625d5605dd157f9cf6f1d96e60a4d8d051817aef Mon Sep 17 00:00:00 2001
From: Josh Blum 
Date: Mon, 31 Jan 2011 16:29:12 -0800
Subject: usrp: added set and get master clock rates to usrp API

implemented set and get rates in usrp1 (its all soft)

implemented set and get rates in usrp2 (only 100MHz)
---
 host/include/uhd/usrp/mboard_props.hpp |  1 +
 host/include/uhd/usrp/multi_usrp.hpp   | 20 ++++++++++++++++++++
 host/lib/usrp/multi_usrp.cpp           | 14 ++++++++++++++
 host/lib/usrp/usrp1/clock_ctrl.cpp     | 21 +++++++++++----------
 host/lib/usrp/usrp1/clock_ctrl.hpp     |  7 +++++++
 host/lib/usrp/usrp1/mboard_impl.cpp    |  8 ++++++++
 host/lib/usrp/usrp2/mboard_impl.cpp    |  8 ++++++++
 7 files changed, 69 insertions(+), 10 deletions(-)

(limited to 'host/lib')

diff --git a/host/include/uhd/usrp/mboard_props.hpp b/host/include/uhd/usrp/mboard_props.hpp
index c82bfc21a..d04ad012c 100644
--- a/host/include/uhd/usrp/mboard_props.hpp
+++ b/host/include/uhd/usrp/mboard_props.hpp
@@ -31,6 +31,7 @@ namespace uhd{ namespace usrp{
     enum mboard_prop_t{
         MBOARD_PROP_NAME            = 'n', //ro, std::string
         MBOARD_PROP_OTHERS          = 'o', //ro, prop_names_t
+        MBOARD_PROP_CLOCK_RATE      = 'c', //rw, double
         MBOARD_PROP_RX_DSP          = 'd', //ro, wax::obj
         MBOARD_PROP_RX_DSP_NAMES    = 'D', //ro, prop_names_t
         MBOARD_PROP_TX_DSP          = 'u', //ro, wax::obj
diff --git a/host/include/uhd/usrp/multi_usrp.hpp b/host/include/uhd/usrp/multi_usrp.hpp
index 6e17c0ea8..c77b5d6d2 100644
--- a/host/include/uhd/usrp/multi_usrp.hpp
+++ b/host/include/uhd/usrp/multi_usrp.hpp
@@ -106,6 +106,26 @@ public:
     /*******************************************************************
      * Mboard methods
      ******************************************************************/
+
+    /*!
+     * Set the master clock rate.
+     * This controls the rate of the clock that feeds the FPGA DSP.
+     * On some devices, this re-tunes the clock to the specified rate.
+     * If the specified rate is not available, this method will throw.
+     * On other devices, this method notifies the software of the rate,
+     * but requires the the user has made the necessary hardware change.
+     * \param rate the new master clock rate in Hz
+     * \param mboard the motherboard index 0 to M-1
+     */
+    virtual void set_master_clock_rate(double rate, size_t mboard = ALL_MBOARDS) = 0;
+
+    /*!
+     * Get the master clock rate.
+     * \param mboard the motherboard index 0 to M-1
+     * \return the master clock rate in Hz.
+     */
+    virtual double get_master_clock_rate(size_t mboard = 0) = 0;
+
     /*!
      * Get a printable summary for this USRP configuration.
      * \return a printable string
diff --git a/host/lib/usrp/multi_usrp.cpp b/host/lib/usrp/multi_usrp.cpp
index 17458496b..817d7b085 100644
--- a/host/lib/usrp/multi_usrp.cpp
+++ b/host/lib/usrp/multi_usrp.cpp
@@ -53,6 +53,20 @@ public:
     /*******************************************************************
      * Mboard methods
      ******************************************************************/
+    void set_master_clock_rate(double rate, size_t mboard){
+        if (mboard != ALL_MBOARDS){
+            _mboard(mboard)[MBOARD_PROP_CLOCK_RATE] = rate;
+            return;
+        }
+        for (size_t m = 0; m < get_num_mboards(); m++){
+            set_master_clock_rate(rate, m);
+        }
+    }
+
+    double get_master_clock_rate(size_t mboard){
+        return _mboard(mboard)[MBOARD_PROP_CLOCK_RATE].as();
+    }
+
     std::string get_pp_string(void){
         std::string buff = str(boost::format(
             "%s USRP:\n"
diff --git a/host/lib/usrp/usrp1/clock_ctrl.cpp b/host/lib/usrp/usrp1/clock_ctrl.cpp
index 68c5f5320..156f2b0c4 100644
--- a/host/lib/usrp/usrp1/clock_ctrl.cpp
+++ b/host/lib/usrp/usrp1/clock_ctrl.cpp
@@ -29,32 +29,33 @@ using namespace uhd;
 /***********************************************************************
  * Constants
  **********************************************************************/
-static const double master_clock_rate = 64e6;
+static const double default_master_clock_rate = 64e6;
 
 /***********************************************************************
  * Clock Control Implementation
  **********************************************************************/
 class usrp1_clock_ctrl_impl : public usrp1_clock_ctrl {
 public:
-    usrp1_clock_ctrl_impl(usrp1_iface::sptr iface)
-    {
-        _iface = iface;
+    usrp1_clock_ctrl_impl(usrp1_iface::sptr iface): _iface(iface){
+        this->set_master_clock_freq(default_master_clock_rate);
     }
 
-    double get_master_clock_freq(void)
-    {
-        return master_clock_rate; 
+    void set_master_clock_freq(double freq){
+        _freq = freq;
+    }
+
+    double get_master_clock_freq(void){
+        return _freq;
     }
 
 private:
     usrp1_iface::sptr _iface;
-
+    double _freq;
 };
 
 /***********************************************************************
  * Clock Control Make
  **********************************************************************/
-usrp1_clock_ctrl::sptr usrp1_clock_ctrl::make(usrp1_iface::sptr iface)
-{
+usrp1_clock_ctrl::sptr usrp1_clock_ctrl::make(usrp1_iface::sptr iface){
     return sptr(new usrp1_clock_ctrl_impl(iface));
 }
diff --git a/host/lib/usrp/usrp1/clock_ctrl.hpp b/host/lib/usrp/usrp1/clock_ctrl.hpp
index 366869dab..645472f02 100644
--- a/host/lib/usrp/usrp1/clock_ctrl.hpp
+++ b/host/lib/usrp/usrp1/clock_ctrl.hpp
@@ -39,6 +39,13 @@ public:
      */
     static sptr make(usrp1_iface::sptr iface);
 
+    /*!
+     * Set the rate of the fpga clock line.
+     * Note: does not really set, its all software.
+     * \param freq the new clock rate in Hz
+     */
+    virtual void set_master_clock_freq(double freq) = 0;
+
     /*!
      * Get the rate of the fpga clock line.
      * \return the fpga clock rate in Hz
diff --git a/host/lib/usrp/usrp1/mboard_impl.cpp b/host/lib/usrp/usrp1/mboard_impl.cpp
index 23c8f03c4..6d5bf466d 100644
--- a/host/lib/usrp/usrp1/mboard_impl.cpp
+++ b/host/lib/usrp/usrp1/mboard_impl.cpp
@@ -317,6 +317,10 @@ void usrp1_impl::mboard_get(const wax::obj &key_, wax::obj &val)
         val = _soft_time_ctrl->get_time();
         return;
 
+    case MBOARD_PROP_CLOCK_RATE:
+        val = _clock_ctrl->get_master_clock_freq();
+        return;
+
     default: UHD_THROW_PROP_GET_ERROR();
     }
 }
@@ -379,6 +383,10 @@ void usrp1_impl::mboard_set(const wax::obj &key, const wax::obj &val)
         _soft_time_ctrl->set_time(val.as());
         return;
 
+    case MBOARD_PROP_CLOCK_RATE:
+        _clock_ctrl->set_master_clock_freq(val.as());
+        return;
+
     default: UHD_THROW_PROP_SET_ERROR();
     }
 }
diff --git a/host/lib/usrp/usrp2/mboard_impl.cpp b/host/lib/usrp/usrp2/mboard_impl.cpp
index 3e7e8b01f..784f662d9 100644
--- a/host/lib/usrp/usrp2/mboard_impl.cpp
+++ b/host/lib/usrp/usrp2/mboard_impl.cpp
@@ -355,6 +355,10 @@ void usrp2_mboard_impl::get(const wax::obj &key_, wax::obj &val){
         val = _iface->mb_eeprom;
         return;
 
+    case MBOARD_PROP_CLOCK_RATE:
+        val = this->get_master_clock_freq();
+        return;
+
     default: UHD_THROW_PROP_GET_ERROR();
     }
 }
@@ -412,6 +416,10 @@ void usrp2_mboard_impl::set(const wax::obj &key, const wax::obj &val){
         _iface->mb_eeprom = mboard_eeprom_t(*_iface, mboard_eeprom_t::MAP_N100);
         return;
 
+    case MBOARD_PROP_CLOCK_RATE:
+        UHD_ASSERT_THROW(val.as() == this->get_master_clock_freq());
+        return;
+
     default: UHD_THROW_PROP_SET_ERROR();
     }
 }
-- 
cgit v1.2.3