X-Git-Url: http://sigrok.org/gitweb/?a=blobdiff_plain;ds=sidebyside;f=pv%2Fdata%2Fdecode%2Fdecoder.cpp;h=9792ace43412ad6d01bb44596d3676641b24f9cc;hb=0672b79b535b5044d703515d1ad83bfd03d288bc;hp=bd1d307fb5d00006bb4aa01d611d37467c681b82;hpb=73a25a6e488f1813c1cd12da085a16e4f91ed4da;p=pulseview.git
diff --git a/pv/data/decode/decoder.cpp b/pv/data/decode/decoder.cpp
index bd1d307f..9792ace4 100644
--- a/pv/data/decode/decoder.cpp
+++ b/pv/data/decode/decoder.cpp
@@ -14,22 +14,22 @@
* 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, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ * along with this program; if not, see .
*/
#include
+#include
+
#include
#include
#include "decoder.hpp"
-#include
+#include
+#include
-using std::set;
using std::map;
-using std::shared_ptr;
using std::string;
namespace pv {
@@ -37,9 +37,52 @@ namespace data {
namespace decode {
Decoder::Decoder(const srd_decoder *const dec) :
- decoder_(dec),
- shown_(true)
+ srd_decoder_(dec),
+ visible_(true),
+ decoder_inst_(nullptr)
{
+ // Query the annotation output classes
+ uint32_t i = 0;
+ for (GSList *l = dec->annotations; l; l = l->next) {
+ char **ann_class = (char**)l->data;
+ char *name = ann_class[0];
+ char *desc = ann_class[1];
+ ann_classes_.push_back({i++, name, desc, nullptr, true}); // Visible by default
+ }
+
+ // Query the binary output classes
+ i = 0;
+ for (GSList *l = dec->binary; l; l = l->next) {
+ char **bin_class = (char**)l->data;
+ char *name = bin_class[0];
+ char *desc = bin_class[1];
+ bin_classes_.push_back({i++, name, desc});
+ }
+
+ // Query the annotation rows and reference them by the classes that use them
+ uint32_t row_count = 0;
+ for (const GSList *rl = srd_decoder_->annotation_rows; rl; rl = rl->next)
+ row_count++;
+ rows_.reserve(row_count);
+
+ i = 0;
+ for (const GSList *rl = srd_decoder_->annotation_rows; rl; rl = rl->next) {
+ const srd_decoder_annotation_row *const srd_row = (srd_decoder_annotation_row *)rl->data;
+ assert(srd_row);
+ rows_.push_back({i++, this, srd_row});
+
+ // FIXME PV can crash from .at() if a PD's ann classes are defined incorrectly
+ for (const GSList *cl = srd_row->ann_classes; cl; cl = cl->next)
+ ann_classes_.at((size_t)cl->data).row = &(rows_.back());
+ }
+
+ if (rows_.empty()) {
+ // Make sure there is a row for PDs without row declarations
+ rows_.push_back({0, this});
+
+ for (AnnotationClass& c : ann_classes_)
+ c.row = &(rows_.back());
+ }
}
Decoder::~Decoder()
@@ -48,34 +91,37 @@ Decoder::~Decoder()
g_variant_unref(option.second);
}
-const srd_decoder* Decoder::decoder() const
+const srd_decoder* Decoder::get_srd_decoder() const
+{
+ return srd_decoder_;
+}
+
+const char* Decoder::name() const
{
- return decoder_;
+ return srd_decoder_->name;
}
-bool Decoder::shown() const
+bool Decoder::visible() const
{
- return shown_;
+ return visible_;
}
-void Decoder::show(bool show)
+void Decoder::set_visible(bool visible)
{
- shown_ = show;
+ visible_ = visible;
}
-const map >&
-Decoder::channels() const
+const vector& Decoder::channels() const
{
return channels_;
}
-void Decoder::set_channels(std::map > channels)
+void Decoder::set_channels(vector channels)
{
channels_ = channels;
}
-const std::map& Decoder::options() const
+const map& Decoder::options() const
{
return options_;
}
@@ -85,33 +131,39 @@ void Decoder::set_option(const char *id, GVariant *value)
assert(value);
g_variant_ref(value);
options_[id] = value;
+
+ // If we have a decoder instance, apply option value immediately
+ apply_all_options();
}
-bool Decoder::have_required_channels() const
+void Decoder::apply_all_options()
{
- for (GSList *l = decoder_->channels; l; l = l->next) {
- const srd_channel *const pdch = (const srd_channel*)l->data;
- assert(pdch);
- if (channels_.find(pdch) == channels_.end())
- return false;
- }
+ if (decoder_inst_) {
+ GHashTable *const opt_hash = g_hash_table_new_full(g_str_hash,
+ g_str_equal, g_free, (GDestroyNotify)g_variant_unref);
- return true;
+ for (const auto& option : options_) {
+ GVariant *const value = option.second;
+ g_variant_ref(value);
+ g_hash_table_replace(opt_hash, (void*)g_strdup(
+ option.first.c_str()), value);
+ }
+
+ srd_inst_option_set(decoder_inst_, opt_hash);
+ g_hash_table_destroy(opt_hash);
+ }
}
-set< shared_ptr > Decoder::get_data()
+bool Decoder::have_required_channels() const
{
- set< shared_ptr > data;
- for (const auto& channel : channels_) {
- shared_ptr signal(channel.second);
- assert(signal);
- data.insert(signal->logic_data());
- }
+ for (DecodeChannel *ch : channels_)
+ if (!ch->assigned_signal && !ch->is_optional)
+ return false;
- return data;
+ return true;
}
-srd_decoder_inst* Decoder::create_decoder_inst(srd_session *session) const
+srd_decoder_inst* Decoder::create_decoder_inst(srd_session *session)
{
GHashTable *const opt_hash = g_hash_table_new_full(g_str_hash,
g_str_equal, g_free, (GDestroyNotify)g_variant_unref);
@@ -123,30 +175,105 @@ srd_decoder_inst* Decoder::create_decoder_inst(srd_session *session) const
option.first.c_str()), value);
}
- srd_decoder_inst *const decoder_inst = srd_inst_new(
- session, decoder_->id, opt_hash);
+ if (decoder_inst_)
+ qDebug() << "WARNING: previous decoder instance" << decoder_inst_ << "exists";
+
+ decoder_inst_ = srd_inst_new(session, srd_decoder_->id, opt_hash);
g_hash_table_destroy(opt_hash);
- if (!decoder_inst)
+ if (!decoder_inst_)
return nullptr;
// Setup the channels
+ GArray *const init_pin_states = g_array_sized_new(false, true,
+ sizeof(uint8_t), channels_.size());
+
+ g_array_set_size(init_pin_states, channels_.size());
+
GHashTable *const channels = g_hash_table_new_full(g_str_hash,
g_str_equal, g_free, (GDestroyNotify)g_variant_unref);
- for (const auto& channel : channels_) {
- shared_ptr signal(channel.second);
- GVariant *const gvar = g_variant_new_int32(
- signal->base()->index());
+ for (DecodeChannel *ch : channels_) {
+ if (!ch->assigned_signal)
+ continue;
+
+ init_pin_states->data[ch->id] = ch->initial_pin_state;
+
+ GVariant *const gvar = g_variant_new_int32(ch->bit_id); // bit_id = bit position
g_variant_ref_sink(gvar);
- g_hash_table_insert(channels, channel.first->id, gvar);
+ // key is channel name (pdch->id), value is bit position in each sample (gvar)
+ g_hash_table_insert(channels, ch->pdch_->id, gvar);
}
- srd_inst_channel_set_all(decoder_inst, channels);
+ srd_inst_channel_set_all(decoder_inst_, channels);
+
+ srd_inst_initial_pins_set_all(decoder_inst_, init_pin_states);
+ g_array_free(init_pin_states, true);
+
+ return decoder_inst_;
+}
- return decoder_inst;
+void Decoder::invalidate_decoder_inst()
+{
+ decoder_inst_ = nullptr;
+}
+
+vector Decoder::get_rows()
+{
+ vector result;
+
+ for (Row& row : rows_)
+ result.push_back(&row);
+
+ return result;
+}
+
+Row* Decoder::get_row_by_id(size_t id)
+{
+ if (id > rows_.size())
+ return nullptr;
+
+ return &(rows_[id]);
+}
+
+vector Decoder::ann_classes() const
+{
+ vector result;
+
+ for (const AnnotationClass& c : ann_classes_)
+ result.push_back(&c);
+
+ return result;
+}
+
+vector Decoder::ann_classes()
+{
+ vector result;
+
+ for (AnnotationClass& c : ann_classes_)
+ result.push_back(&c);
+
+ return result;
+}
+
+AnnotationClass* Decoder::get_ann_class_by_id(size_t id)
+{
+ if (id >= ann_classes_.size())
+ return nullptr;
+
+ return &(ann_classes_[id]);
+}
+
+uint32_t Decoder::get_binary_class_count() const
+{
+ return bin_classes_.size();
+}
+
+const DecodeBinaryClassInfo* Decoder::get_binary_class(uint32_t id) const
+{
+ return &(bin_classes_.at(id));
}
-} // decode
-} // data
-} // pv
+} // namespace decode
+} // namespace data
+} // namespace pv