diff options
Diffstat (limited to 'host/lib/usrp/b200')
| -rw-r--r-- | host/lib/usrp/b200/b200_iface.hpp | 3 | ||||
| -rw-r--r-- | host/lib/usrp/b200/b200_impl.cpp | 121 | ||||
| -rw-r--r-- | host/lib/usrp/b200/b200_impl.hpp | 61 | ||||
| -rw-r--r-- | host/lib/usrp/b200/b200_io_impl.cpp | 156 | 
4 files changed, 284 insertions, 57 deletions
| diff --git a/host/lib/usrp/b200/b200_iface.hpp b/host/lib/usrp/b200/b200_iface.hpp index 83adfdd64..1821865d3 100644 --- a/host/lib/usrp/b200/b200_iface.hpp +++ b/host/lib/usrp/b200/b200_iface.hpp @@ -26,7 +26,10 @@  #include "ad9361_ctrl.hpp"  const static boost::uint16_t B200_VENDOR_ID  = 0x2500; +const static boost::uint16_t B200_VENDOR_NI_ID = 0x3923;  const static boost::uint16_t B200_PRODUCT_ID = 0x0020; +const static boost::uint16_t B200_PRODUCT_NI_ID = 0x7813; +const static boost::uint16_t B210_PRODUCT_NI_ID = 0x7814;  const static boost::uint16_t FX3_VID = 0x04b4;  const static boost::uint16_t FX3_DEFAULT_PID = 0x00f3;  const static boost::uint16_t FX3_REENUM_PID = 0x00f0; diff --git a/host/lib/usrp/b200/b200_impl.cpp b/host/lib/usrp/b200/b200_impl.cpp index aa309e7eb..1523f1858 100644 --- a/host/lib/usrp/b200/b200_impl.cpp +++ b/host/lib/usrp/b200/b200_impl.cpp @@ -22,7 +22,7 @@  #include <uhd/utils/cast.hpp>  #include <uhd/exception.hpp>  #include <uhd/utils/static.hpp> -#include <uhd/utils/images.hpp> +#include <uhd/utils/paths.hpp>  #include <uhd/utils/safe_call.hpp>  #include <uhd/usrp/dboard_eeprom.hpp>  #include <boost/format.hpp> @@ -87,14 +87,16 @@ static device_addrs_t b200_find(const device_addr_t &hint)      //since an address and resource is intended for a different, non-USB, device.      if (hint.has_key("addr") || hint.has_key("resource")) return b200_addrs; -    boost::uint16_t vid, pid; +    size_t found = 0; +    std::vector<usb_device_handle::vid_pid_pair_t> vid_pid_pair_list;//vid pid pair search list for devices.      if(hint.has_key("vid") && hint.has_key("pid") && hint.has_key("type") && hint["type"] == "b200") { -        vid = uhd::cast::hexstr_cast<boost::uint16_t>(hint.get("vid")); -        pid = uhd::cast::hexstr_cast<boost::uint16_t>(hint.get("pid")); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(uhd::cast::hexstr_cast<boost::uint16_t>(hint.get("vid")), +                                                                    uhd::cast::hexstr_cast<boost::uint16_t>(hint.get("pid"))));      } else { -        vid = B200_VENDOR_ID; -        pid = B200_PRODUCT_ID; +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_ID, B200_PRODUCT_ID)); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_NI_ID, B200_PRODUCT_NI_ID)); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_NI_ID, B210_PRODUCT_NI_ID));      }      // Important note: @@ -104,8 +106,9 @@ static device_addrs_t b200_find(const device_addr_t &hint)      // This requirement is a courtesy of libusb1.0 on windows.      //find the usrps and load firmware -    size_t found = 0; -    BOOST_FOREACH(usb_device_handle::sptr handle, usb_device_handle::get_device_list(vid, pid)) { +    std::vector<usb_device_handle::sptr> uhd_usb_device_vector = usb_device_handle::get_device_list(vid_pid_pair_list); + +    BOOST_FOREACH(usb_device_handle::sptr handle, uhd_usb_device_vector) {          //extract the firmware path for the b200          std::string b200_fw_image;          try{ @@ -115,7 +118,7 @@ static device_addrs_t b200_find(const device_addr_t &hint)              UHD_MSG(warning) << boost::format(                  "Could not locate B200 firmware.\n"                  "Please install the images package. %s\n" -            ) % print_images_error(); +            ) % print_utility_error("uhd_images_downloader.py");              return b200_addrs;          }          UHD_LOG << "the firmware image: " << b200_fw_image << std::endl; @@ -138,7 +141,7 @@ static device_addrs_t b200_find(const device_addr_t &hint)      //search for the device until found or timeout      while (boost::get_system_time() < timeout_time and b200_addrs.empty() and found != 0)      { -        BOOST_FOREACH(usb_device_handle::sptr handle, usb_device_handle::get_device_list(vid, pid)) +        BOOST_FOREACH(usb_device_handle::sptr handle, usb_device_handle::get_device_list(vid_pid_pair_list))          {              usb_control::sptr control;              try{control = usb_control::make(handle, 0);} @@ -155,12 +158,16 @@ static device_addrs_t b200_find(const device_addr_t &hint)              {                  switch (boost::lexical_cast<boost::uint16_t>(mb_eeprom["product"]))                  { +                //0x0001 and 0x7737 are Ettus B200 product Ids.                  case 0x0001:                  case 0x7737: +                case B200_PRODUCT_NI_ID:                      new_addr["product"] = "B200";                      break; -                case 0x7738: +                //0x0002 and 0x7738 are Ettus B210 product Ids.                  case 0x0002: +                case 0x7738: +                case B210_PRODUCT_NI_ID:                      new_addr["product"] = "B210";                      break;                  default: UHD_MSG(error) << "B200 unknown product code: " << mb_eeprom["product"] << std::endl; @@ -204,13 +211,50 @@ b200_impl::b200_impl(const device_addr_t &device_addr)      //try to match the given device address with something on the USB bus      boost::uint16_t vid = B200_VENDOR_ID;      boost::uint16_t pid = B200_PRODUCT_ID; +    bool specified_vid = false; +    bool specified_pid = false; +      if (device_addr.has_key("vid")) +    {          vid = uhd::cast::hexstr_cast<boost::uint16_t>(device_addr.get("vid")); +        specified_vid = true; +    } +      if (device_addr.has_key("pid")) +    {          pid = uhd::cast::hexstr_cast<boost::uint16_t>(device_addr.get("pid")); +        specified_pid = true; +    } -    std::vector<usb_device_handle::sptr> device_list = -        usb_device_handle::get_device_list(vid, pid); +    std::vector<usb_device_handle::vid_pid_pair_t> vid_pid_pair_list;//search list for devices. + +    // Search only for specified VID and PID if both specified +    if (specified_vid && specified_pid) +    { +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(vid,pid)); +    } +    // Search for all supported PIDs limited to specified VID if only VID specified +    else if (specified_vid) +    { +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(vid,B200_PRODUCT_ID)); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(vid,B200_PRODUCT_NI_ID)); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(vid,B210_PRODUCT_NI_ID)); +    } +    // Search for all supported VIDs limited to specified PID if only PID specified +    else if (specified_pid) +    { +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_ID,pid)); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_NI_ID,pid)); +    } +    // Search for all supported devices if neither VID nor PID specified +    else +    { +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_ID,B200_PRODUCT_ID)); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_NI_ID,B200_PRODUCT_NI_ID)); +        vid_pid_pair_list.push_back(usb_device_handle::vid_pid_pair_t(B200_VENDOR_NI_ID,B210_PRODUCT_NI_ID)); +    } + +    std::vector<usb_device_handle::sptr> device_list = usb_device_handle::get_device_list(vid_pid_pair_list);      //locate the matching handle in the device list      usb_device_handle::sptr handle; @@ -244,13 +288,17 @@ b200_impl::b200_impl(const device_addr_t &device_addr)      {          switch (boost::lexical_cast<boost::uint16_t>(mb_eeprom["product"]))          { +        //0x0001 and 0x7737 are Ettus B200 product Ids.          case 0x0001:          case 0x7737: +        case B200_PRODUCT_NI_ID:              product_name = "B200";              default_file_name = B200_FPGA_FILE_NAME;              break; -        case 0x7738: +        //0x0002 and 0x7738 are Ettus B210 product Ids.          case 0x0002: +        case 0x7738: +        case B210_PRODUCT_NI_ID:              product_name = "B210";              default_file_name = B210_FPGA_FILE_NAME;              break; @@ -416,6 +464,7 @@ b200_impl::b200_impl(const device_addr_t &device_addr)          .publish(boost::bind(&b200_impl::get_tick_rate, this))          .subscribe(boost::bind(&b200_impl::update_tick_rate, this, _1));      _tree->create<time_spec_t>(mb_path / "time" / "cmd"); +    _tree->create<bool>(mb_path / "auto_tick_rate").set(false);      ////////////////////////////////////////////////////////////////////      // and do the misc mboard sensors @@ -516,6 +565,11 @@ b200_impl::b200_impl(const device_addr_t &device_addr)          _tree->access<double>(mb_path / "rx_dsps" / str(boost::format("%u") % i)/ "rate/value").set(B200_DEFAULT_RATE);          _tree->access<double>(mb_path / "tx_dsps" / str(boost::format("%u") % i) / "rate/value").set(B200_DEFAULT_RATE);      } +    // We can automatically choose a master clock rate, but not if the user specifies one +    _tree->access<bool>(mb_path / "auto_tick_rate").set(not device_addr.has_key("master_clock_rate")); +    if (not device_addr.has_key("master_clock_rate")) { +        UHD_MSG(status) << "Setting master clock rate selection to 'automatic'." << std::endl; +    }      //GPS installed: use external ref, time, and init time spec      if (_gps and _gps->gps_detected()) @@ -579,7 +633,7 @@ void b200_impl::setup_radio(const size_t dspno)      _tree->create<meta_range_t>(rx_dsp_path / "rate" / "range")          .publish(boost::bind(&rx_dsp_core_3000::get_host_rates, perif.ddc));      _tree->create<double>(rx_dsp_path / "rate" / "value") -        .coerce(boost::bind(&rx_dsp_core_3000::set_host_rate, perif.ddc, _1)) +        .coerce(boost::bind(&b200_impl::coerce_rx_samp_rate, this, perif.ddc, dspno, _1))          .subscribe(boost::bind(&b200_impl::update_rx_samp_rate, this, dspno, _1))          .set(0.0); // Can only set this after tick rate is initialized.      _tree->create<double>(rx_dsp_path / "freq" / "value") @@ -603,7 +657,7 @@ void b200_impl::setup_radio(const size_t dspno)      _tree->create<meta_range_t>(tx_dsp_path / "rate" / "range")          .publish(boost::bind(&tx_dsp_core_3000::get_host_rates, perif.duc));      _tree->create<double>(tx_dsp_path / "rate" / "value") -        .coerce(boost::bind(&tx_dsp_core_3000::set_host_rate, perif.duc, _1)) +        .coerce(boost::bind(&b200_impl::coerce_tx_samp_rate, this, perif.duc, dspno, _1))          .subscribe(boost::bind(&b200_impl::update_tx_samp_rate, this, dspno, _1))          .set(0.0); // Can only set this after tick rate is initialized.      _tree->create<double>(tx_dsp_path / "freq" / "value") @@ -721,7 +775,7 @@ void b200_impl::codec_loopback_self_test(wb_iface::sptr iface)  /***********************************************************************   * Sample and tick rate comprehension below   **********************************************************************/ -void b200_impl::enforce_tick_rate_limits(size_t chan_count, double tick_rate, const char* direction /*= NULL*/) +void b200_impl::enforce_tick_rate_limits(size_t chan_count, double tick_rate, const std::string &direction /*= ""*/)  {      const size_t max_chans = 2;      if (chan_count > max_chans) @@ -729,7 +783,7 @@ void b200_impl::enforce_tick_rate_limits(size_t chan_count, double tick_rate, co          throw uhd::value_error(boost::str(              boost::format("cannot not setup %d %s channels (maximum is %d)")                  % chan_count -                % (direction ? direction : "data") +                % (direction.empty() ? "data" : direction)                  % max_chans          ));      } @@ -743,20 +797,26 @@ void b200_impl::enforce_tick_rate_limits(size_t chan_count, double tick_rate, co                      % (tick_rate/1e6)                      % (max_tick_rate/1e6)                      % chan_count -                    % (direction ? direction : "data") +                    % (direction.empty() ? "data" : direction)              ));          }      }  } -double b200_impl::set_tick_rate(const double rate) +double b200_impl::set_tick_rate(const double new_tick_rate)  { -    UHD_MSG(status) << (boost::format("Asking for clock rate %.6f MHz\n") % (rate/1e6)); - -    check_tick_rate_with_current_streamers(rate);   // Defined in b200_io_impl.cpp +    UHD_MSG(status) << (boost::format("Asking for clock rate %.6f MHz... ") % (new_tick_rate/1e6)) << std::flush; +    check_tick_rate_with_current_streamers(new_tick_rate);   // Defined in b200_io_impl.cpp + +    // Make sure the clock rate is actually changed before doing +    // the full Monty of setting regs and loopback tests etc. +    if (std::abs(new_tick_rate - _tick_rate) < 1.0) { +        UHD_MSG(status) << "OK" << std::endl; +        return _tick_rate; +    } -    _tick_rate = _codec_ctrl->set_clock_rate(rate); -    UHD_MSG(status) << (boost::format("Actually got clock rate %.6f MHz\n") % (_tick_rate/1e6)); +    _tick_rate = _codec_ctrl->set_clock_rate(new_tick_rate); +    UHD_MSG(status) << std::endl << (boost::format("Actually got clock rate %.6f MHz.") % (_tick_rate/1e6)) << std::endl;      //reset after clock rate change      this->reset_codec_dcm(); @@ -781,11 +841,11 @@ void b200_impl::check_fw_compat(void)      if (compat_major != B200_FW_COMPAT_NUM_MAJOR){          throw uhd::runtime_error(str(boost::format( -            "Expected firmware compatibility number 0x%x, but got 0x%x.%x:\n" +            "Expected firmware compatibility number %d.%d, but got %d.%d:\n"              "The firmware build is not compatible with the host code build.\n"              "%s" -        ) % int(B200_FW_COMPAT_NUM_MAJOR) % compat_major % compat_minor -          % print_images_error())); +        )   % int(B200_FW_COMPAT_NUM_MAJOR) % int(B200_FW_COMPAT_NUM_MINOR) +            % compat_major % compat_minor % print_utility_error("uhd_images_downloader.py")));      }      _tree->create<std::string>("/mboards/0/fw_version").set(str(boost::format("%u.%u")                  % compat_major % compat_minor)); @@ -802,11 +862,10 @@ void b200_impl::check_fpga_compat(void)      if (compat_major != B200_FPGA_COMPAT_NUM){          throw uhd::runtime_error(str(boost::format( -            "Expected FPGA compatibility number 0x%x, but got 0x%x.%x:\n" +            "Expected FPGA compatibility number %d, but got %d:\n"              "The FPGA build is not compatible with the host code build.\n"              "%s" -        ) % int(B200_FPGA_COMPAT_NUM) % compat_major % compat_minor -          % print_images_error())); +        ) % int(B200_FPGA_COMPAT_NUM) % compat_major % print_utility_error("uhd_images_downloader.py")));      }      _tree->create<std::string>("/mboards/0/fpga_version").set(str(boost::format("%u.%u")                  % compat_major % compat_minor)); diff --git a/host/lib/usrp/b200/b200_impl.hpp b/host/lib/usrp/b200/b200_impl.hpp index 8f03d81ad..71592f60b 100644 --- a/host/lib/usrp/b200/b200_impl.hpp +++ b/host/lib/usrp/b200/b200_impl.hpp @@ -45,9 +45,9 @@  #include <uhd/transport/bounded_buffer.hpp>  #include <boost/weak_ptr.hpp>  #include "recv_packet_demuxer_3000.hpp" -static const boost::uint8_t  B200_FW_COMPAT_NUM_MAJOR = 0x07; -static const boost::uint8_t  B200_FW_COMPAT_NUM_MINOR = 0x00; -static const boost::uint16_t B200_FPGA_COMPAT_NUM = 0x04; +static const boost::uint8_t  B200_FW_COMPAT_NUM_MAJOR = 7; +static const boost::uint8_t  B200_FW_COMPAT_NUM_MINOR = 0; +static const boost::uint16_t B200_FPGA_COMPAT_NUM = 4;  static const double          B200_BUS_CLOCK_RATE = 100e6;  static const double          B200_DEFAULT_TICK_RATE = 32e6;  static const double          B200_DEFAULT_FREQ = 100e6; // Hz @@ -98,7 +98,13 @@ public:      uhd::rx_streamer::sptr get_rx_stream(const uhd::stream_args_t &args);      uhd::tx_streamer::sptr get_tx_stream(const uhd::stream_args_t &args);      bool recv_async_msg(uhd::async_metadata_t &, double); -    void check_streamer_args(const uhd::stream_args_t &args, double tick_rate, const char* direction = NULL); + +    //! Check that the combination of stream args and tick rate are valid. +    // +    // Basically figures out the arguments for enforce_tick_rate_limits() +    // and calls said method. If arguments are invalid, throws a +    // uhd::value_error. +    void check_streamer_args(const uhd::stream_args_t &args, double tick_rate, const std::string &direction = "");  private:      //controllers @@ -190,11 +196,56 @@ private:      double _tick_rate;      double get_tick_rate(void){return _tick_rate;}      double set_tick_rate(const double rate); + +    /*! \brief Choose a tick rate (master clock rate) that works well for the given sampling rate. +     * +     * This function will try and choose a master clock rate automatically. +     * See the function definition for details on the algorithm. +     * +     * The chosen tick rate is the largest multiple of two that is smaler +     * than the max tick rate. +     * The base rate is either given explicitly, or is the lcm() of the tx +     * and rx sampling rates. In that case, it reads the rates directly +     * from the property tree. It also tries to guess the number of channels +     * (for the max possible tick rate) by checking the available streamers. +     * This value, too, can explicitly be given. +     * +     * \param rate If this is given, it will be used as a minimum rate, or +     *             argument to lcm(). +     * \param tree_dsp_path The sampling rate from this property tree path +     *                      will be ignored. +     * \param num_chans If given, specifies the number of channels. +     */ +    void set_auto_tick_rate( +            const double rate=0, +            const uhd::fs_path &tree_dsp_path="", +            size_t num_chans=0 +    ); +      void update_tick_rate(const double); -    void enforce_tick_rate_limits(size_t chan_count, double tick_rate, const char* direction = NULL); + +    /*! Check if \p tick_rate works with \p chan_count channels. +     * +     * Throws a uhd::value_error if not. +     */ +    void enforce_tick_rate_limits(size_t chan_count, double tick_rate, const std::string  &direction = "");      void check_tick_rate_with_current_streamers(double rate); +    /*! Return the max number of channels on active rx_streamer or tx_streamer objects associated with this device. +     * +     * \param direction Set to "TX" to only check tx_streamers, "RX" to only check +     *                  rx_streamers. Any other value will check if \e any active +     *                  streamers are available. +     * \return Return the number of tx streamers (direction=="TX"), the number of rx +     *         streamers (direction=="RX") or the total number of streamers. +     */ +    size_t max_chan_count(const std::string &direction=""); + +    //! Coercer, attached to the "rate/value" property on the rx dsps. +    double coerce_rx_samp_rate(rx_dsp_core_3000::sptr, size_t, const double);      void update_rx_samp_rate(const size_t, const double); +    //! Coercer, attached to the "rate/value" property on the tx dsps. +    double coerce_tx_samp_rate(tx_dsp_core_3000::sptr, size_t, const double);      void update_tx_samp_rate(const size_t, const double);  }; diff --git a/host/lib/usrp/b200/b200_io_impl.cpp b/host/lib/usrp/b200/b200_io_impl.cpp index 262c95f0d..60b925517 100644 --- a/host/lib/usrp/b200/b200_io_impl.cpp +++ b/host/lib/usrp/b200/b200_io_impl.cpp @@ -1,5 +1,5 @@  // -// Copyright 2012-2013 Ettus Research LLC +// Copyright 2012-2014 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 @@ -21,8 +21,10 @@  #include "../../transport/super_recv_packet_handler.hpp"  #include "../../transport/super_send_packet_handler.hpp"  #include "async_packet_handler.hpp" +#include <uhd/utils/math.hpp>  #include <boost/bind.hpp>  #include <boost/make_shared.hpp> +#include <boost/math/common_factor.hpp>  #include <set>  using namespace uhd; @@ -34,30 +36,32 @@ using namespace uhd::transport;   **********************************************************************/  void b200_impl::check_tick_rate_with_current_streamers(double rate)  { -    size_t max_tx_chan_count = 0, max_rx_chan_count = 0; +    // Defined in b200_impl.cpp +    enforce_tick_rate_limits(max_chan_count("RX"), rate, "RX"); +    enforce_tick_rate_limits(max_chan_count("TX"), rate, "TX"); +} + +// direction can either be "TX", "RX", or empty (default) +size_t b200_impl::max_chan_count(const std::string &direction /* = "" */) +{ +    size_t max_count = 0;      BOOST_FOREACH(radio_perifs_t &perif, _radio_perifs)      { -        { +        if ((direction == "RX" or direction.empty()) and not perif.rx_streamer.expired()) {              boost::shared_ptr<sph::recv_packet_streamer> rx_streamer =                  boost::dynamic_pointer_cast<sph::recv_packet_streamer>(perif.rx_streamer.lock()); -            if (rx_streamer) -                max_rx_chan_count = std::max(max_rx_chan_count, rx_streamer->get_num_channels()); +            max_count = std::max(max_count, rx_streamer->get_num_channels());          } - -        { +        if ((direction == "TX" or direction.empty()) and not perif.tx_streamer.expired()) {              boost::shared_ptr<sph::send_packet_streamer> tx_streamer =                  boost::dynamic_pointer_cast<sph::send_packet_streamer>(perif.tx_streamer.lock()); -            if (tx_streamer) -                max_tx_chan_count = std::max(max_tx_chan_count, tx_streamer->get_num_channels()); +            max_count = std::max(max_count, tx_streamer->get_num_channels());          }      } - -    // Defined in b200_impl.cpp -    enforce_tick_rate_limits(max_rx_chan_count, rate, "RX"); -    enforce_tick_rate_limits(max_tx_chan_count, rate, "TX"); +    return max_count;  } -void b200_impl::check_streamer_args(const uhd::stream_args_t &args, double tick_rate, const char* direction /*= NULL*/) +void b200_impl::check_streamer_args(const uhd::stream_args_t &args, double tick_rate, const std::string &direction /*= ""*/)  {      std::set<size_t> chans_set;      for (size_t stream_i = 0; stream_i < args.channels.size(); stream_i++) @@ -69,26 +73,120 @@ void b200_impl::check_streamer_args(const uhd::stream_args_t &args, double tick_      enforce_tick_rate_limits(chans_set.size(), tick_rate, direction);   // Defined in b200_impl.cpp  } -void b200_impl::update_tick_rate(const double rate) +void b200_impl::set_auto_tick_rate( +        const double rate, +        const fs_path &tree_dsp_path, +        size_t num_chans +) { +    if (num_chans == 0) { // Divine them +        num_chans = std::max(size_t(1), max_chan_count()); +    } + +    // See also the doxygen documentation for these steps in b200_impl.hpp +    // Step 1: Obtain LCM and max rate from all relevant dsps +    boost::uint32_t lcm_rate = (rate == 0) ? 1 : static_cast<boost::uint32_t>(floor(rate + 0.5)); +    for (int i = 0; i < 2; i++) { // Loop through rx and tx +        std::string dir = (i == 0) ? "tx" : "rx"; +        // We have no way of knowing which DSPs are used, so we check them all. +        BOOST_FOREACH(const std::string &dsp_no, _tree->list(str(boost::format("/mboards/0/%s_dsps") % dir))) { +            fs_path dsp_path = str(boost::format("/mboards/0/%s_dsps/%s") % dir % dsp_no); +            if (dsp_path == tree_dsp_path) { +                continue; +            } +            double this_dsp_rate = _tree->access<double>(dsp_path / "rate/value").get(); +            // If this_dsp_rate == B200_DEFAULT_RATE, we assume the user did not actually set +            // the sampling rate. If the user *did* set the rate to +            // B200_DEFAULT_RATE on all DSPs, then this will still work (see below). +            // If the user set one DSP to B200_DEFAULT_RATE and the other to +            // a different rate, this also works if the rates are integer multiples +            // of one another. Only for certain configurations of B200_DEFAULT_RATE +            // and another rate that is not an integer multiple, this will be problematic. +            // Since this case is less common than the case where a rate is left unset, +            // we don't handle that but rather explain that corner case in the documentation. +            if (this_dsp_rate == B200_DEFAULT_RATE) { +                continue; +            } +            lcm_rate = boost::math::lcm<boost::uint32_t>( +                    lcm_rate, +                    static_cast<boost::uint32_t>(floor(this_dsp_rate + 0.5)) +            ); +        } +    } +    if (lcm_rate == 1) { +        lcm_rate = static_cast<boost::uint32_t>(B200_DEFAULT_RATE); +    } + +    // Step 2: Determine whether if we can use lcm_rate (preferred), +    // or have to give up because too large: +    const double max_tick_rate = +        ((num_chans <= 1) ? ad9361_device_t::AD9361_RECOMMENDED_MAX_CLOCK_RATE : ad9361_device_t::AD9361_MAX_CLOCK_RATE/2); +    double base_rate = static_cast<double>(lcm_rate); +    if (base_rate > max_tick_rate) { +        UHD_MSG(warning) +            << "Cannot automatically determine an appropriate tick rate for these sampling rates." << std::endl +            << "Consider using different sampling rates, or manually specify a suitable master clock rate." << std::endl; +        return; // Let the others handle this +    } + +    // Step 3: Choose the new rate +    // Rules for choosing the tick rate: +    // Choose a rate that is a power of 2 larger than the sampling rate, +    // but at least 4. Cannot exceed the max tick rate, of course, but must +    // be larger than the minimum tick rate. +    // An equation that does all that is: +    // +    // f_auto = r * 2^floor(log2(f_max/r)) +    // +    // where r is the base rate and f_max is the maximum tick rate. The case +    // where floor() yields 1 must be caught. +    const double min_tick_rate = _codec_ctrl->get_clock_rate_range().start(); +    // We use shifts here instead of 2^x because exp2() is not available in all compilers, +    // also this guarantees no rounding issues. The type cast to int32_t serves as floor(): +    boost::int32_t multiplier = (1 << boost::int32_t(uhd::math::log2(max_tick_rate / base_rate))); +    if (multiplier == 2 and base_rate >= min_tick_rate) { +        // Don't bother (see above) +        multiplier = 1; +    } +    double new_rate = base_rate * multiplier; +    UHD_ASSERT_THROW(new_rate >= min_tick_rate); +    UHD_ASSERT_THROW(new_rate <= max_tick_rate); + +    if (_tree->access<double>("/mboards/0/tick_rate").get() != new_rate) { +        _tree->access<double>("/mboards/0/tick_rate").set(new_rate); +    } +} + +void b200_impl::update_tick_rate(const double new_tick_rate)  { -    check_tick_rate_with_current_streamers(rate); +    check_tick_rate_with_current_streamers(new_tick_rate);      BOOST_FOREACH(radio_perifs_t &perif, _radio_perifs)      {          boost::shared_ptr<sph::recv_packet_streamer> my_streamer =              boost::dynamic_pointer_cast<sph::recv_packet_streamer>(perif.rx_streamer.lock()); -        if (my_streamer) my_streamer->set_tick_rate(rate); -        perif.framer->set_tick_rate(_tick_rate); +        if (my_streamer) my_streamer->set_tick_rate(new_tick_rate); +        perif.framer->set_tick_rate(new_tick_rate);      }      BOOST_FOREACH(radio_perifs_t &perif, _radio_perifs)      {          boost::shared_ptr<sph::send_packet_streamer> my_streamer =              boost::dynamic_pointer_cast<sph::send_packet_streamer>(perif.tx_streamer.lock()); -        if (my_streamer) my_streamer->set_tick_rate(rate); -        perif.deframer->set_tick_rate(_tick_rate); +        if (my_streamer) my_streamer->set_tick_rate(new_tick_rate); +        perif.deframer->set_tick_rate(new_tick_rate);      }  } + +double b200_impl::coerce_rx_samp_rate(rx_dsp_core_3000::sptr ddc, size_t dspno, const double rx_rate) +{ +    // Have to set tick rate first, or the ddc will change the requested rate based on default tick rate +    if (_tree->access<bool>("/mboards/0/auto_tick_rate").get()) { +        const std::string dsp_path = (boost::format("/mboards/0/rx_dsps/%s") % dspno).str(); +        set_auto_tick_rate(rx_rate, dsp_path); +    } +    return ddc->set_host_rate(rx_rate); +} +  void b200_impl::update_rx_samp_rate(const size_t dspno, const double rate)  {      boost::shared_ptr<sph::recv_packet_streamer> my_streamer = @@ -99,6 +197,16 @@ void b200_impl::update_rx_samp_rate(const size_t dspno, const double rate)      my_streamer->set_scale_factor(adj);  } +double b200_impl::coerce_tx_samp_rate(tx_dsp_core_3000::sptr duc, size_t dspno, const double tx_rate) +{ +    // Have to set tick rate first, or the duc will change the requested rate based on default tick rate +    if (_tree->access<bool>("/mboards/0/auto_tick_rate").get()) { +        const std::string dsp_path = (boost::format("/mboards/0/tx_dsps/%s") % dspno).str(); +        set_auto_tick_rate(tx_rate, dsp_path); +    } +    return duc->set_host_rate(tx_rate); +} +  void b200_impl::update_tx_samp_rate(const size_t dspno, const double rate)  {      boost::shared_ptr<sph::send_packet_streamer> my_streamer = @@ -264,6 +372,9 @@ rx_streamer::sptr b200_impl::get_rx_stream(const uhd::stream_args_t &args_)      if (args.otw_format.empty()) args.otw_format = "sc16";      args.channels = args.channels.empty()? std::vector<size_t>(1, 0) : args.channels; +    if (_tree->access<bool>("/mboards/0/auto_tick_rate").get()) { +        set_auto_tick_rate(0, "", args.channels.size()); +    }      check_streamer_args(args, this->get_tick_rate(), "RX");      boost::shared_ptr<sph::recv_packet_streamer> my_streamer; @@ -371,7 +482,10 @@ tx_streamer::sptr b200_impl::get_tx_stream(const uhd::stream_args_t &args_)      if (args.otw_format.empty()) args.otw_format = "sc16";      args.channels = args.channels.empty()? std::vector<size_t>(1, 0) : args.channels; -    check_streamer_args(args, this->get_tick_rate(), "TX"); +    if (_tree->access<bool>("/mboards/0/auto_tick_rate").get()) { +        set_auto_tick_rate(0, "", args.channels.size()); +    } +    check_streamer_args(args, this->get_tick_rate(), "RX");      boost::shared_ptr<sph::send_packet_streamer> my_streamer;      for (size_t stream_i = 0; stream_i < args.channels.size(); stream_i++) | 
