]> sigrok.org Git - libsigrok.git/blobdiff - device.c
serial: revamp internal API
[libsigrok.git] / device.c
index be647fb248e7be173810a439fe543d41fb10d59a..f437c18a5fdb88e544fc4cdc1821070f641083c5 100644 (file)
--- a/device.c
+++ b/device.c
 
 #include <stdio.h>
 #include <glib.h>
+#include "config.h" /* Needed for HAVE_LIBUSB_1_0 and others. */
 #include "libsigrok.h"
 #include "libsigrok-internal.h"
 
-static GSList *devs = NULL;
+/* Message logging helpers with driver-specific prefix string. */
+#define DRIVER_LOG_DOMAIN "device: "
+#define sr_log(l, s, args...) sr_log(l, DRIVER_LOG_DOMAIN s, ## args)
+#define sr_spew(s, args...) sr_spew(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_dbg(s, args...) sr_dbg(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_info(s, args...) sr_info(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_warn(s, args...) sr_warn(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_err(s, args...) sr_err(DRIVER_LOG_DOMAIN s, ## args)
 
 /**
- * Scan the system for attached logic analyzers / devices.
+ * @file
  *
- * This will try to autodetect all supported logic analyzer devices:
- *
- *  - Those attached via USB (can be reliably detected via USB VID/PID).
- *
- *  - Those using a (real or virtual) serial port (detected by sending
- *    device-specific commands to all OS-specific serial port devices such
- *    as /dev/ttyS*, /dev/ttyUSB*, /dev/ttyACM*, and others).
- *    The autodetection for this kind of devices can potentially be unreliable.
- *
- *    Also, sending various bytes/commands to (all!) devices which happen to
- *    be attached to the system via a (real or virtual) serial port can be
- *    problematic. There is no way for libsigrok to know how unknown devices
- *    react to the bytes libsigrok sends. Potentially they could lead to the
- *    device getting into invalid/error states, losing/overwriting data, or...
- *
- * In addition to the detection, the devices that are found are also
- * initialized automatically. On some devices, this involves a firmware upload,
- * or other such measures.
- *
- * The order in which the system is scanned for devices is not specified. The
- * caller should not assume or rely on any specific order.
- *
- * After the system has been scanned for devices, the list of detected (and
- * supported) devices can be acquired via sr_dev_list().
- *
- * TODO: Error checks?
- * TODO: Option to only scan for specific devices or device classes.
- *
- * @return SR_OK upon success, SR_ERR_BUG upon internal errors.
+ * Device handling in libsigrok.
  */
-SR_API int sr_dev_scan(void)
-{
-       int i;
-       struct sr_dev_driver **drivers;
-
-       drivers = sr_driver_list();
-       if (!drivers[0]) {
-               sr_err("dev: %s: no supported hardware drivers", __func__);
-               return SR_ERR_BUG;
-       }
-
-       /*
-        * Initialize all drivers first. Since the init() call may involve
-        * a firmware upload and associated delay, we may as well get all
-        * of these out of the way first.
-        */
-       for (i = 0; drivers[i]; i++)
-               sr_driver_init(drivers[i]);
-
-       return SR_OK;
-}
 
 /**
- * Return the list of logic analyzer devices libsigrok has detected.
- *
- * If the libsigrok-internal device list is empty, a scan for attached
- * devices -- via a call to sr_dev_scan() -- is performed first.
+ * @defgroup grp_devices Devices
  *
- * TODO: Error handling?
+ * Device handling in libsigrok.
  *
- * @return The list (GSList) of detected devices, or NULL if none were found.
+ * @{
  */
