X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fsigsession.cpp;h=94867959f184fcb547b40463e84f97fccbfea5f6;hp=e8585c5f84db1465a8a482cbe6266eabc60e270f;hb=4d5d5d6aeb7d936fb6b939f6c6531b3f054dac1f;hpb=69dd2b030f0bf289fe3132cf3c0f3725c578a916

diff --git a/pv/sigsession.cpp b/pv/sigsession.cpp
index e8585c5f..94867959 100644
--- a/pv/sigsession.cpp
+++ b/pv/sigsession.cpp
@@ -20,14 +20,17 @@
 
 #include "sigsession.h"
 
-#include "logicdata.h"
-#include "logicdatasnapshot.h"
+#include "data/analog.h"
+#include "data/analogsnapshot.h"
+#include "data/logic.h"
+#include "data/logicsnapshot.h"
+#include "view/analogsignal.h"
 #include "view/logicsignal.h"
 
-#include <QDebug>
-
 #include <assert.h>
 
+#include <QDebug>
+
 using namespace boost;
 using namespace std;
 
@@ -36,7 +39,8 @@ namespace pv {
 // TODO: This should not be necessary
 SigSession* SigSession::_session = NULL;
 
-SigSession::SigSession()
+SigSession::SigSession() :
+	_capture_state(Stopped)
 {
 	// TODO: This should not be necessary
 	_session = this;
@@ -44,7 +48,9 @@ SigSession::SigSession()
 
 SigSession::~SigSession()
 {
-	if(_sampling_thread.get())
+	stop_capture();
+
+	if (_sampling_thread.get())
 		_sampling_thread->join();
 	_sampling_thread.reset();
 
@@ -52,152 +58,327 @@ SigSession::~SigSession()
 	_session = NULL;
 }
 
-void SigSession::load_file(const std::string &name)
+void SigSession::load_file(const string &name,
+	function<void (const QString)> error_handler)
 {
-	if (sr_session_load(name.c_str()) == SR_OK) {
-		/* sigrok session file */
-		sr_session_datafeed_callback_add(data_feed_in_proc);
-		sr_session_start();
-		sr_session_run();
-		sr_session_stop();
-	}
+	stop_capture();
+	_sampling_thread.reset(new boost::thread(
+		&SigSession::load_thread_proc, this, name,
+		error_handler));
+}
+
+SigSession::capture_state SigSession::get_capture_state() const
+{
+	lock_guard<mutex> lock(_sampling_mutex);
+	return _capture_state;
 }
 
 void SigSession::start_capture(struct sr_dev_inst *sdi,
-	uint64_t record_length, uint64_t sample_rate)
+	uint64_t record_length,
+	function<void (const QString)> error_handler)
 {
-	// Check sampling isn't already active
-	if(_sampling_thread.get())
-		_sampling_thread->join();
+	stop_capture();
+
+	// Check that at least one probe is enabled
+	const GSList *l;
+	for (l = sdi->probes; l; l = l->next) {
+		sr_probe *const probe = (sr_probe*)l->data;
+		assert(probe);
+		if (probe->enabled)
+			break;
+	}
+
+	if (!l) {
+		error_handler(tr("No probes enabled."));
+		return;
+	}
 
+	// Begin the session
 	_sampling_thread.reset(new boost::thread(
 		&SigSession::sample_thread_proc, this, sdi,
-		record_length, sample_rate));
+		record_length, error_handler));
 }
 
-vector< shared_ptr<view::Signal> >& SigSession::get_signals()
+void SigSession::stop_capture()
 {
+	if (get_capture_state() == Stopped)
+		return;
+
+	sr_session_stop();
+
+	// Check that sampling stopped
+	if (_sampling_thread.get())
+		_sampling_thread->join();
+	_sampling_thread.reset();
+}
+
+vector< shared_ptr<view::Signal> > SigSession::get_signals()
+{
+	lock_guard<mutex> lock(_signals_mutex);
 	return _signals;
 }
 
-boost::shared_ptr<LogicData> SigSession::get_data()
+boost::shared_ptr<data::Logic> SigSession::get_data()
 {
 	return _logic_data;
 }
 
