]> sigrok.org Git - pulseview.git/blobdiff - pv/data/decoder.cpp
Adjust to libsigrokdecode API changes
[pulseview.git] / pv / data / decoder.cpp
index 65fc8fff7c858312fa77a902d8df9d33ea3acddd..9e7145f34fe4ffd2c715f141d9701dfb4b0b7a38 100644 (file)
@@ -22,6 +22,8 @@
 
 #include <boost/thread/thread.hpp>
 
+#include <stdexcept>
+
 #include <QDebug>
 
 #include "decoder.h"
@@ -41,17 +43,18 @@ const double Decoder::DecodeMargin = 1.0;
 const double Decoder::DecodeThreshold = 0.2;
 const int64_t Decoder::DecodeChunkLength = 4096;
 
+mutex Decoder::_global_decode_mutex;
+
 Decoder::Decoder(const srd_decoder *const dec,
        std::map<const srd_probe*,
-               boost::shared_ptr<pv::view::Signal> > probes,
+               boost::shared_ptr<pv::view::LogicSignal> > probes,
        GHashTable *options) :
        _decoder(dec),
        _probes(probes),
-       _options(options),
-       _session(NULL),
-       _decoder_inst(NULL)
+       _options(options)
 {
        init_decoder();
+
        begin_decode();
 }
 
@@ -71,10 +74,16 @@ const srd_decoder* Decoder::get_decoder() const
 const vector< shared_ptr<view::decode::Annotation> >
        Decoder::annotations() const
 {
-       lock_guard<mutex> lock(_annotations_mutex);
+       lock_guard<mutex> lock(_mutex);
        return _annotations;
 }
 
+QString Decoder::error_message()
+{
+       lock_guard<mutex> lock(_mutex);
+       return _error_message;
+}
+
 void Decoder::begin_decode()
 {
        _decode_thread.interrupt();
@@ -86,12 +95,9 @@ void Decoder::begin_decode()
        // We get the logic data of the first probe in the list.
        // This works because we are currently assuming all
        // LogicSignals have the same data/snapshot
-       shared_ptr<pv::view::Signal> sig = (*_probes.begin()).second;
+       shared_ptr<pv::view::LogicSignal> sig = (*_probes.begin()).second;
        assert(sig);
-       const pv::view::LogicSignal *const l =
-               dynamic_cast<pv::view::LogicSignal*>(sig.get());
-       assert(l);
-       shared_ptr<data::Logic> data = l->data();
+       shared_ptr<data::Logic> data = sig->data();
 
        _decode_thread = boost::thread(&Decoder::decode_proc, this,
                data);
@@ -112,42 +118,18 @@ void Decoder::init_decoder()
                        shared_ptr<pv::data::Logic> data(
                                logic_signal->data());
                        if (data) {
-                               _samplerate = data->get_samplerate();
                                _start_time = data->get_start_time();
+                               _samplerate = data->get_samplerate();
+                               if (_samplerate == 0.0)
+                                       _samplerate = 1.0;
                        }
                }
        }
