X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=hwplugin.c;h=06cd96905397c4c42d9ca2f97947a4fa8d74463c;hb=b2ff95063a845ec7130d97fc3d8e9c65fe45f902;hp=f3401dc784ec9a3b9e8c9d70a382e358e4e30871;hpb=3a4d09c0de68c3ae9f287d243967431b8e3de0b7;p=libsigrok.git
diff --git a/hwplugin.c b/hwplugin.c
index f3401dc7..06cd9690 100644
--- a/hwplugin.c
+++ b/hwplugin.c
@@ -17,29 +17,28 @@
* along with this program. If not, see .
*/
-#include "config.h"
#include
#include
#include
#include
#include
#include
-#include
-#include
+#include "sigrok.h"
+#include "sigrok-internal.h"
/* The list of loaded plugins lives here. */
-GSList *plugins;
+static GSList *plugins;
/*
* This enumerates which plugin capabilities correspond to user-settable
* options.
*/
/* TODO: This shouldn't be a global. */
-struct sr_hwcap_option sr_hwcap_options[] = {
+SR_API struct sr_hwcap_option sr_hwcap_options[] = {
{SR_HWCAP_SAMPLERATE, SR_T_UINT64, "Sample rate", "samplerate"},
{SR_HWCAP_CAPTURE_RATIO, SR_T_UINT64, "Pre-trigger capture ratio", "captureratio"},
{SR_HWCAP_PATTERN_MODE, SR_T_CHAR, "Pattern generator mode", "patternmode"},
- {SR_HWCAP_RLE, SR_T_CHAR, "Run Length Encoding", "rle"},
+ {SR_HWCAP_RLE, SR_T_BOOL, "Run Length Encoding", "rle"},
{0, 0, NULL, NULL},
};
@@ -59,7 +58,7 @@ extern struct sr_device_plugin zeroplus_logic_cube_plugin_info;
extern struct sr_device_plugin asix_sigma_plugin_info;
#endif
#ifdef HAVE_LA_CHRONOVU_LA8
-extern struct device_plugin chronovu_la8_plugin_info;
+extern SR_PRIV struct device_plugin chronovu_la8_plugin_info;
#endif
#ifdef HAVE_LA_LINK_MSO19
extern struct sr_device_plugin link_mso19_plugin_info;
@@ -68,9 +67,8 @@ extern struct sr_device_plugin link_mso19_plugin_info;
extern struct sr_device_plugin alsa_plugin_info;
#endif
-
/* TODO: No linked list needed, this can be a simple array. */
-int load_hwplugins(void)
+SR_PRIV int load_hwplugins(void)
{
#ifdef HAVE_LA_DEMO
plugins = g_slist_append(plugins, (gpointer *)&demo_plugin_info);
@@ -99,30 +97,45 @@ int load_hwplugins(void)
plugins = g_slist_append(plugins, (gpointer *)&alsa_plugin_info);
#endif
-
return SR_OK;
}
-GSList *sr_list_hwplugins(void)
+SR_API GSList *sr_list_hwplugins(void)
{
return plugins;
}
-int sr_init_hwplugins(struct sr_device_plugin *plugin)
+SR_API int sr_init_hwplugins(struct sr_device_plugin *plugin)
{
- int num_devices, num_probes, i;
+ int num_devices, num_probes, i, j;
+ int num_initialized_devices = 0;
+ struct sr_device *device;
+ char **probe_names;
- g_message("initializing %s plugin", plugin->name);
+ sr_dbg("initializing %s plugin", plugin->name);
num_devices = plugin->init(NULL);
for (i = 0; i < num_devices; i++) {
- num_probes = (int)plugin->get_device_info(i, SR_DI_NUM_PROBES);
- sr_device_new(plugin, i, num_probes);
+ num_probes = GPOINTER_TO_INT(
+ plugin->get_device_info(i, SR_DI_NUM_PROBES));
+ probe_names = (char **)plugin->get_device_info(i,
+ SR_DI_PROBE_NAMES);
+
+ if (!probe_names) {
+ sr_warn("hwplugin: %s: plugin %s does not return a "
+ "list of probe names", __func__, plugin->name);
+ continue;
+ }
+
+ device = sr_device_new(plugin, i);
+ for (j = 0; j < num_probes; j++)
+ sr_device_probe_add(device, probe_names[j]);
+ num_initialized_devices++;
}
- return num_devices;
+ return num_initialized_devices;
}
-void sr_cleanup_hwplugins(void)
+SR_PRIV void sr_cleanup_hwplugins(void)
{
struct sr_device_plugin *plugin;
GSList *l;
@@ -132,16 +145,17 @@ void sr_cleanup_hwplugins(void)
if (plugin->cleanup)
plugin->cleanup();
}
-
}
-struct sr_device_instance *sr_device_instance_new(int index, int status,
+SR_PRIV struct sr_device_instance *sr_device_instance_new(int index, int status,
const char *vendor, const char *model, const char *version)
{
struct sr_device_instance *sdi;
- if (!(sdi = g_malloc(sizeof(struct sr_device_instance))))
+ if (!(sdi = g_try_malloc(sizeof(struct sr_device_instance)))) {
+ sr_err("hwplugin: %s: sdi malloc failed", __func__);
return NULL;
+ }
sdi->index = index;
sdi->status = status;
@@ -150,13 +164,12 @@ struct sr_device_instance *sr_device_instance_new(int index, int status,
sdi->model = model ? g_strdup(model) : NULL;
sdi->version = version ? g_strdup(version) : NULL;
sdi->priv = NULL;
- sdi->usb = NULL;
return sdi;
}
-struct sr_device_instance *sr_get_device_instance(GSList *device_instances,
- int device_index)
+SR_PRIV struct sr_device_instance *sr_get_device_instance(
+ GSList *device_instances, int device_index)
{
struct sr_device_instance *sdi;
GSList *l;
@@ -171,41 +184,26 @@ struct sr_device_instance *sr_get_device_instance(GSList *device_instances,
return NULL;
}
-void sr_device_instance_free(struct sr_device_instance *sdi)
+SR_PRIV void sr_device_instance_free(struct sr_device_instance *sdi)
{
- switch (sdi->instance_type) {
-#ifdef HAVE_LIBUSB_1_0
- case SR_USB_INSTANCE:
- sr_usb_device_instance_free(sdi->usb);
- break;
-#endif
- case SR_SERIAL_INSTANCE:
- sr_serial_device_instance_free(sdi->serial);
- break;
- default:
- /* No specific type, nothing extra to free. */
- break;
- }
-
- if (sdi->priv)
- g_free(sdi->priv);
-
+ g_free(sdi->priv);
g_free(sdi->vendor);
g_free(sdi->model);
g_free(sdi->version);
g_free(sdi);
-
}
#ifdef HAVE_LIBUSB_1_0
-struct sr_usb_device_instance *sr_usb_device_instance_new(uint8_t bus,
+SR_PRIV struct sr_usb_device_instance *sr_usb_device_instance_new(uint8_t bus,
uint8_t address, struct libusb_device_handle *hdl)
{
struct sr_usb_device_instance *udi;
- if (!(udi = malloc(sizeof(struct sr_usb_device_instance))))
+ if (!(udi = g_try_malloc(sizeof(struct sr_usb_device_instance)))) {
+ sr_err("hwplugin: %s: udi malloc failed", __func__);
return NULL;
+ }
udi->bus = bus;
udi->address = address;
@@ -214,36 +212,39 @@ struct sr_usb_device_instance *sr_usb_device_instance_new(uint8_t bus,
return udi;
}
-void sr_usb_device_instance_free(struct sr_usb_device_instance *usb)
+SR_PRIV void sr_usb_device_instance_free(struct sr_usb_device_instance *usb)
{
/* Avoid compiler warnings. */
- usb = usb;
+ (void)usb;
/* Nothing to do for this device instance type. */
}
#endif
-struct sr_serial_device_instance *sr_serial_device_instance_new(
+SR_PRIV struct sr_serial_device_instance *sr_serial_device_instance_new(
const char *port, int fd)
{
struct sr_serial_device_instance *serial;
- if (!(serial = malloc(sizeof(struct sr_serial_device_instance))))
+ if (!(serial = g_try_malloc(sizeof(struct sr_serial_device_instance)))) {
+ sr_err("hwplugin: %s: serial malloc failed", __func__);
return NULL;
+ }
- serial->port = strdup(port);
+ serial->port = g_strdup(port);
serial->fd = fd;
return serial;
}
-void sr_serial_device_instance_free(struct sr_serial_device_instance *serial)
+SR_PRIV void sr_serial_device_instance_free(
+ struct sr_serial_device_instance *serial)
{
- free(serial->port);
+ g_free(serial->port);
}
-int sr_find_hwcap(int *capabilities, int hwcap)
+SR_API int sr_find_hwcap(int *capabilities, int hwcap)
{
int i;
@@ -255,7 +256,7 @@ int sr_find_hwcap(int *capabilities, int hwcap)
return FALSE;
}
-struct sr_hwcap_option *sr_find_hwcap_option(int hwcap)
+SR_API struct sr_hwcap_option *sr_find_hwcap_option(int hwcap)
{
int i;
@@ -269,12 +270,12 @@ struct sr_hwcap_option *sr_find_hwcap_option(int hwcap)
/* unnecessary level of indirection follows. */
-void sr_source_remove(int fd)
+SR_PRIV void sr_source_remove(int fd)
{
sr_session_source_remove(fd);
}
-void sr_source_add(int fd, int events, int timeout,
+SR_PRIV void sr_source_add(int fd, int events, int timeout,
sr_receive_data_callback rcv_cb, void *user_data)
{
sr_session_source_add(fd, events, timeout, rcv_cb, user_data);