-void SigSession::sample_thread_proc(struct sr_dev_inst *sdi,
-	uint64_t record_length, uint64_t sample_rate)
+void SigSession::set_capture_state(capture_state state)
 {
-	sr_session_new();
+	lock_guard<mutex> lock(_sampling_mutex);
+	_capture_state = state;
+	capture_state_changed(state);
+}
+
+void SigSession::load_thread_proc(const string name,
+	function<void (const QString)> error_handler)
+{
+	if (sr_session_load(name.c_str()) != SR_OK) {
+		error_handler(tr("Failed to load file."));
+		return;
+	}
+
 	sr_session_datafeed_callback_add(data_feed_in_proc);
 
-	if (sr_session_dev_add(sdi) != SR_OK) {
-		qDebug() << "Failed to use device.";
-		sr_session_destroy();
+	if (sr_session_start() != SR_OK) {
+		error_handler(tr("Failed to start session."));
 		return;
 	}
 
-	if (sr_dev_config_set(sdi, SR_HWCAP_LIMIT_SAMPLES,
-		&record_length) != SR_OK) {
-		qDebug() << "Failed to configure time-based sample limit.";
+	set_capture_state(Running);
+
+	sr_session_run();
+	sr_session_stop();
+
+	set_capture_state(Stopped);
+}
+
+void SigSession::sample_thread_proc(struct sr_dev_inst *sdi,
+	uint64_t record_length,
+	function<void (const QString)> error_handler)
+{
+	assert(sdi);
+	assert(error_handler);
+
+	sr_session_new();
+	sr_session_datafeed_callback_add(data_feed_in_proc);
+
+	if (sr_session_dev_add(sdi) != SR_OK) {
+		error_handler(tr("Failed to use device."));
 		sr_session_destroy();
 		return;
 	}
 
-	if (sr_dev_config_set(sdi, SR_HWCAP_SAMPLERATE,
-		&sample_rate) != SR_OK) {
-		qDebug() << "Failed to configure samplerate.";
+	// Set the sample limit
+	if (sr_config_set(sdi, SR_CONF_LIMIT_SAMPLES,
+		g_variant_new_uint64(record_length)) != SR_OK) {
+		error_handler(tr("Failed to configure "
+			"time-based sample limit."));
 		sr_session_destroy();
 		return;
 	}
 
 	if (sr_session_start() != SR_OK) {
-		qDebug() << "Failed to start session.";
+		error_handler(tr("Failed to start session."));
 		return;
 	}
 
+	set_capture_state(Running);
+
 	sr_session_run();
 	sr_session_destroy();
+
+	set_capture_state(Stopped);
 }
 
-void SigSession::data_feed_in(const struct sr_dev_inst *sdi,
-	struct sr_datafeed_packet *packet)
+void SigSession::feed_in_header(const sr_dev_inst *sdi)
 {
-	using view::LogicSignal;
-
-	assert(sdi);
-	assert(packet);
+	shared_ptr<view::Signal> signal;
+	GVariant *gvar;
+	uint64_t sample_rate = 0;
+	unsigned int logic_probe_count = 0;
+	unsigned int analog_probe_count = 0;
+
+	// Detect what data types we will receive
+	for (const GSList *l = sdi->probes; l; l = l->next) {
+		const sr_probe *const probe = (const sr_probe *)l->data;
+		if (!probe->enabled)
+			continue;
+
+		switch(probe->type) {
+		case SR_PROBE_LOGIC:
+			logic_probe_count++;
+			break;
 
-	switch (packet->type) {
-	case SR_DF_HEADER:
-	{
-		lock_guard<mutex> lock(_data_mutex);
-		_signals.clear();
-		break;
+		case SR_PROBE_ANALOG:
+			analog_probe_count++;
+			break;
+		}
 	}
 
-	case SR_DF_META_LOGIC:
+	// Read out the sample rate
+	assert(sdi->driver);
+
+	const int ret = sr_config_get(sdi->driver, SR_CONF_SAMPLERATE,
+		&gvar, sdi);
+	assert(ret == SR_OK);
+	sample_rate = g_variant_get_uint64(gvar);
+	g_variant_unref(gvar);
+
+	// Create data containers for the coming data snapshots
 	{
-		assert(packet->payload);
+		lock_guard<mutex> data_lock(_data_mutex);
 
-		lock_guard<mutex> lock(_data_mutex);
+		if (logic_probe_count != 0) {
+			_logic_data.reset(new data::Logic(
+				logic_probe_count, sample_rate));
+			assert(_logic_data);
+		}
 
-		const sr_datafeed_meta_logic &meta_logic =
-			*(sr_datafeed_meta_logic*)packet->payload;
+		if (analog_probe_count != 0) {
+			_analog_data.reset(new data::Analog(sample_rate));
+			assert(_analog_data);
+		}
+	}
 
-		// Create an empty LogiData for coming data snapshots
-		_logic_data.reset(new LogicData(meta_logic));
-		assert(_logic_data);
-		if(!_logic_data)
-			break;
+	// Make the logic probe list
+	{
+		lock_guard<mutex> lock(_signals_mutex);
 
-		// Add the signals
-		for (int i = 0; i < meta_logic.num_probes; i++)
-		{
+		_signals.clear();
+
+		for (const GSList *l = sdi->probes; l; l = l->next) {
 			const sr_probe *const probe =
-				(const sr_probe*)g_slist_nth_data(
-					sdi->probes, i);
-			if(probe->enabled)
-			{
-				shared_ptr<LogicSignal> signal(
-					new LogicSignal(probe->name,
-						_logic_data,
-						probe->index));
-				_signals.push_back(signal);
+				(const sr_probe *)l->data;
+			assert(probe);
+			if (!probe->enabled)
+				continue;
+
+			switch(probe->type) {
+			case SR_PROBE_LOGIC:
+				signal = shared_ptr<view::Signal>(
+					new view::LogicSignal(probe->name,
+						_logic_data, probe->index));
+				break;
+
+			case SR_PROBE_ANALOG:
+				signal = shared_ptr<view::Signal>(
+					new view::AnalogSignal(probe->name,
+						_analog_data, probe->index));
+				break;
 			}
+
+			_signals.push_back(signal);
 		}
 
 		signals_changed();
-		break;
 	}
+}
 
-	case SR_DF_LOGIC:
+void SigSession::feed_in_meta(const sr_dev_inst *sdi,
+	const sr_datafeed_meta &meta)
+{
+	(void)sdi;
+
+	for (const GSList *l = meta.config; l; l = l->next) {
+		const sr_config *const src = (const sr_config*)l->data;
+		switch (src->key) {
+		case SR_CONF_SAMPLERATE:
+			/// @todo handle samplerate changes
+			/// samplerate = (uint64_t *)src->value;
+			break;
+		default:
+			// Unknown metadata is not an error.
+			break;
+		}
+	}
+}
+
+void SigSession::feed_in_logic(const sr_datafeed_logic &logic)
+{
+	lock_guard<mutex> lock(_data_mutex);
+
+	if (!_logic_data)
+	{
+		qDebug() << "Unexpected logic packet";
+		return;
+	}
+
+	if (!_cur_logic_snapshot)
+	{
+		// Create a new data snapshot
+		_cur_logic_snapshot = shared_ptr<data::LogicSnapshot>(
+			new data::LogicSnapshot(logic));
+		_logic_data->push_snapshot(_cur_logic_snapshot);
+	}
+	else
+	{
+		// Append to the existing data snapshot
+		_cur_logic_snapshot->append_payload(logic);
+	}
+
+	data_updated();
+}
+
+void SigSession::feed_in_analog(const sr_datafeed_analog &analog)
+{
+	lock_guard<mutex> lock(_data_mutex);
+
+	if(!_analog_data)
+	{
+		qDebug() << "Unexpected analog packet";
+		return;	// This analog packet was not expected.
+	}
+
+	if (!_cur_analog_snapshot)
+	{
+		// Create a new data snapshot
+		_cur_analog_snapshot = shared_ptr<data::AnalogSnapshot>(
+			new data::AnalogSnapshot(analog));
+		_analog_data->push_snapshot(_cur_analog_snapshot);
+	}
+	else
 	{
-		lock_guard<mutex> lock(_data_mutex);
+		// Append to the existing data snapshot
+		_cur_analog_snapshot->append_payload(analog);
+	}
+
+	data_updated();
+}
+
+void SigSession::data_feed_in(const struct sr_dev_inst *sdi,
+	const struct sr_datafeed_packet *packet)
+{
+	assert(sdi);
+	assert(packet);
+
+	switch (packet->type) {
+	case SR_DF_HEADER:
+		feed_in_header(sdi);
+		break;
+
+	case SR_DF_META:
 		assert(packet->payload);
-		if(!_cur_logic_snapshot)
-		{
-			// Create a new data snapshot
-			_cur_logic_snapshot = shared_ptr<LogicDataSnapshot>(
-				new LogicDataSnapshot(
-				*(sr_datafeed_logic*)packet->payload));
-			_logic_data->push_snapshot(_cur_logic_snapshot);
-		}
-		else
-		{
-			// Append to the existing data snapshot
-			_cur_logic_snapshot->append_payload(
-				*(sr_datafeed_logic*)packet->payload);
-		}
+		feed_in_meta(sdi,
+			*(const sr_datafeed_meta*)packet->payload);
+		break;
+
+	case SR_DF_LOGIC:
+		assert(packet->payload);
+		feed_in_logic(*(const sr_datafeed_logic*)packet->payload);
+		break;
 
+	case SR_DF_ANALOG:
+		assert(packet->payload);
+		feed_in_analog(*(const sr_datafeed_analog*)packet->payload);
 		break;
-	}
 
 	case SR_DF_END:
 	{
 		{
 			lock_guard<mutex> lock(_data_mutex);
 			_cur_logic_snapshot.reset();
+			_cur_analog_snapshot.reset();
 		}
 		data_updated();
 		break;
@@ -206,7 +387,7 @@ void SigSession::data_feed_in(const struct sr_dev_inst *sdi,
 }
 
 void SigSession::data_feed_in_proc(const struct sr_dev_inst *sdi,
-	struct sr_datafeed_packet *packet)
+	const struct sr_datafeed_packet *packet)
 {
 	assert(_session);
 	_session->data_feed_in(sdi, packet);