-
-       srd_session_new(&_session);
-       assert(_session);
-
-       _decoder_inst = srd_inst_new(_session, _decoder->id, _options);
-       if(!_decoder_inst) {
-               qDebug() << "Failed to initialise decoder";
-               return;
-       }
-
-       _decoder_inst->data_samplerate = _samplerate;
-
-       GHashTable *probes = g_hash_table_new_full(g_str_hash,
-               g_str_equal, g_free, (GDestroyNotify)g_variant_unref);
-
-       for(map<const srd_probe*, shared_ptr<view::Signal> >::
-               const_iterator i = _probes.begin();
-               i != _probes.end(); i++)
-       {
-               shared_ptr<view::Signal> signal((*i).second);
-               GVariant *const gvar = g_variant_new_int32(
-                       signal->probe()->index);
-               g_variant_ref_sink(gvar);
-               g_hash_table_insert(probes, (*i).first->id, gvar);
-       }
-
-       srd_inst_probe_set_all(_decoder_inst, probes);
 }
 
 void Decoder::decode_proc(shared_ptr<data::Logic> data)
 {
+       srd_session *session;
        uint8_t chunk[DecodeChunkLength];
 
        assert(data);
@@ -162,36 +144,64 @@ void Decoder::decode_proc(shared_ptr<data::Logic> data)
        const shared_ptr<pv::data::LogicSnapshot> &snapshot =
                snapshots.front();
        const int64_t sample_count = snapshot->get_sample_count() - 1;
-       double samplerate = data->get_samplerate();
-
-       // Show sample rate as 1Hz when it is unknown
-       if (samplerate == 0.0)
-               samplerate = 1.0;
 
-       srd_session_config_set(_session, SRD_CONF_NUM_PROBES,
-               g_variant_new_uint64(_probes.size()));
-       srd_session_config_set(_session, SRD_CONF_UNITSIZE,
-               g_variant_new_uint64(snapshot->unit_size()));
-       srd_session_config_set(_session, SRD_CONF_SAMPLERATE,
-               g_variant_new_uint64((uint64_t)samplerate));
+       // Create the session
+       srd_session_new(&session);
+       assert(session);
 
-       srd_session_start(_session);
+       srd_session_metadata_set(session, SRD_CONF_SAMPLERATE,
+               g_variant_new_uint64((uint64_t)_samplerate));
 
-       srd_pd_output_callback_add(_session, SRD_OUTPUT_ANN,
+       srd_pd_output_callback_add(session, SRD_OUTPUT_ANN,
                Decoder::annotation_callback, this);
 
+       // Create the decoder instance
+       srd_decoder_inst *const decoder_inst = srd_inst_new(
+               session, _decoder->id, _options);
+       if(!decoder_inst) {
+               _error_message = tr("Failed to initialise decoder");
+               return;
+       }
+
+       // Setup the probes
+       GHashTable *const probes = g_hash_table_new_full(g_str_hash,
+               g_str_equal, g_free, (GDestroyNotify)g_variant_unref);
+
+       for(map<const srd_probe*, shared_ptr<view::LogicSignal> >::
+               const_iterator i = _probes.begin();
+               i != _probes.end(); i++)
+       {
+               shared_ptr<view::Signal> signal((*i).second);
+               GVariant *const gvar = g_variant_new_int32(
+                       signal->probe()->index);
+               g_variant_ref_sink(gvar);
+               g_hash_table_insert(probes, (*i).first->id, gvar);
+       }
+
+       srd_inst_probe_set_all(decoder_inst, probes);
+
+       // Start the session
+       srd_session_start(session);
+
        for (int64_t i = 0;
                !this_thread::interruption_requested() && i < sample_count;
                i += DecodeChunkLength)
        {
+               lock_guard<mutex> decode_lock(_global_decode_mutex);
+
                const int64_t chunk_end = min(
                        i + DecodeChunkLength, sample_count);
                snapshot->get_samples(chunk, i, chunk_end);
 
-               if (srd_session_send(_session, i, chunk, chunk_end - i) !=
-                       SRD_OK)
+               if (srd_session_send(session, i, i + sample_count,
+                               chunk, chunk_end - i) != SRD_OK) {
+                       _error_message = tr("Failed to initialise decoder");
                        break;
+               }
        }
+
+       // Destroy the session
+       srd_session_destroy(session);
 }
 
 void Decoder::annotation_callback(srd_proto_data *pdata, void *decoder)
@@ -204,7 +214,7 @@ void Decoder::annotation_callback(srd_proto_data *pdata, void *decoder)
        Decoder *const d = (Decoder*)decoder;
 
        shared_ptr<Annotation> a(new Annotation(pdata));
-       lock_guard<mutex> lock(d->_annotations_mutex);
+       lock_guard<mutex> lock(d->_mutex);
        d->_annotations.push_back(a);
 
        d->new_decode_data();