diff --git a/src/sigrok.c b/src/sigrok.c
index 7e2fa6c1b29ca2990dd5362d4164b69db67bc1df..eecb54c8c5fd8c1ad8a607358925d1510cdbe233 100644 (file)
--- a/src/sigrok.c
+++ b/src/sigrok.c
/*
* collectd - src/sigrok.c
/*
* collectd - src/sigrok.c
- * Copyright (C) 2013 Bert Vermeulen <bert@biot.com>
+ * Copyright (C) 2013 Bert Vermeulen
*
* 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
*
* 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
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authors:
+ * Bert Vermeulen <bert at biot.com>
*/
#include "collectd.h"
*/
#include "collectd.h"
/* Minimum interval between dispatches coming from this plugin. The RRD
* plugin, at least, complains when written to with sub-second intervals.*/
/* Minimum interval between dispatches coming from this plugin. The RRD
* plugin, at least, complains when written to with sub-second intervals.*/
-#define DEFAULT_MIN_DISPATCH_INTERVAL TIME_T_TO_CDTIME_T(1)
+#define DEFAULT_MIN_DISPATCH_INTERVAL TIME_T_TO_CDTIME_T(0)
static pthread_t sr_thread;
static int sr_thread_running = FALSE;
static pthread_t sr_thread;
static int sr_thread_running = FALSE;
if (msg_loglevel <= loglevel) {
vsnprintf(s, 512, format, args);
if (msg_loglevel <= loglevel) {
vsnprintf(s, 512, format, args);
- plugin_log(LOG_INFO, "sigrok: %s", s);
+ plugin_log(LOG_INFO, "sigrok plugin: %s", s);
}
return 0;
}
return 0;
int i;
if (!(cfdev = malloc(sizeof(struct config_device)))) {
int i;
if (!(cfdev = malloc(sizeof(struct config_device)))) {
- ERROR("malloc() failed.");
+ ERROR("sigrok plugin: malloc() failed.");
return -1;
}
memset(cfdev, 0, sizeof(*cfdev));
if (cf_util_get_string(ci, &cfdev->name)) {
free(cfdev);
return -1;
}
memset(cfdev, 0, sizeof(*cfdev));
if (cf_util_get_string(ci, &cfdev->name)) {
free(cfdev);
- WARNING("Invalid device name.");
+ WARNING("sigrok plugin: Invalid device name.");
return -1;
}
cfdev->min_dispatch_interval = DEFAULT_MIN_DISPATCH_INTERVAL;
for (i = 0; i < ci->children_num; i++) {
oconfig_item_t *item = ci->children + i;
return -1;
}
cfdev->min_dispatch_interval = DEFAULT_MIN_DISPATCH_INTERVAL;
for (i = 0; i < ci->children_num; i++) {
oconfig_item_t *item = ci->children + i;
- if (item->values_num != 1) {
- free(cfdev);
- WARNING("Missing value for '%s'.", item->key);
- return -1;
- }
if (!strcasecmp(item->key, "driver"))
cf_util_get_string(item, &cfdev->driver);
else if (!strcasecmp(item->key, "conn"))
cf_util_get_string(item, &cfdev->conn);
else if (!strcasecmp(item->key, "serialcomm"))
cf_util_get_string(item, &cfdev->serialcomm);
if (!strcasecmp(item->key, "driver"))
cf_util_get_string(item, &cfdev->driver);
else if (!strcasecmp(item->key, "conn"))
cf_util_get_string(item, &cfdev->conn);
else if (!strcasecmp(item->key, "serialcomm"))
cf_util_get_string(item, &cfdev->serialcomm);
- else if (!strcasecmp(item->key, "interval"))
+ else if (!strcasecmp(item->key, "minimuminterval"))
cf_util_get_cdtime(item, &cfdev->min_dispatch_interval);
else
cf_util_get_cdtime(item, &cfdev->min_dispatch_interval);
else
- WARNING("Invalid keyword '%s'.", item->key);
+ WARNING("sigrok plugin: Invalid keyword \"%s\".",
+ item->key);
}
config_devices = g_slist_append(config_devices, cfdev);
}
config_devices = g_slist_append(config_devices, cfdev);
static int sigrok_config(oconfig_item_t *ci)
{
static int sigrok_config(oconfig_item_t *ci)
{
- int tmp, i;
+ int i;
for (i = 0; i < ci->children_num; i++) {
oconfig_item_t *item = ci->children + i;
for (i = 0; i < ci->children_num; i++) {
oconfig_item_t *item = ci->children + i;
- if (!strcasecmp(item->key, "loglevel")) {
- if (cf_util_get_int(item, &tmp) || tmp < 0 || tmp > 5) {
- ERROR("Invalid loglevel");
+ if (strcasecmp("LogLevel", item->key) == 0) {
+ int status;
+ int tmp = -1;
+
+ status = cf_util_get_int (item, &tmp);
+ if (status != 0)
+ continue;
+ else if ((tmp < 0) || (tmp > 5)) {
+ ERROR ("sigrok plugin: The \"LogLevel\" "
+ "configuration option expects "
+ "an integer between 0 and 5 "
+ "(inclusive); you provided %i.",
+ tmp);
continue;
}
loglevel = tmp;
} else if (!strcasecmp(item->key, "Device"))
sigrok_config_device(item);
else
continue;
}
loglevel = tmp;
} else if (!strcasecmp(item->key, "Device"))
sigrok_config_device(item);
else
- WARNING("Invalid keyword '%s'.", item->key);
+ WARNING("sigrok plugin: Invalid keyword \"%s\".",
+ item->key);
}
return 0;
}
return 0;
s = "temperature";
else if (analog->mq == SR_MQ_RELATIVE_HUMIDITY)
s = "humidity";
s = "temperature";
else if (analog->mq == SR_MQ_RELATIVE_HUMIDITY)
s = "humidity";
+ else if (analog->mq == SR_MQ_SOUND_PRESSURE_LEVEL)
+ s = "spl";
else
s = "gauge";
else
s = "gauge";
}
cfdev = NULL;
}
}
cfdev = NULL;
}
+
if (!cfdev) {
if (!cfdev) {
- ERROR("Unknown device instance in sigrok driver %s.", sdi->driver->name);
+ ERROR("sigrok plugin: Received data from driver \"%s\" but "
+ "can't find a configuration / device matching "
+ "it.", sdi->driver->name);
return;
}
if (packet->type == SR_DF_END) {
/* TODO: try to restart acquisition after a delay? */
return;
}
if (packet->type == SR_DF_END) {
/* TODO: try to restart acquisition after a delay? */
- WARNING("sigrok: acquisition for '%s' ended.", cfdev->name);
+ WARNING("sigrok plugin: acquisition for \"%s\" ended.",
+ cfdev->name);
return;
}
if (packet->type != SR_DF_ANALOG)
return;
return;
}
if (packet->type != SR_DF_ANALOG)
return;
- if (cdtime() - cfdev->last_dispatch < cfdev->min_dispatch_interval)
+ if ((cfdev->min_dispatch_interval != 0)
+ && ((cdtime() - cfdev->last_dispatch)
+ < cfdev->min_dispatch_interval))
return;
/* Ignore all but the first sample on the first probe. */
return;
/* Ignore all but the first sample on the first probe. */
ssnprintf(vl.plugin_instance, sizeof(vl.plugin_instance),
"%s", cfdev->name);
sstrncpy(vl.type, sigrok_value_type(analog), sizeof(vl.type));
ssnprintf(vl.plugin_instance, sizeof(vl.plugin_instance),
"%s", cfdev->name);
sstrncpy(vl.type, sigrok_value_type(analog), sizeof(vl.type));
- plugin_dispatch_values(&vl);
+ plugin_dispatch_values(&vl);
cfdev->last_dispatch = cdtime();
cfdev->last_dispatch = cdtime();
-
}
}
-static void free_drvopts(struct sr_config *src)
+static void sigrok_free_drvopts(struct sr_config *src)
{
g_variant_unref(src->data);
g_free(src);
{
g_variant_unref(src->data);
g_free(src);
drvopts = g_slist_append(drvopts, src);
}
devlist = sr_driver_scan(drv, drvopts);
drvopts = g_slist_append(drvopts, src);
}
devlist = sr_driver_scan(drv, drvopts);
- g_slist_free_full(drvopts, (GDestroyNotify)free_drvopts);
+ g_slist_free_full(drvopts, (GDestroyNotify)sigrok_free_drvopts);
if (!devlist) {
/* Not an error, but the user should know about it. */
if (!devlist) {
/* Not an error, but the user should know about it. */
- WARNING("No device found for '%s'.", cfdev->name);
+ WARNING("sigrok plugin: No device found for \"%s\".",
+ cfdev->name);
return 0;
}
if (g_slist_length(devlist) > 1) {
return 0;
}
if (g_slist_length(devlist) > 1) {
- INFO("sigrok: %d sigrok devices for device entry '%s': must be 1.",
+ INFO("sigrok plugin: %d sigrok devices for device entry "
+ "\"%s\": must be 1.",
g_slist_length(devlist), cfdev->name);
return -1;
}
g_slist_length(devlist), cfdev->name);
return -1;
}
cfdev->sdi->vendor ? cfdev->sdi->vendor : "",
cfdev->sdi->model ? cfdev->sdi->model : "",
cfdev->sdi->version ? cfdev->sdi->version : "");
cfdev->sdi->vendor ? cfdev->sdi->vendor : "",
cfdev->sdi->model ? cfdev->sdi->model : "",
cfdev->sdi->version ? cfdev->sdi->version : "");
- INFO("sigrok: Device '%s' is a %s", cfdev->name, hwident);
+ INFO("sigrok plugin: Device \"%s\" is a %s", cfdev->name, hwident);
if (sr_dev_open(cfdev->sdi) != SR_OK)
return -1;
if (sr_dev_open(cfdev->sdi) != SR_OK)
return -1;
sr_log_loglevel_set(loglevel);
if ((ret = sr_init(&sr_ctx)) != SR_OK) {
sr_log_loglevel_set(loglevel);
if ((ret = sr_init(&sr_ctx)) != SR_OK) {
- ERROR("Failed to initialize libsigrok: %s.", sr_strerror(ret));
+ ERROR("sigrok plugin: Failed to initialize libsigrok: %s.",
+ sr_strerror(ret));
return NULL;
}
return NULL;
}
}
}
if (!drv) {
}
}
if (!drv) {
- ERROR("sigrok: Unknown driver '%s'.", cfdev->driver);
+ ERROR("sigrok plugin: Unknown driver \"%s\".",
+ cfdev->driver);
return NULL;
}
return NULL;
}
if (num_devices > 0) {
/* Do this only when we're sure there's hardware to talk to. */
if (num_devices > 0) {
/* Do this only when we're sure there's hardware to talk to. */
- if (sr_session_datafeed_callback_add(sigrok_feed_callback, NULL) != SR_OK)
+ if (sr_session_datafeed_callback_add(sigrok_feed_callback, NULL)
+ != SR_OK)
return NULL;
/* Start acquisition on all devices. */
return NULL;
/* Start acquisition on all devices. */
int status;
if (sr_thread_running) {
int status;
if (sr_thread_running) {
- ERROR("sigrok: Thread already running.");
+ ERROR("sigrok plugin: Thread already running.");
return -1;
}
if ((status = plugin_thread_create(&sr_thread, NULL, sigrok_read_thread,
NULL)) != 0) {
return -1;
}
if ((status = plugin_thread_create(&sr_thread, NULL, sigrok_read_thread,
NULL)) != 0) {
- ERROR("sigrok: Failed to create thread: %s.", strerror(status));
+ ERROR("sigrok plugin: Failed to create thread: %s.",
+ strerror(status));
return -1;
}
sr_thread_running = TRUE;
return -1;
}
sr_thread_running = TRUE;
void module_register(void)
{
void module_register(void)
{
-
plugin_register_complex_config("sigrok", sigrok_config);
plugin_register_init("sigrok", sigrok_init);
plugin_register_shutdown("sigrok", sigrok_shutdown);
plugin_register_complex_config("sigrok", sigrok_config);
plugin_register_init("sigrok", sigrok_init);
plugin_register_shutdown("sigrok", sigrok_shutdown);
-
}
}