-SR_API GSList *sr_dev_list(void)
-{
-       if (!devs)
-               sr_dev_scan();
 
-       return devs;
-}
-
-/**
- * Create a new device.
- *
- * The device is added to the (libsigrok-internal) list of devices, but
- * additionally a pointer to the newly created device is also returned.
- *
- * The device has no probes attached to it yet after this call. You can
- * use sr_dev_probe_add() to add one or more probes.
- *
- * TODO: Should return int, so that we can return SR_OK, SR_ERR_* etc.
- *
- * It is the caller's responsibility to g_free() the allocated memory when
- * no longer needed. TODO: Using which API function?
- *
- * @param driver TODO.
- *               If 'driver' is NULL, the created device is a "virtual" one.
- * @param driver_index TODO
- *
- * @return Pointer to the newly allocated device, or NULL upon errors.
- */
-SR_API struct sr_dev *sr_dev_new(const struct sr_dev_driver *driver,
-                                int driver_index)
+/** @private */
+SR_PRIV struct sr_probe *sr_probe_new(int index, int type,
+               gboolean enabled, const char *name)
 {
-       struct sr_dev *dev;
-
-       /* TODO: Check if driver_index valid? */
+       struct sr_probe *probe;
 
-       if (!(dev = g_try_malloc0(sizeof(struct sr_dev)))) {
-               sr_err("dev: %s: dev malloc failed", __func__);
+       if (!(probe = g_try_malloc0(sizeof(struct sr_probe)))) {
+               sr_err("Probe malloc failed.");
                return NULL;
        }
 
-       dev->driver = (struct sr_dev_driver *)driver;
-       dev->driver_index = driver_index;
-       devs = g_slist_append(devs, dev);
+       probe->index = index;
+       probe->type = type;
+       probe->enabled = enabled;
+       if (name)
+               probe->name = g_strdup(name);
 
-       return dev;
-}
-
-/**
- * Add a probe with the specified name to the specified device.
- *
- * The added probe is automatically enabled (the 'enabled' field is TRUE).
- *
- * The 'trigger' field of the added probe is set to NULL. A trigger can be
- * added via sr_dev_trigger_set().
- *
- * TODO: Are duplicate names allowed?
- * TODO: Do we enforce a maximum probe number for a device?
- * TODO: Error if the max. probe number for the specific LA is reached, e.g.
- *       if the caller tries to add more probes than the device actually has.
- *
- * @param dev The device to which to add a probe with the specified name.
- *            Must not be NULL.
- * @param name The name of the probe to add to this device. Must not be NULL.
- *             TODO: Maximum length, allowed characters, etc.
- *
- * @return SR_OK upon success, SR_ERR_MALLOC upon memory allocation errors,
- *         or SR_ERR_ARG upon invalid arguments.
- *         If something other than SR_OK is returned, 'dev' is unchanged.
- */
-SR_API int sr_dev_probe_add(struct sr_dev *dev, const char *name)
-{
-       struct sr_probe *p;
-       int probenum;
-
-       if (!dev) {
-               sr_err("dev: %s: dev was NULL", __func__);
-               return SR_ERR_ARG;
-       }
-
-       if (!name) {
-               sr_err("dev: %s: name was NULL", __func__);
-               return SR_ERR_ARG;
-       }
-
-       /* TODO: Further checks to ensure name is valid. */
-
-       probenum = g_slist_length(dev->probes) + 1;
-
-       if (!(p = g_try_malloc0(sizeof(struct sr_probe)))) {
-               sr_err("dev: %s: p malloc failed", __func__);
-               return SR_ERR_MALLOC;
-       }
-
-       p->index = probenum;
-       p->enabled = TRUE;
-       p->name = g_strdup(name);
-       p->trigger = NULL;
-       dev->probes = g_slist_append(dev->probes, p);
-
-       return SR_OK;
-}
-
-/**
- * Find the probe with the specified number in the specified device.
- *
- * TODO
- *
- * @param dev TODO. Must not be NULL.
- * @param probenum The number of the probe whose 'struct sr_probe' we want.
- *                 Note that the probe numbers start at 1 (not 0!).
- *
- * TODO: Should return int.
- * TODO: probenum should be unsigned.
- *
- * @return A pointer to the requested probe's 'struct sr_probe', or NULL
- *         if the probe could not be found.
- */
-SR_API struct sr_probe *sr_dev_probe_find(const struct sr_dev *dev,
-                                         int probenum)
-{
-       GSList *l;
-       struct sr_probe *p, *found_probe;
-
-       if (!dev) {
-               sr_err("dev: %s: dev was NULL", __func__);
-               return NULL; /* TODO: SR_ERR_ARG */
-       }
-
-       /* TODO: Sanity check on probenum. */
-
-       found_probe = NULL;
-       for (l = dev->probes; l; l = l->next) {
-               p = l->data;
-               /* TODO: Check for p != NULL. */
-               if (p->index == probenum) {
-                       found_probe = p;
-                       break;
-               }
-       }
-
-       return found_probe;
+       return probe;
 }
 
 /**
@@ -238,39 +72,38 @@ SR_API struct sr_probe *sr_dev_probe_find(const struct sr_dev *dev,
  * If the probe already has a different name assigned to it, it will be
  * removed, and the new name will be saved instead.
  *
- * @param dev TODO
+ * @param sdi The device instance the probe is connected to.
  * @param probenum The number of the probe whose name to set.
- *                 Note that the probe numbers start at 1 (not 0!).
- * @param name The new name that the specified probe should get.
+ *                 Note that the probe numbers start at 0.
+ * @param name The new name that the specified probe should get. A copy
+ *             of the string is made.
  *
- * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments, or SR_ERR
- *         upon other errors.
- *         If something other than SR_OK is returned, 'dev' is unchanged.
+ * @return SR_OK on success, or SR_ERR_ARG on invalid arguments.
  */
-SR_API int sr_dev_probe_name_set(struct sr_dev *dev, int probenum,
-                                const char *name)
+SR_API int sr_dev_probe_name_set(const struct sr_dev_inst *sdi,
+               int probenum, const char *name)
 {
-       struct sr_probe *p;
+       GSList *l;
+       struct sr_probe *probe;
+       int ret;
 
-       if (!dev) {
-               sr_err("dev: %s: dev was NULL", __func__);
+       if (!sdi) {
+               sr_err("%s: sdi was NULL", __func__);
                return SR_ERR_ARG;
        }
 
-       p = sr_dev_probe_find(dev, probenum);
-       if (!p) {
-               sr_err("dev: %s: probe %d not found", __func__, probenum);
-               return SR_ERR; /* TODO: More specific error? */
+       ret = SR_ERR_ARG;
+       for (l = sdi->probes; l; l = l->next) {
+               probe = l->data;
+               if (probe->index == probenum) {
+                       g_free(probe->name);
+                       probe->name = g_strdup(name);
+                       ret = SR_OK;
+                       break;
+               }
        }
 
-       /* TODO: Sanity check on 'name'. */
-
-       /* If the probe already has a name, kill it first. */
-       g_free(p->name);
-
-       p->name = g_strdup(name);
-
-       return SR_OK;
+       return ret;
 }
 
 /**
@@ -305,61 +138,17 @@ SR_API int sr_dev_probe_enable(const struct sr_dev_inst *sdi, int probenum,
        return ret;
 }
 
-/**
- * Remove all triggers set up for the specified device.
- *
- * TODO: Better description.
- *
- * @param dev TODO
- *
- * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments.
- *         If something other than SR_OK is returned, 'dev' is unchanged.
- */
-SR_API int sr_dev_trigger_remove_all(struct sr_dev *dev)
-{
-       struct sr_probe *p;
-       unsigned int pnum; /* TODO: uint16_t? */
-
-       if (!dev) {
-               sr_err("dev: %s: dev was NULL", __func__);
-               return SR_ERR_ARG;
-       }
-
-       if (!dev->probes) {
-               sr_err("dev: %s: dev->probes was NULL", __func__);
-               return SR_ERR_ARG;
-       }
-
-       for (pnum = 1; pnum <= g_slist_length(dev->probes); pnum++) {
-               p = sr_dev_probe_find(dev, pnum);
-               /* TODO: Silently ignore probes which cannot be found? */
-               if (p) {
-                       g_free(p->trigger);
-                       p->trigger = NULL;
-               }
-       }
-
-       return SR_OK;
-}
-
 /**
  * Add a trigger to the specified device (and the specified probe).
  *
  * If the specified probe of this device already has a trigger, it will
  * be silently replaced.
  *
- * TODO: Better description.
- * TODO: Describe valid format of the 'trigger' string.
- *
- * @param dev TODO. Must not be NULL.
- * @param probenum The number of the probe. TODO.
- *                 Note that the probe numbers start at 1 (not 0!).
- * @param trigger TODO.
- *                TODO: Is NULL allowed?
+ * @param sdi Must not be NULL.
+ * @param probenum The probe number, starting from 0.
+ * @param trigger Trigger string, in the format used by sigrok-cli
  *
- * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments, or SR_ERR
- *         upon other errors.
- *         If something other than SR_OK is returned, 'dev' is unchanged.
+ * @return SR_OK on success, or SR_ERR_ARG on invalid arguments.
  */
 SR_API int sr_dev_trigger_set(const struct sr_dev_inst *sdi, int probenum,
                const char *trigger)
@@ -387,84 +176,189 @@ SR_API int sr_dev_trigger_set(const struct sr_dev_inst *sdi, int probenum,
 }
 
 /**
- * Determine whether the specified device has the specified capability.
+ * Determine whether the specified device instance has the specified
+ * capability.
  *
- * @param dev Pointer to the device to be checked. Must not be NULL.
+ * @param sdi Pointer to the device instance to be checked. Must not be NULL.
  *            If the device's 'driver' field is NULL (virtual device), this
  *            function will always return FALSE (virtual devices don't have
  *            a hardware capabilities list).
  * @param hwcap The capability that should be checked (whether it's supported
  *              by the specified device).
  *
- * @return TRUE, if the device has the specified capability, FALSE otherwise.
+ * @return TRUE if the device has the specified capability, FALSE otherwise.
  *         FALSE is also returned upon invalid input parameters or other
  *         error conditions.
  */
-SR_API gboolean sr_dev_has_hwcap(const struct sr_dev *dev, int hwcap)
+SR_API gboolean sr_dev_has_hwcap(const struct sr_dev_inst *sdi, int hwcap)
 {
        const int *hwcaps;
        int i;
 
-       sr_spew("dev: %s: requesting hwcap %d", __func__, hwcap);
+       if (!sdi || !sdi->driver)
+               return FALSE;
 
-       if (!dev) {
-               sr_err("dev: %s: dev was NULL", __func__);
+       if (sdi->driver->info_get(SR_DI_HWCAPS,
+                       (const void **)&hwcaps, NULL) != SR_OK)
                return FALSE;
+
+       for (i = 0; hwcaps[i]; i++) {
+               if (hwcaps[i] == hwcap)
+                       return TRUE;
        }
 
-       /*
-        * Virtual devices (which have dev->driver set to NULL) always say that
-        * they don't have the capability (they can't call hwcap_get_all()).
-        */
-       if (!dev->driver) {
-               sr_dbg("dev: %s: dev->driver was NULL, this seems to be "
-                      "a virtual device without capabilities", __func__);
-               return FALSE;
+       return FALSE;
+}
+
+/** @private */
+SR_PRIV struct sr_dev_inst *sr_dev_inst_new(int index, int status,
+               const char *vendor, const char *model, const char *version)
+{
+       struct sr_dev_inst *sdi;
+
+       if (!(sdi = g_try_malloc(sizeof(struct sr_dev_inst)))) {
+               sr_err("%s: sdi malloc failed", __func__);
+               return NULL;
        }
 
-       /* TODO: Sanity check on 'hwcap'. */
+       sdi->driver = NULL;
+       sdi->index = index;
+       sdi->status = status;
+       sdi->inst_type = -1;
+       sdi->vendor = vendor ? g_strdup(vendor) : NULL;
+       sdi->model = model ? g_strdup(model) : NULL;
+       sdi->version = version ? g_strdup(version) : NULL;
+       sdi->probes = NULL;
+       sdi->priv = NULL;
+
+       return sdi;
+}
 
-       if (dev->driver->info_get(SR_DI_HWCAPS,
-                       (const void **)&hwcaps, NULL) != SR_OK) {
-               sr_err("dev: %s: dev has no capabilities", __func__);
-               return FALSE;
+/** @private */
+SR_PRIV void sr_dev_inst_free(struct sr_dev_inst *sdi)
+{
+       struct sr_probe *probe;
+       GSList *l;
+
+       for (l = sdi->probes; l; l = l->next) {
+               probe = l->data;
+               g_free(probe->name);
+               g_free(probe);
        }
 
-       for (i = 0; hwcaps[i]; i++) {
-               if (hwcaps[i] != hwcap)
-                       continue;
-               sr_spew("dev: %s: found hwcap %d", __func__, hwcap);
-               return TRUE;
+       g_free(sdi->priv);
+       g_free(sdi->vendor);
+       g_free(sdi->model);
+       g_free(sdi->version);
+       g_free(sdi);
+
+}
+
+#ifdef HAVE_LIBUSB_1_0
+
+/** @private */
+SR_PRIV struct sr_usb_dev_inst *sr_usb_dev_inst_new(uint8_t bus,
+                       uint8_t address, struct libusb_device_handle *hdl)
+{
+       struct sr_usb_dev_inst *udi;
+
+       if (!(udi = g_try_malloc(sizeof(struct sr_usb_dev_inst)))) {
+               sr_err("%s: udi malloc failed", __func__);
+               return NULL;
        }
 
-       sr_spew("dev: %s: hwcap %d not found", __func__, hwcap);
+       udi->bus = bus;
+       udi->address = address;
+       udi->devhdl = hdl;
 
-       return FALSE;
+       return udi;
 }
 
-/**
- * Returns information about the given device.
- *
- * @param dev Pointer to the device to be checked. Must not be NULL.
- *            The device's 'driver' field must not be NULL either.
- * @param id The type of information.
- * @param data The return value. Must not be NULL.
+/** @private */
+SR_PRIV void sr_usb_dev_inst_free(struct sr_usb_dev_inst *usb)
+{
+       (void)usb;
+
+       /* Nothing to do for this device instance type. */
+}
+
+#endif
+
+/** @private
+ * @param pathname OS-specific serial port specification. Examples:
+ * "/dev/ttyUSB0", "/dev/ttyACM1", "/dev/tty.Modem-0", "COM1".
+ * @param serialcomm A serial communication parameters string, in the form
+ * of <speed>/<data bits><parity><stopbits>, for example "9600/8n1" or
+ * "600/7o2". This is an optional parameter; it may be filled in later.
+ * @return A pointer to a newly initialized struct sr_serial_dev_inst,
+ * or NULL on error.
  *
- * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments, or SR_ERR
- *         upon other errors.
+ * Both parameters are copied to newly allocated strings, and freed
+ * automatically by sr_serial_dev_inst_free().
  */
-SR_API int sr_dev_info_get(const struct sr_dev *dev, int id, const void **data)
+SR_PRIV struct sr_serial_dev_inst *sr_serial_dev_inst_new(const char *port,
+               const char *serialcomm)
 {
-       if ((dev == NULL) || (dev->driver == NULL))
-               return SR_ERR_ARG;
+       struct sr_serial_dev_inst *serial;
 
-       if (data == NULL)
-               return SR_ERR_ARG;
+       if (!port) {
+               sr_err("hwdriver: serial port required");
+               return NULL;
+       }
+
+       if (!(serial = g_try_malloc0(sizeof(struct sr_serial_dev_inst)))) {
+               sr_err("hwdriver: serial malloc failed");
+               return NULL;
+       }
+
+       serial->port = g_strdup(port);
+       if (serialcomm)
+               serial->serialcomm = g_strdup(serialcomm);
+       serial->fd = -1;
+
+       return serial;
+}
+
+/** @private */
+SR_PRIV void sr_serial_dev_inst_free(struct sr_serial_dev_inst *serial)
+{
 
-       *data = dev->driver->dev_info_get(dev->driver_index, id);
+       g_free(serial->port);
+       g_free(serial->serialcomm);
 
-       if (*data == NULL)
+}
+
+SR_API int sr_dev_config_set(const struct sr_dev_inst *sdi, int hwcap,
+               const void *value)
+{
+       int ret;
+
+       if (!sdi || !sdi->driver || !sdi->driver->dev_config_set) {
+               sr_err("Unable to set config option.");
                return SR_ERR;
+       }
 
-       return SR_OK;
+       ret = sdi->driver->dev_config_set(sdi, hwcap, value);
+
+       return ret;
 }
+
+SR_API GSList *sr_dev_inst_list(const struct sr_dev_driver *driver)
+{
+
+       if (driver && driver->dev_list)
+               return driver->dev_list();
+       else
+               return NULL;
+}
+
+SR_API int sr_dev_inst_clear(const struct sr_dev_driver *driver)
+{
+
+       if (driver && driver->dev_clear)
+               return driver->dev_clear();
+       else
+               return SR_OK;
+}
+
+/** @} */