Merge branch 'next' into lime

13 files changed, 234 insertions, 62 deletions

diff --git a/src/ConfigParser.cpp b/src/ConfigParser.cpp
index 296ecdb..dd8a150 100644
--- a/src/ConfigParser.cpp
+++ b/src/ConfigParser.cpp
@@ -235,6 +235,7 @@ static void parse_configfile(
         sdr_device_config.rx_antenna = pt.Get("uhdoutput.rx_antenna", "RX2");
         sdr_device_config.rxgain = pt.GetReal("uhdoutput.rxgain", 0.0);
         sdr_device_config.frequency = pt.GetReal("uhdoutput.frequency", 0);
+        sdr_device_config.bandwidth = pt.GetReal("uhdoutput.bandwidth", 0);
         std::string chan = pt.Get("uhdoutput.channel", "");
         sdr_device_config.dabMode = mod_settings.dabMode;
 
@@ -287,6 +288,7 @@ static void parse_configfile(
         outputsoapy_conf.tx_antenna = pt.Get("soapyoutput.tx_antenna", "");
         outputsoapy_conf.lo_offset = pt.GetReal("soapyoutput.lo_offset", 0.0);
         outputsoapy_conf.frequency = pt.GetReal("soapyoutput.frequency", 0);
+        outputsoapy_conf.bandwidth = pt.GetReal("soapyoutput.bandwidth", 0);
         std::string chan = pt.Get("soapyoutput.channel", "");
         outputsoapy_conf.dabMode = mod_settings.dabMode;
 
diff --git a/src/DabMod.cpp b/src/DabMod.cpp
index 8267060..d340b30 100644
--- a/src/DabMod.cpp
+++ b/src/DabMod.cpp
@@ -326,11 +326,11 @@ int launch_modulator(int argc, char* argv[])
             // setMaxDelay wants number of AF packets, which correspond to 24ms ETI frames
             ediInput.setMaxDelay(lroundf(mod_settings.edi_max_delay_ms / 24.0f));
         }
-        EdiUdpInput ediUdpInput(ediInput);
+        EdiTransport ediTransport(ediInput);
 
-        ediUdpInput.Open(mod_settings.inputName);
-        if (not ediUdpInput.isEnabled()) {
-            throw runtime_error("inputTransport is edi, but ediUdpInput is not enabled");
+        ediTransport.Open(mod_settings.inputName);
+        if (not ediTransport.isEnabled()) {
+            throw runtime_error("inputTransport is edi, but ediTransport is not enabled");
         }
         Flowgraph flowgraph;
 
@@ -349,16 +349,27 @@ int launch_modulator(int argc, char* argv[])
 
         bool first_frame = true;
 
+        auto frame_received_tp = chrono::steady_clock::now();
+
         while (running) {
             while (running and not ediReader.isFrameReady()) {
                 try {
-                    ediUdpInput.rxPacket();
+                    bool packet_received = ediTransport.rxPacket();
+                    if (packet_received) {
+                        frame_received_tp = chrono::steady_clock::now();
+                    }
                 }
                 catch (const std::runtime_error& e) {
                     etiLog.level(warn) << "EDI input: " << e.what();
                     running = 0;
                     break;
                 }
+
+                if (frame_received_tp + chrono::seconds(10) < chrono::steady_clock::now()) {
+                    etiLog.level(error) << "No EDI data received in 10 seconds.";
+                    running = 0;
+                    break;
+                }
             }
 
             if (not running) {
@@ -512,12 +523,21 @@ int launch_modulator(int argc, char* argv[])
     return ret;
 }
 
+struct zmq_input_timeout : public std::exception
+{
+    const char* what() const throw()
+    {
+        return "InputZMQ timeout";
+    }
+};
+
 static run_modulator_state_t run_modulator(modulator_data& m)
 {
     auto ret = run_modulator_state_t::failure;
     try {
         bool first_frame = true;
         int last_eti_fct = -1;
+        auto last_frame_received = chrono::steady_clock::now();
 
         while (running) {
             int framesize;
@@ -530,6 +550,8 @@ static run_modulator_state_t run_modulator(modulator_data& m)
                     break;
                 }
 
+                last_frame_received = chrono::steady_clock::now();
+
                 m.framecount++;
 
                 PDEBUG("*****************************************\n");
@@ -581,7 +603,28 @@ static run_modulator_state_t run_modulator(modulator_data& m)
                 else if (dynamic_pointer_cast<InputZeroMQReader>(m.inputReader)) {
                     /* An empty frame marks a timeout. We ignore it, but we are
                      * now able to handle SIGINT properly.
+                     *
+                     * Also, we reconnect zmq every 10 seconds to avoid some
+                     * issues, discussed in
+                     * https://stackoverflow.com/questions/26112992/zeromq-pub-sub-on-unreliable-connection
+                     *
+                     * > It is possible that the PUB socket sees the error
+                     * > while the SUB socket does not.
+                     * >
+                     * > The ZMTP RFC has a proposal for heartbeating that would
+                     * > solve this problem.  The current best solution is for
+                     * > PUB sockets to send heartbeats (e.g. 1 per second) when
+                     * > traffic is low, and for SUB sockets to disconnect /
+                     * > reconnect if they stop getting these.
+                     *
+                     * We don't need a heartbeat, because our application is constant frame rate,
+                     * the frames themselves can act as heartbeats.
                      */
+
+                    const auto now = chrono::steady_clock::now();
+                    if (last_frame_received + chrono::seconds(10) < now) {
+                        throw zmq_input_timeout();
+                    }
                 }
 #endif // defined(HAVE_ZEROMQ)
                 else if (dynamic_pointer_cast<InputTcpReader>(m.inputReader)) {
@@ -598,6 +641,11 @@ static run_modulator_state_t run_modulator(modulator_data& m)
             }
         }
     }
+    catch (const zmq_input_timeout&) {
+        // The ZeroMQ input timeout
+        etiLog.level(warn) << "Timeout";
+        ret = run_modulator_state_t::again;
+    }
     catch (const zmq_input_overflow& e) {
         // The ZeroMQ input has overflowed its buffer
         etiLog.level(warn) << e.what();
diff --git a/src/EtiReader.cpp b/src/EtiReader.cpp
index 4c5ad79..94c362a 100644
--- a/src/EtiReader.cpp
+++ b/src/EtiReader.cpp
@@ -547,7 +547,7 @@ void EdiReader::assemble()
     m_frameReady = true;
 }
 
-EdiUdpInput::EdiUdpInput(EdiDecoder::ETIDecoder& decoder) :
+EdiTransport::EdiTransport(EdiDecoder::ETIDecoder& decoder) :
     m_enabled(false),
     m_port(0),
     m_bindto("0.0.0.0"),
@@ -555,49 +555,100 @@ EdiUdpInput::EdiUdpInput(EdiDecoder::ETIDecoder& decoder) :
     m_decoder(decoder) { }
 
 
-void EdiUdpInput::Open(const std::string& uri)
+void EdiTransport::Open(const std::string& uri)
 {
     etiLog.level(info) << "Opening EDI :" << uri;
-    size_t found_port = uri.find_first_of(":", 6);
-    if (found_port == string::npos) {
-        throw std::invalid_argument("EDI input port must be provided");
-    }
 
-    m_port = std::stoi(uri.substr(found_port+1));
-    std::string host_full = uri.substr(6, found_port-6);// ignore udp://
-    size_t found_mcast = host_full.find_first_of("@"); //have multicast address:
-    if (found_mcast != string::npos) {
-        if (found_mcast > 0) {
-            m_bindto = host_full.substr(0, found_mcast);
+    const string proto = uri.substr(0, 3);
+    if (proto == "udp") {
+        size_t found_port = uri.find_first_of(":", 6);
+        if (found_port == string::npos) {
+            throw std::invalid_argument("EDI UDP input port must be provided");
         }
-        m_mcastaddr = host_full.substr(found_mcast+1);
-    }
-    else if (found_port != 6) {
-        m_bindto=host_full;
+
+        m_port = std::stoi(uri.substr(found_port+1));
+        std::string host_full = uri.substr(6, found_port-6);// skip udp://
+        size_t found_mcast = host_full.find_first_of("@"); //have multicast address:
+        if (found_mcast != string::npos) {
+            if (found_mcast > 0) {
+                m_bindto = host_full.substr(0, found_mcast);
+            }
+            m_mcastaddr = host_full.substr(found_mcast+1);
+        }
+        else if (found_port != 6) {
+            m_bindto=host_full;
+        }
+
+        etiLog.level(info) << "EDI UDP input: host:" << m_bindto <<
+            ", source:" << m_mcastaddr << ", port:" << m_port;
+
+        // The max_fragments_queued is only a protection against a runaway
+        // memory usage.
+        // Rough calculation:
+        // 300 seconds, 24ms per frame, up to 20 fragments per frame
+        const size_t max_fragments_queued = 20 * 300 * 1000 / 24;
+
+        m_udp_rx.start(m_port, m_bindto, m_mcastaddr, max_fragments_queued);
+        m_proto = Proto::UDP;
+        m_enabled = true;
     }
+    else if (proto == "tcp") {
+        size_t found_port = uri.find_first_of(":", 6);
+        if (found_port == string::npos) {
+            throw std::invalid_argument("EDI TCP input port must be provided");
+        }
 
-    etiLog.level(info) << "EDI input: host:" << m_bindto <<
-        ", source:" << m_mcastaddr << ", port:" << m_port;
+        m_port = std::stoi(uri.substr(found_port+1));
+        const std::string hostname = uri.substr(6, found_port-6);// skip tcp://
 
-    // The max_fragments_queued is only a protection against a runaway
-    // memory usage.
-    // Rough calculation:
-    // 300 seconds, 24ms per frame, up to 20 fragments per frame
-    const size_t max_fragments_queued = 20 * 300 * 1000 / 24;
+        etiLog.level(info) << "EDI TCP connect to " << hostname << ":" << m_port;
 
-    m_udp_rx.start(m_port, m_bindto, m_mcastaddr, max_fragments_queued);
-    m_enabled = true;
+        m_tcpclient.connect(hostname, m_port);
+        m_proto = Proto::TCP;
+        m_enabled = true;
+    }
+    else {
+        throw std::invalid_argument("ETI protocol '" + proto + "' unknown");
+    }
 }
 
-bool EdiUdpInput::rxPacket()
+bool EdiTransport::rxPacket()
 {
-    auto udp_data = m_udp_rx.get_packet_buffer();
+    switch (m_proto) {
+        case Proto::UDP:
+            {
+                auto udp_data = m_udp_rx.get_packet_buffer();
 
-    if (udp_data.empty()) {
-        return false;
-    }
+                if (udp_data.empty()) {
+                    return false;
+                }
 
-    m_decoder.push_packet(udp_data);
-    return true;
+                m_decoder.push_packet(udp_data);
+                return true;
+            }
+        case Proto::TCP:
+            {
+                m_tcpbuffer.resize(4096);
+                const int timeout_ms = 1000;
+                try {
+                    ssize_t ret = m_tcpclient.recv(m_tcpbuffer.data(), m_tcpbuffer.size(), 0, timeout_ms);
+                    if (ret == 0 or ret == -1) {
+                        return false;
+                    }
+                    else if (ret > (ssize_t)m_tcpbuffer.size()) {
+                        throw logic_error("EDI TCP: invalid recv() return value");
+                    }
+                    else {
+                        m_tcpbuffer.resize(ret);
+                        m_decoder.push_bytes(m_tcpbuffer);
+                        return true;
+                    }
+                }
+                catch (const TCPSocket::Timeout&) {
+                    return false;
+                }
+            }
+    }
+    throw logic_error("Incomplete rxPacket implementation!");
 }
 #endif // HAVE_EDI
diff --git a/src/EtiReader.h b/src/EtiReader.h
index 554231e..38f7903 100644
--- a/src/EtiReader.h
+++ b/src/EtiReader.h
@@ -34,6 +34,7 @@
 #include "Eti.h"
 #include "Log.h"
 #include "FicSource.h"
+#include "Socket.h"
 #include "SubchannelSource.h"
 #include "TimestampDecoder.h"
 #include "lib/edi/ETIDecoder.hpp"
@@ -185,13 +186,12 @@ private:
 };
 
 /* The EDI input does not use the inputs defined in InputReader.h, as they were
- * designed for ETI. It uses the EdiUdpInput which in turn uses a threaded
+ * designed for ETI. It uses the EdiTransport which in turn uses a threaded
  * receiver.
  */
-
-class EdiUdpInput {
+class EdiTransport {
     public:
-        EdiUdpInput(EdiDecoder::ETIDecoder& decoder);
+        EdiTransport(EdiDecoder::ETIDecoder& decoder);
 
         void Open(const std::string& uri);
 
@@ -209,7 +209,11 @@ class EdiUdpInput {
         std::string m_bindto;
         std::string m_mcastaddr;
 
+        enum class Proto { UDP, TCP };
+        Proto m_proto;
         UdpReceiver m_udp_rx;
+        std::vector<uint8_t> m_tcpbuffer;
+        TCPClient m_tcpclient;
         EdiDecoder::ETIDecoder& m_decoder;
 };
 #endif
diff --git a/src/InputZeroMQReader.cpp b/src/InputZeroMQReader.cpp
index 3661748..1ebc1ca 100644
--- a/src/InputZeroMQReader.cpp
+++ b/src/InputZeroMQReader.cpp
@@ -42,6 +42,8 @@
 
 using namespace std;
 
+constexpr int ZMQ_TIMEOUT_MS = 100;
+
 #define NUM_FRAMES_PER_ZMQ_MESSAGE 4
 /* A concatenation of four ETI frames,
  * whose maximal size is 6144.
@@ -76,6 +78,9 @@ InputZeroMQReader::InputZeroMQReader() :
 InputZeroMQReader::~InputZeroMQReader()
 {
     m_running = false;
+    // This avoids the ugly "context was terminated" error because it lets
+    // poll do its thing first
+    this_thread::sleep_for(chrono::milliseconds(2 * ZMQ_TIMEOUT_MS));
     m_zmqcontext.close();
     if (m_recv_thread.joinable()) {
         m_recv_thread.join();
@@ -190,8 +195,7 @@ void InputZeroMQReader::RecvProcess()
             zmq::pollitem_t items[1];
             items[0].socket = subscriber;
             items[0].events = ZMQ_POLLIN;
-            const int zmq_timeout_ms = 100;
-            const int num_events = zmq::poll(items, 1, zmq_timeout_ms);
+            const int num_events = zmq::poll(items, 1, ZMQ_TIMEOUT_MS);
             if (num_events == 0) {
                 message_t msg;
                 msg.timeout = true;
diff --git a/src/RemoteControl.cpp b/src/RemoteControl.cpp
index 808153a..1065456 100644
--- a/src/RemoteControl.cpp
+++ b/src/RemoteControl.cpp
@@ -523,16 +523,21 @@ void RemoteControllerZmq::process()
                 }
                 else if (msg.size() == 2 && command == "show") {
                     std::string module((char*) msg[1].data(), msg[1].size());
-                    list< vector<string> > r = rcs.get_param_list_values(module);
-                    size_t r_size = r.size();
-                    for (auto &param_val : r) {
-                        std::stringstream ss;
-                        ss << param_val[0] << ": " << param_val[1] << endl;
-                        zmq::message_t zmsg(ss.str().size());
-                        memcpy(zmsg.data(), ss.str().data(), ss.str().size());
-
-                        int flag = (--r_size > 0) ? ZMQ_SNDMORE : 0;
-                        repSocket.send(zmsg, flag);
+                    try {
+                        list< vector<string> > r = rcs.get_param_list_values(module);
+                        size_t r_size = r.size();
+                        for (auto &param_val : r) {
+                            std::stringstream ss;
+                            ss << param_val[0] << ": " << param_val[1] << endl;
+                            zmq::message_t zmsg(ss.str().size());
+                            memcpy(zmsg.data(), ss.str().data(), ss.str().size());
+
+                            int flag = (--r_size > 0) ? ZMQ_SNDMORE : 0;
+                            repSocket.send(zmsg, flag);
+                        }
+                    }
+                    catch (const ParameterError &err) {
+                        send_fail_reply(repSocket, err.what());
                     }
                 }
                 else if (msg.size() == 3 && command == "get") {
diff --git a/src/TimeInterleaver.cpp b/src/TimeInterleaver.cpp
index 7e19af8..1afdefd 100644
--- a/src/TimeInterleaver.cpp
+++ b/src/TimeInterleaver.cpp
@@ -64,7 +64,7 @@ int TimeInterleaver::process(Buffer* const dataIn, Buffer* dataOut)
     unsigned char* out = reinterpret_cast<unsigned char*>(dataOut->getData());
 
     for (size_t i = 0; i < dataOut->getLength();) {
-        d_history.push_front(d_history.back());
+        d_history.push_front(move(d_history.back()));
         d_history.pop_back();
         for (uint_fast16_t j = 0; j < d_framesize;) {
             d_history[0][j] = in[i];
diff --git a/src/output/SDR.cpp b/src/output/SDR.cpp
index 95b54f9..01312ff 100644
--- a/src/output/SDR.cpp
+++ b/src/output/SDR.cpp
@@ -76,6 +76,7 @@ SDR::SDR(SDRDeviceConfig& config, std::shared_ptr<SDRDevice> device) :
 
     RC_ADD_PARAMETER(txgain, "TX gain");
     RC_ADD_PARAMETER(rxgain, "RX gain for DPD feedback");
+    RC_ADD_PARAMETER(bandwidth, "Analog front-end bandwidth");
     RC_ADD_PARAMETER(freq, "Transmission frequency");
     RC_ADD_PARAMETER(muting, "Mute the output by stopping the transmitter");
     RC_ADD_PARAMETER(temp, "Temperature in degrees C of the device");
@@ -381,6 +382,10 @@ void SDR::set_parameter(const string& parameter, const string& value)
         ss >> m_config.rxgain;
         m_device->set_rxgain(m_config.rxgain);
     }
+    else if (parameter == "bandwidth") {
+        ss >> m_config.bandwidth;
+        m_device->set_bandwidth(m_config.bandwidth);
+    }
     else if (parameter == "freq") {
         ss >> m_config.frequency;
         m_device->tune(m_config.lo_offset, m_config.frequency);
@@ -415,6 +420,9 @@ const string SDR::get_parameter(const string& parameter) const
     else if (parameter == "rxgain") {
         ss << m_config.rxgain;
     }
+    else if (parameter == "bandwidth") {
+        ss << m_config.bandwidth;
+    }
     else if (parameter == "freq") {
         ss << m_config.frequency;
     }
diff --git a/src/output/SDRDevice.h b/src/output/SDRDevice.h
index 7ab4bc8..d84ebf9 100644
--- a/src/output/SDRDevice.h
+++ b/src/output/SDRDevice.h
@@ -63,7 +63,9 @@ struct SDRDeviceConfig {
     double txgain = 0.0;
     double rxgain = 0.0;
     bool enableSync = false;
+    double bandwidth = 0.0;
     unsigned upsample = 1;
+
     // When working with timestamps, mute the frames that
     // do not have a timestamp
     bool muteNoTimestamps = false;
@@ -124,6 +126,8 @@ class SDRDevice {
         virtual double get_real_secs(void) const = 0;
         virtual void set_rxgain(double rxgain) = 0;
         virtual double get_rxgain(void) const = 0;
+        virtual void set_bandwidth(double bandwidth) = 0;
+        virtual double get_bandwidth(void) const = 0;
         virtual size_t receive_frame(
                 complexf *buf,
                 size_t num_samples,
diff --git a/src/output/Soapy.cpp b/src/output/Soapy.cpp
index 8c84b84..4846279 100644
--- a/src/output/Soapy.cpp
+++ b/src/output/Soapy.cpp
@@ -2,7 +2,7 @@
    Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Her Majesty the
    Queen in Right of Canada (Communications Research Center Canada)
 
-   Copyright (C) 2018
+   Copyright (C) 2019
    Matthias P. Braendli, matthias.braendli@mpb.li
 
     http://opendigitalradio.org
@@ -90,15 +90,23 @@ Soapy::Soapy(SDRDeviceConfig& config) :
         std::fixed << std::setprecision(3) <<
         m_conf.frequency / 1000.0 << " kHz.";
 
+    if (m_conf.bandwidth > 0) {
+        m_device->setBandwidth(SOAPY_SDR_TX, 0, m_conf.bandwidth);
+        m_device->setBandwidth(SOAPY_SDR_RX, 0, m_conf.bandwidth);
+        etiLog.level(info) << "SoapySDR:Actual TX bandwidth: " <<
+            std::fixed << std::setprecision(2) <<
+            m_device->getBandwidth(SOAPY_SDR_TX, 0);
+    }
+
     m_device->setGain(SOAPY_SDR_TX, 0, m_conf.txgain);
-    etiLog.level(info) << "SoapySDR:Actual tx gain: " <<
+    etiLog.level(info) << "SoapySDR:Actual TX gain: " <<
         std::fixed << std::setprecision(2) <<
         m_device->getGain(SOAPY_SDR_TX, 0);
 
     if (not m_conf.tx_antenna.empty()) {
         m_device->setAntenna(SOAPY_SDR_TX, 0, m_conf.tx_antenna);
     }
-    etiLog.level(info) << "SoapySDR:Actual tx antenna: " <<
+    etiLog.level(info) << "SoapySDR:Actual TX antenna: " <<
         m_device->getAntenna(SOAPY_SDR_TX, 0);
 
     if (m_device->hasHardwareTime()) {
@@ -157,6 +165,20 @@ double Soapy::get_txgain(void) const
     return m_device->getGain(SOAPY_SDR_TX, 0);
 }
 
+void Soapy::set_bandwidth(double bandwidth)
+{
+    m_conf.bandwidth = bandwidth;
+    if (not m_device) throw runtime_error("Soapy device not set up");
+    m_device->setBandwidth(SOAPY_SDR_TX, 0, m_conf.bandwidth);
+    m_device->setBandwidth(SOAPY_SDR_RX, 0, m_conf.bandwidth);
+}
+
+double Soapy::get_bandwidth(void) const
+{
+    if (not m_device) throw runtime_error("Soapy device not set up");
+    return m_device->getBandwidth(SOAPY_SDR_TX, 0);
+}
+
 SDRDevice::RunStatistics Soapy::get_run_statistics(void) const
 {
     RunStatistics rs;
diff --git a/src/output/Soapy.h b/src/output/Soapy.h
index 9feb0b8..4ee53ca 100644
--- a/src/output/Soapy.h
+++ b/src/output/Soapy.h
@@ -2,7 +2,7 @@
    Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Her Majesty the
    Queen in Right of Canada (Communications Research Center Canada)
 
-   Copyright (C) 2018
+   Copyright (C) 2019
    Matthias P. Braendli, matthias.braendli@mpb.li
 
     http://opendigitalradio.org
@@ -63,6 +63,8 @@ class Soapy : public Output::SDRDevice
         virtual double get_tx_freq(void) const override;
         virtual void set_txgain(double txgain) override;
         virtual double get_txgain(void) const override;
+        virtual void set_bandwidth(double bandwidth) override;
+        virtual double get_bandwidth(void) const override;
         virtual void transmit_frame(const struct FrameData& frame) override;
         virtual RunStatistics get_run_statistics(void) const override;
         virtual double get_real_secs(void) const override;
diff --git a/src/output/UHD.cpp b/src/output/UHD.cpp
index c6c500b..3cf5aef 100644
--- a/src/output/UHD.cpp
+++ b/src/output/UHD.cpp
@@ -2,7 +2,7 @@
    Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Her Majesty the
    Queen in Right of Canada (Communications Research Center Canada)
 
-   Copyright (C) 2018
+   Copyright (C) 2019
    Matthias P. Braendli, matthias.braendli@mpb.li
 
     http://opendigitalradio.org
@@ -193,6 +193,15 @@ UHD::UHD(SDRDeviceConfig& config) :
         throw std::runtime_error("Cannot set USRP sample rate. Aborted.");
     }
 
+    if (m_conf.bandwidth > 0) {
+        m_usrp->set_tx_bandwidth(m_conf.bandwidth);
+        m_usrp->set_rx_bandwidth(m_conf.bandwidth);
+
+        etiLog.level(info) << "OutputUHD:Actual TX bandwidth: " <<
+            std::fixed << std::setprecision(2) <<
+            m_usrp->get_tx_bandwidth();
+    }
+
     tune(m_conf.lo_offset, m_conf.frequency);
 
     m_conf.frequency = m_usrp->get_tx_freq();
@@ -294,6 +303,18 @@ double UHD::get_txgain(void) const
     return m_usrp->get_tx_gain();
 }
 
+void UHD::set_bandwidth(double bandwidth)
+{
+    m_usrp->set_tx_bandwidth(bandwidth);
+    m_usrp->set_rx_bandwidth(bandwidth);
+    m_conf.bandwidth = m_usrp->get_tx_bandwidth();
+}
+
+double UHD::get_bandwidth(void) const
+{
+    return m_usrp->get_tx_bandwidth();
+}
+
 void UHD::transmit_frame(const struct FrameData& frame)
 {
     const double tx_timeout = 20.0;
@@ -338,8 +359,7 @@ void UHD::transmit_frame(const struct FrameData& frame)
 
         num_acc_samps += num_tx_samps;
 
-        md_tx.time_spec = md_tx.time_spec +
-            uhd::time_spec_t(0, num_tx_samps/m_conf.sampleRate);
+        md_tx.time_spec += uhd::time_spec_t(0, num_tx_samps/m_conf.sampleRate);
 
         if (num_tx_samps == 0) {
             etiLog.log(warn,
diff --git a/src/output/UHD.h b/src/output/UHD.h
index f42b6e8..29867fb 100644
--- a/src/output/UHD.h
+++ b/src/output/UHD.h
@@ -2,7 +2,7 @@
    Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Her Majesty the
    Queen in Right of Canada (Communications Research Center Canada)
 
-   Copyright (C) 2018
+   Copyright (C) 2019
    Matthias P. Braendli, matthias.braendli@mpb.li
 
     http://opendigitalradio.org
@@ -77,6 +77,8 @@ class UHD : public Output::SDRDevice
         virtual double get_tx_freq(void) const override;
         virtual void set_txgain(double txgain) override;
         virtual double get_txgain(void) const override;
+        virtual void set_bandwidth(double bandwidth) override;
+        virtual double get_bandwidth(void) const override;
         virtual void transmit_frame(const struct FrameData& frame) override;
         virtual RunStatistics get_run_statistics(void) const override;
         virtual double get_real_secs(void) const override;