X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=hardware%2Ffluke-dmm%2Fapi.c;h=d6c322accdc9416a0ccc3e214d5d8dc31cc7ec48;hb=545f9786390a89b8bb6868907afa555fc0a6ece4;hp=bba2f1291a1e271519cf548906c01a98536dc0be;hpb=fb480d578e423cd0f5702d1251e3cdccb990c9fb;p=libsigrok.git diff --git a/hardware/fluke-dmm/api.c b/hardware/fluke-dmm/api.c index bba2f129..d6c322ac 100644 --- a/hardware/fluke-dmm/api.c +++ b/hardware/fluke-dmm/api.c @@ -20,7 +20,6 @@ #include #include "libsigrok.h" #include "libsigrok-internal.h" -#include "config.h" #include "fluke-dmm.h" #include #include @@ -28,12 +27,31 @@ #include #include +static const int hwopts[] = { + SR_HWOPT_CONN, + SR_HWOPT_SERIALCOMM, + 0, +}; + +static const int hwcaps[] = { + SR_HWCAP_MULTIMETER, + SR_HWCAP_LIMIT_SAMPLES, + SR_HWCAP_LIMIT_MSEC, + SR_HWCAP_CONTINUOUS, + 0, +}; + +static const char *probe_names[] = { + "Probe", + NULL, +}; SR_PRIV struct sr_dev_driver flukedmm_driver_info; static struct sr_dev_driver *di = &flukedmm_driver_info; static const struct flukedmm_profile supported_flukedmm[] = { - { FLUKE_187, "187" }, + { FLUKE_187, "187", 100 }, + { FLUKE_287, "287", 100 }, }; @@ -95,7 +113,7 @@ static int serial_readline(int fd, char **buf, int *buflen, uint64_t timeout_ms) if (len > 0) { *buflen += len; *(*buf + *buflen) = '\0'; - if (*buflen > 0 && *(*buf + *buflen - 1) == '\n') { + if (*buflen > 0 && *(*buf + *buflen - 1) == '\r') { /* Strip LF and terminate. */ *(*buf + --*buflen) = '\0'; break; @@ -111,56 +129,29 @@ static int serial_readline(int fd, char **buf, int *buflen, uint64_t timeout_ms) return SR_OK; } -static GSList *hw_scan(GSList *options) +static GSList *fluke_scan(const char *conn, const char *serialcomm) { struct sr_dev_inst *sdi; struct drv_context *drvc; struct dev_context *devc; - struct sr_hwopt *opt; struct sr_probe *probe; - GSList *l, *devices; - int retry, len, fd, i, s; - const char *conn, *serialcomm; + GSList *devices; + int fd, retry, len, i, s; char buf[128], *b, **tokens; - drvc = di->priv; - drvc->instances = NULL; - - devices = NULL; - conn = serialcomm = NULL; - for (l = options; l; l = l->next) { - opt = l->data; - switch (opt->hwopt) { - case SR_HWOPT_CONN: - conn = opt->value; - break; - case SR_HWOPT_SERIALCOMM: - serialcomm = opt->value; - break; - } - } - if (!conn) { - sr_dbg("fluke-dmm: no serial port provided"); - return NULL; - } - if (!serialcomm) { - sr_dbg("fluke-dmm: no serial communication parameters provided"); - return NULL; - } - if ((fd = serial_open(conn, O_RDWR|O_NONBLOCK)) == -1) { sr_err("fluke-dmm: unable to open %s: %s", conn, strerror(errno)); return NULL; } - if (serial_set_paramstr(fd, serialcomm) != SR_OK) { - sr_err("fluke-dmm: unable to set serial parameters: %s", - strerror(errno)); + sr_err("fluke-dmm: unable to set serial parameters"); return NULL; } + drvc = di->priv; b = buf; retry = 0; + devices = NULL; /* We'll try the discovery sequence three times in case the device * is not in an idle state when we send ID. */ while (!devices && retry < 3) { @@ -178,10 +169,8 @@ static GSList *hw_scan(GSList *options) serial_readline(fd, &b, &len, 150); if (len != 1) continue; - if (buf[0] != '0') { - sr_dbg("fluke-dmm: got ID response %.2x", buf[0]); + if (buf[0] != '0') continue; - } /* If CMD_ACK was OK, ID string follows. */ len = 128; @@ -205,6 +194,7 @@ static GSList *hw_scan(GSList *options) } devc->profile = &supported_flukedmm[i]; devc->serial = sr_serial_dev_inst_new(conn, -1); + devc->serialcomm = g_strdup(serialcomm); sdi->priv = devc; sdi->driver = di; if (!(probe = sr_probe_new(0, SR_PROBE_ANALOG, TRUE, "P1"))) @@ -217,12 +207,46 @@ static GSList *hw_scan(GSList *options) } g_strfreev(tokens); } - serial_close(fd); return devices; } +static GSList *hw_scan(GSList *options) +{ + struct sr_hwopt *opt; + GSList *l, *devices; + const char *conn, *serialcomm; + + conn = serialcomm = NULL; + for (l = options; l; l = l->next) { + opt = l->data; + switch (opt->hwopt) { + case SR_HWOPT_CONN: + conn = opt->value; + break; + case SR_HWOPT_SERIALCOMM: + serialcomm = opt->value; + break; + } + } + if (!conn) + return NULL; + + if (serialcomm) { + /* Use the provided comm specs. */ + devices = fluke_scan(conn, serialcomm); + } else { + /* Try 115200, as used on 287/289. */ + devices = fluke_scan(conn, "115200/8n1"); + if (!devices) + /* Fall back to 9600 for 187/189. */ + devices = fluke_scan(conn, "9600/8n1"); + } + + return devices; +} + static GSList *hw_dev_list(void) { struct drv_context *drvc; @@ -234,16 +258,42 @@ static GSList *hw_dev_list(void) static int hw_dev_open(struct sr_dev_inst *sdi) { + struct dev_context *devc; + + if (!(devc = sdi->priv)) { + sr_err("fluke-dmm: sdi->priv was NULL."); + return SR_ERR_BUG; + } - /* TODO */ + devc->serial->fd = serial_open(devc->serial->port, O_RDWR | O_NONBLOCK); + if (devc->serial->fd == -1) { + sr_err("fluke-dmm: Couldn't open serial port '%s'.", + devc->serial->port); + return SR_ERR; + } + if (serial_set_paramstr(devc->serial->fd, devc->serialcomm) != SR_OK) { + sr_err("fluke-dmm: unable to set serial parameters"); + return SR_ERR; + } + sdi->status = SR_ST_ACTIVE; return SR_OK; } static int hw_dev_close(struct sr_dev_inst *sdi) { + struct dev_context *devc; - /* TODO */ + if (!(devc = sdi->priv)) { + sr_err("fluke-dmm: sdi->priv was NULL."); + return SR_ERR_BUG; + } + + if (devc->serial && devc->serial->fd != -1) { + serial_close(devc->serial->fd); + devc->serial->fd = -1; + sdi->status = SR_ST_INACTIVE; + } return SR_OK; } @@ -253,8 +303,6 @@ static int hw_cleanup(void) clear_instances(); - /* TODO */ - return SR_OK; } @@ -262,9 +310,21 @@ static int hw_info_get(int info_id, const void **data, const struct sr_dev_inst *sdi) { + (void)sdi; switch (info_id) { - /* TODO */ + case SR_DI_HWOPTS: + *data = hwopts; + break; + case SR_DI_HWCAPS: + *data = hwcaps; + break; + case SR_DI_NUM_PROBES: + *data = GINT_TO_POINTER(1); + break; + case SR_DI_PROBE_NAMES: + *data = probe_names; + break; default: return SR_ERR_ARG; } @@ -275,26 +335,82 @@ static int hw_info_get(int info_id, const void **data, static int hw_dev_config_set(const struct sr_dev_inst *sdi, int hwcap, const void *value) { - int ret; + struct dev_context *devc; if (sdi->status != SR_ST_ACTIVE) return SR_ERR; - ret = SR_OK; + if (!(devc = sdi->priv)) { + sr_err("fluke-dmm: sdi->priv was NULL."); + return SR_ERR_BUG; + } + switch (hwcap) { - /* TODO */ + case SR_HWCAP_LIMIT_MSEC: + /* TODO: not yet implemented */ + if (*(const uint64_t *)value == 0) { + sr_err("fluke-dmm: LIMIT_MSEC can't be 0."); + return SR_ERR; + } + devc->limit_msec = *(const uint64_t *)value; + sr_dbg("fluke-dmm: Setting time limit to %" PRIu64 "ms.", + devc->limit_msec); + break; + case SR_HWCAP_LIMIT_SAMPLES: + devc->limit_samples = *(const uint64_t *)value; + sr_dbg("fluke-dmm: Setting sample limit to %" PRIu64 ".", + devc->limit_samples); + break; default: - ret = SR_ERR_ARG; + sr_err("fluke-dmm: Unknown capability: %d.", hwcap); + return SR_ERR; + break; } - return ret; + return SR_OK; } static int hw_dev_acquisition_start(const struct sr_dev_inst *sdi, void *cb_data) { + struct sr_datafeed_packet packet; + struct sr_datafeed_header header; + struct sr_datafeed_meta_analog meta; + struct dev_context *devc; + + if (!(devc = sdi->priv)) { + sr_err("fluke-dmm: sdi->priv was NULL."); + return SR_ERR_BUG; + } + + sr_dbg("fluke-dmm: Starting acquisition."); + + devc->cb_data = cb_data; - /* TODO */ + /* Send header packet to the session bus. */ + sr_dbg("fluke-dmm: Sending SR_DF_HEADER."); + packet.type = SR_DF_HEADER; + packet.payload = (uint8_t *)&header; + header.feed_version = 1; + gettimeofday(&header.starttime, NULL); + sr_session_send(devc->cb_data, &packet); + + /* Send metadata about the SR_DF_ANALOG packets to come. */ + sr_dbg("fluke-dmm: Sending SR_DF_META_ANALOG."); + packet.type = SR_DF_META_ANALOG; + packet.payload = &meta; + meta.num_probes = 1; + sr_session_send(devc->cb_data, &packet); + + /* Poll every 100ms, or whenever some data comes in. */ + sr_source_add(devc->serial->fd, G_IO_IN, 50, fluke_receive_data, (void *)sdi); + + if (serial_write(devc->serial->fd, "QM\r", 3) == -1) { + sr_err("fluke-dmm: unable to send QM: %s", strerror(errno)); + return SR_ERR; + } + devc->cmd_sent_at = g_get_monotonic_time() / 1000; + devc->expect_response = TRUE; return SR_OK; } @@ -302,13 +418,26 @@ static int hw_dev_acquisition_start(const struct sr_dev_inst *sdi, static int hw_dev_acquisition_stop(const struct sr_dev_inst *sdi, void *cb_data) { - - (void)cb_data; + struct sr_datafeed_packet packet; + struct dev_context *devc; if (sdi->status != SR_ST_ACTIVE) return SR_ERR; - /* TODO */ + if (!(devc = sdi->priv)) { + sr_err("fluke-dmm: sdi->priv was NULL."); + return SR_ERR_BUG; + } + + sr_dbg("fluke-dmm: Stopping acquisition."); + + sr_source_remove(devc->serial->fd); + hw_dev_close((struct sr_dev_inst *)sdi); + + /* Send end packet to the session bus. */ + sr_dbg("fluke-dmm: Sending SR_DF_END."); + packet.type = SR_DF_END; + sr_session_send(cb_data, &packet); return SR_OK; }