]> sigrok.org Git - libsigrok.git/blobdiff - src/hardware/serial-dmm/api.c
output/csv: use intermediate time_t var, silence compiler warning
[libsigrok.git] / src / hardware / serial-dmm / api.c
index 5c0e63051dc6a03bfb75e9efa54dbbfe72ae84a4..9af2bd5bc56e8b79a035fa78a24d613f8c6aa036 100644 (file)
@@ -34,11 +34,14 @@ static const uint32_t scanopts[] = {
        SR_CONF_SERIALCOMM,
 };
 
-static const uint32_t devopts[] = {
+static const uint32_t drvopts[] = {
        SR_CONF_MULTIMETER,
+};
+
+static const uint32_t devopts[] = {
        SR_CONF_CONTINUOUS,
-       SR_CONF_LIMIT_SAMPLES | SR_CONF_SET,
-       SR_CONF_LIMIT_MSEC | SR_CONF_SET,
+       SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET,
 };
 
 static GSList *scan(struct sr_dev_driver *di, GSList *options)
@@ -50,13 +53,16 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        struct sr_dev_inst *sdi;
        struct dev_context *devc;
        struct sr_serial_dev_inst *serial;
-       int dropped, ret;
-       size_t len;
+       int ret;
+       size_t dropped, len, packet_len;
        uint8_t buf[128];
+       size_t ch_idx;
+       char ch_name[12];
 
        dmm = (struct dmm_info *)di;
 
-       conn = serialcomm = NULL;
+       conn = dmm->conn;
+       serialcomm = dmm->serialcomm;
        for (l = options; l; l = l->next) {
                src = l->data;
                switch (src->key) {
@@ -71,24 +77,27 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        if (!conn)
                return NULL;
 
-       if (!serialcomm)
-               serialcomm = dmm->conn;
-
        serial = sr_serial_dev_inst_new(conn, serialcomm);
 
        if (serial_open(serial, SERIAL_RDWR) != SR_OK)
                return NULL;
-
        sr_info("Probing serial port %s.", conn);
 
+       if (dmm->after_open) {
+               ret = dmm->after_open(serial);
+               if (ret != SR_OK) {
+                       sr_err("Activity after port open failed: %d.", ret);
+                       return NULL;
+               }
+       }
+
        devices = NULL;
-       serial_flush(serial);
 
        /* Request a packet if the DMM requires this. */
        if (dmm->packet_request) {
                if ((ret = dmm->packet_request(serial)) < 0) {
                        sr_err("Failed to request packet: %d.", ret);
-                       return FALSE;
+                       return NULL;
                }
        }
 
@@ -96,39 +105,79 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
         * There's no way to get an ID from the multimeter. It just sends data
         * periodically (or upon request), so the best we can do is check if
         * the packets match the expected format.
-        */
-
-       /* Let's get a bit of data and see if we can find a packet. */
-       len = sizeof(buf);
-       ret = serial_stream_detect(serial, buf, &len, dmm->packet_size,
-                                  dmm->packet_valid, 3000,
-                                  dmm->baudrate);
-       if (ret != SR_OK)
-               goto scan_cleanup;
-
-       /*
+        *
         * If we dropped more than two packets worth of data, something is
         * wrong. We shouldn't quit however, since the dropped bytes might be
         * just zeroes at the beginning of the stream. Those can occur as a
         * combination of the nonstandard cable that ships with some devices
         * and the serial port or USB to serial adapter.
         */
+       len = sizeof(buf);
+       ret = serial_stream_detect(serial, buf, &len, dmm->packet_size,
+               dmm->packet_valid, dmm->packet_valid_len, &packet_len, 3000);
+       if (ret != SR_OK)
+               goto scan_cleanup;
        dropped = len - dmm->packet_size;
-       if (dropped > 2 * dmm->packet_size)
-               sr_warn("Had to drop too much data.");
-
+       if (dropped > 2 * packet_len)
+               sr_warn("Packet search dropped a lot of data.");
        sr_info("Found device on port %s.", conn);
 
-       sdi = g_malloc0(sizeof(struct sr_dev_inst));
+       /*
+        * Setup optional additional callbacks when sub device drivers
+        * happen to provide them. (This is a compromise to do it here,
+        * and not extend the DMM_CONN() et al set of macros.)
+        */
+       if (strcmp(dmm->di.name, "brymen-bm52x") == 0) {
+               /* Applicable to BM520s but not to BM820s. */
+               dmm->dmm_state_init = brymen_bm52x_state_init;
+               dmm->dmm_state_free = brymen_bm52x_state_free;
+               dmm->config_get = brymen_bm52x_config_get;
+               dmm->config_set = brymen_bm52x_config_set;
+               dmm->config_list = brymen_bm52x_config_list;
+               dmm->acquire_start = brymen_bm52x_acquire_start;
+       }
+       if (dmm->dmm_state_init)
+               dmm->dmm_state = dmm->dmm_state_init();
+
+       /* Setup the device instance. */
+       sdi = g_malloc0(sizeof(*sdi));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup(dmm->vendor);
        sdi->model = g_strdup(dmm->device);
-       devc = g_malloc0(sizeof(struct dev_context));
+       devc = g_malloc0(sizeof(*devc));
        sr_sw_limits_init(&devc->limits);
        sdi->inst_type = SR_INST_SERIAL;
        sdi->conn = serial;
        sdi->priv = devc;
-       sr_channel_new(sdi, 0, SR_CHANNEL_ANALOG, TRUE, "P1");
+
+       /* Create (optionally device dependent) channel(s). */
+       dmm->channel_count = 1;
+       if (dmm->packet_parse == sr_brymen_bm52x_parse)
+               dmm->channel_count = BRYMEN_BM52X_DISPLAY_COUNT;
+       if (dmm->packet_parse == sr_brymen_bm86x_parse)
+               dmm->channel_count = BRYMEN_BM86X_DISPLAY_COUNT;
+       if (dmm->packet_parse == sr_eev121gw_3displays_parse) {
+               dmm->channel_count = EEV121GW_DISPLAY_COUNT;
+               dmm->channel_formats = eev121gw_channel_formats;
+       }
+       if (dmm->packet_parse == sr_metex14_4packets_parse)
+               dmm->channel_count = 4;
+       if (dmm->packet_parse == sr_ms2115b_parse) {
+               dmm->channel_count = MS2115B_DISPLAY_COUNT;
+               dmm->channel_formats = ms2115b_channel_formats;
+       }
+       for (ch_idx = 0; ch_idx < dmm->channel_count; ch_idx++) {
+               size_t ch_num;
+               const char *fmt;
+               fmt = "P%zu";
+               if (dmm->channel_formats && dmm->channel_formats[ch_idx])
+                       fmt = dmm->channel_formats[ch_idx];
+               ch_num = ch_idx + 1;
+               snprintf(ch_name, sizeof(ch_name), fmt, ch_num);
+               sr_channel_new(sdi, ch_idx, SR_CHANNEL_ANALOG, TRUE, ch_name);
+       }
+
+       /* Add found device to result set. */
        devices = g_slist_append(devices, sdi);
 
 scan_cleanup:
@@ -137,27 +186,85 @@ scan_cleanup:
        return std_scan_complete(di, devices);
 }
 
-static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sdi,
-               const struct sr_channel_group *cg)
+static int config_get(uint32_t key, GVariant **data,
+       const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
 {
        struct dev_context *devc;
+       struct dmm_info *dmm;
 
-       (void)cg;
+       if (!sdi)
+               return SR_ERR_ARG;
+       devc = sdi->priv;
 
+       switch (key) {
+       case SR_CONF_LIMIT_SAMPLES:
+       case SR_CONF_LIMIT_FRAMES:
+       case SR_CONF_LIMIT_MSEC:
+               return sr_sw_limits_config_get(&devc->limits, key, data);
+       default:
+               dmm = (struct dmm_info *)sdi->driver;
+               if (!dmm || !dmm->config_get)
+                       return SR_ERR_NA;
+               return dmm->config_get(dmm->dmm_state, key, data, sdi, cg);
+       }
+       /* UNREACH */
+}
+
+static int config_set(uint32_t key, GVariant *data,
+       const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
+{
+       struct dev_context *devc;
+       struct dmm_info *dmm;
+
+       if (!sdi)
+               return SR_ERR_ARG;
        devc = sdi->priv;
+       (void)cg;
 
-       return sr_sw_limits_config_set(&devc->limits, key, data);
+       switch (key) {
+       case SR_CONF_LIMIT_SAMPLES:
+       case SR_CONF_LIMIT_FRAMES:
+       case SR_CONF_LIMIT_MSEC:
+               return sr_sw_limits_config_set(&devc->limits, key, data);
+       default:
+               dmm = (struct dmm_info *)sdi->driver;
+               if (!dmm || !dmm->config_set)
+                       return SR_ERR_NA;
+               return dmm->config_set(dmm->dmm_state, key, data, sdi, cg);
+       }
 }
 
-static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *sdi,
-               const struct sr_channel_group *cg)
+static int config_list(uint32_t key, GVariant **data,
+       const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
 {
-       return STD_CONFIG_LIST(key, data, sdi, cg, scanopts, NULL, devopts);
+       struct dmm_info *dmm;
+       int ret;
+
+       /* Use common logic for standard keys. */
+       if (!sdi)
+               return STD_CONFIG_LIST(key, data, sdi, cg, scanopts, drvopts, devopts);
+
+       /*
+        * Check for device specific config_list handler. ERR N/A from
+        * that handler is non-fatal, just falls back to common logic.
+        */
+       dmm = (struct dmm_info *)sdi->driver;
+       if (dmm && dmm->config_list) {
+               ret = dmm->config_list(dmm->dmm_state, key, data, sdi, cg);
+               if (ret != SR_ERR_NA)
+                       return ret;
+       }
+
+       return STD_CONFIG_LIST(key, data, sdi, cg, scanopts, drvopts, devopts);
 }
 
 static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
+       struct dmm_info *dmm;
+       sr_receive_data_callback cb_func;
+       void *cb_data;
+       int ret;
        struct sr_serial_dev_inst *serial;
 
        devc = sdi->priv;
@@ -165,16 +272,28 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 50ms, or whenever some data comes in. */
+       cb_func = receive_data;
+       cb_data = (void *)sdi;
+       dmm = (struct dmm_info *)sdi->driver;
+       if (dmm && dmm->acquire_start) {
+               ret = dmm->acquire_start(dmm->dmm_state, sdi,
+                       &cb_func, &cb_data);
+               if (ret < 0)
+                       return ret;
+       }
+
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
-                     receive_data, (void *)sdi);
+               cb_func, cb_data);
 
        return SR_OK;
 }
 
-#define DMM(ID, CHIPSET, VENDOR, MODEL, CONN, BAUDRATE, PACKETSIZE, TIMEOUT, \
-                       DELAY, REQUEST, VALID, PARSE, DETAILS) \
+#define DMM_ENTRY(ID, CHIPSET, VENDOR, MODEL, \
+               CONN, SERIALCOMM, PACKETSIZE, TIMEOUT, DELAY, \
+               OPEN, REQUEST, VALID, PARSE, DETAILS, \
+               INIT_STATE, FREE_STATE, VALID_LEN, PARSE_LEN, \
+               CFG_GET, CFG_SET, CFG_LIST, ACQ_START) \
        &((struct dmm_info) { \
                { \
                        .name = ID, \
@@ -185,7 +304,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
                        .scan = scan, \
                        .dev_list = std_dev_list, \
                        .dev_clear = std_dev_clear, \
-                       .config_get = NULL, \
+                       .config_get = config_get, \
                        .config_set = config_set, \
                        .config_list = config_list, \
                        .dev_open = std_serial_dev_open, \
@@ -194,10 +313,36 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
                        .dev_acquisition_stop = std_serial_dev_acquisition_stop, \
                        .context = NULL, \
                }, \
-               VENDOR, MODEL, CONN, BAUDRATE, PACKETSIZE, TIMEOUT, DELAY, \
-               REQUEST, VALID, PARSE, DETAILS, sizeof(struct CHIPSET##_info) \
+               VENDOR, MODEL, CONN, SERIALCOMM, PACKETSIZE, TIMEOUT, DELAY, \
+               REQUEST, 1, NULL, VALID, PARSE, DETAILS, \
+               sizeof(struct CHIPSET##_info), \
+               NULL, INIT_STATE, FREE_STATE, \
+               OPEN, VALID_LEN, PARSE_LEN, \
+               CFG_GET, CFG_SET, CFG_LIST, ACQ_START, \
        }).di
 
+#define DMM_CONN(ID, CHIPSET, VENDOR, MODEL, \
+               CONN, SERIALCOMM, PACKETSIZE, TIMEOUT, DELAY, \
+               REQUEST, VALID, PARSE, DETAILS) \
+       DMM_ENTRY(ID, CHIPSET, VENDOR, MODEL, \
+               CONN, SERIALCOMM, PACKETSIZE, TIMEOUT, DELAY, \
+               NULL, REQUEST, VALID, PARSE, DETAILS, \
+               NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL)
+
+#define DMM(ID, CHIPSET, VENDOR, MODEL, SERIALCOMM, PACKETSIZE, TIMEOUT, \
+               DELAY, REQUEST, VALID, PARSE, DETAILS) \
+       DMM_CONN(ID, CHIPSET, VENDOR, MODEL, \
+               NULL, SERIALCOMM, PACKETSIZE, TIMEOUT, DELAY, \
+               REQUEST, VALID, PARSE, DETAILS)
+
+#define DMM_LEN(ID, CHIPSET, VENDOR, MODEL, \
+               CONN, SERIALCOMM, PACKETSIZE, TIMEOUT, DELAY, \
+               INIT, FREE, OPEN, REQUEST, VALID, PARSE, DETAILS) \
+       DMM_ENTRY(ID, CHIPSET, VENDOR, MODEL, \
+               CONN, SERIALCOMM, PACKETSIZE, TIMEOUT, DELAY, \
+               OPEN, REQUEST, NULL, NULL, DETAILS, \
+               INIT, FREE, VALID, PARSE, NULL, NULL, NULL, NULL)
+
 SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        /*
         * The items are sorted by chipset first and then model name.
@@ -214,7 +359,7 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        /* asycii based meters {{{ */
        DMM(
                "metrix-mx56c", asycii, "Metrix", "MX56C",
-               "2400/8n1", 2400, ASYCII_PACKET_SIZE, 0, 0, NULL,
+               "2400/8n1", ASYCII_PACKET_SIZE, 0, 0, NULL,
                sr_asycii_packet_valid, sr_asycii_parse, NULL
        ),
        /* }}} */
@@ -222,30 +367,73 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "brymen-bm25x", bm25x,
                "Brymen", "BM25x", "9600/8n1/rts=1/dtr=1",
-               9600, BRYMEN_BM25X_PACKET_SIZE, 0, 0, NULL,
+               BRYMEN_BM25X_PACKET_SIZE, 0, 0, NULL,
                sr_brymen_bm25x_packet_valid, sr_brymen_bm25x_parse,
                NULL
        ),
        /* }}} */
+       /* bm52x based meters {{{ */
+       DMM_CONN(
+               "brymen-bm52x", brymen_bm52x, "Brymen", "BM52x",
+               "hid/bu86x", NULL, BRYMEN_BM52X_PACKET_SIZE, 4000, 500,
+               sr_brymen_bm52x_packet_request,
+               sr_brymen_bm52x_packet_valid, sr_brymen_bm52x_parse,
+               NULL
+       ),
+       DMM_CONN(
+               "brymen-bm82x", brymen_bm52x, "Brymen", "BM82x",
+               "hid/bu86x", NULL, BRYMEN_BM52X_PACKET_SIZE, 4000, 500,
+               sr_brymen_bm82x_packet_request,
+               sr_brymen_bm82x_packet_valid, sr_brymen_bm52x_parse,
+               NULL
+       ),
+       /* }}} */
+       /* bm85x based meters {{{ */
+       DMM_LEN(
+               "brymen-bm85x", brymen_bm85x, "Brymen", "BM85x",
+               NULL, "9600/8n1/dtr=1/rts=1",
+               BRYMEN_BM85x_PACKET_SIZE_MIN, 2000, 400,
+               NULL, NULL, /* INIT/FREE for DMM state */
+               brymen_bm85x_after_open, brymen_bm85x_packet_request,
+               brymen_bm85x_packet_valid, brymen_bm85x_parse,
+               NULL
+       ),
+       /* }}} */
+       /* bm86x based meters {{{ */
+       DMM_CONN(
+               "brymen-bm86x", brymen_bm86x, "Brymen", "BM86x",
+               "hid/bu86x", NULL, BRYMEN_BM86X_PACKET_SIZE, 500, 100,
+               sr_brymen_bm86x_packet_request,
+               sr_brymen_bm86x_packet_valid, sr_brymen_bm86x_parse,
+               NULL
+       ),
+       /* }}} */
        /* dtm0660 based meters {{{ */
        DMM(
                "peaktech-3415", dtm0660,
                "PeakTech", "3415", "2400/8n1/rts=0/dtr=1",
-               2400, DTM0660_PACKET_SIZE, 0, 0, NULL,
+               DTM0660_PACKET_SIZE, 0, 0, NULL,
                sr_dtm0660_packet_valid, sr_dtm0660_parse, NULL
        ),
        DMM(
                "velleman-dvm4100", dtm0660,
                "Velleman", "DVM4100", "2400/8n1/rts=0/dtr=1",
-               2400, DTM0660_PACKET_SIZE, 0, 0, NULL,
+               DTM0660_PACKET_SIZE, 0, 0, NULL,
                sr_dtm0660_packet_valid, sr_dtm0660_parse, NULL
        ),
        /* }}} */
+       /* eev121gw based meters {{{ */
+       DMM(
+               "eevblog-121gw", eev121gw, "EEVblog", "121GW",
+               "115200/8n1", EEV121GW_PACKET_SIZE, 0, 0, NULL,
+               sr_eev121gw_packet_valid, sr_eev121gw_3displays_parse, NULL
+       ),
+       /* }}} */
        /* es519xx based meters {{{ */
        DMM(
                "iso-tech-idm103n", es519xx,
                "ISO-TECH", "IDM103N", "2400/7o1/rts=0/dtr=1",
-               2400, ES519XX_11B_PACKET_SIZE, 0, 0, NULL,
+               ES519XX_11B_PACKET_SIZE, 0, 0, NULL,
                sr_es519xx_2400_11b_packet_valid, sr_es519xx_2400_11b_parse,
                NULL
        ),
@@ -261,21 +449,21 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "tenma-72-7750-ser", es519xx,
                "Tenma", "72-7750 (UT-D02 cable)", "19200/7o1/rts=0/dtr=1",
-               19200, ES519XX_11B_PACKET_SIZE, 0, 0, NULL,
+               ES519XX_11B_PACKET_SIZE, 0, 0, NULL,
                sr_es519xx_19200_11b_packet_valid, sr_es519xx_19200_11b_parse,
                NULL
        ),
        DMM(
                "uni-t-ut60g-ser", es519xx,
                "UNI-T", "UT60G (UT-D02 cable)", "19200/7o1/rts=0/dtr=1",
-               19200, ES519XX_11B_PACKET_SIZE, 0, 0, NULL,
+               ES519XX_11B_PACKET_SIZE, 0, 0, NULL,
                sr_es519xx_19200_11b_packet_valid, sr_es519xx_19200_11b_parse,
                NULL
        ),
        DMM(
                "uni-t-ut61e-ser", es519xx,
                "UNI-T", "UT61E (UT-D02 cable)", "19200/7o1/rts=0/dtr=1",
-               19200, ES519XX_14B_PACKET_SIZE, 0, 0, NULL,
+               ES519XX_14B_PACKET_SIZE, 0, 0, NULL,
                sr_es519xx_19200_14b_packet_valid, sr_es519xx_19200_14b_parse,
                NULL
        ),
@@ -283,7 +471,7 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        /* fs9721 based meters {{{ */
        DMM(
                "digitek-dt4000zc", fs9721,
-               "Digitek", "DT4000ZC", "2400/8n1/dtr=1", 2400,
+               "Digitek", "DT4000ZC", "2400/8n1/dtr=1",
                FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_10_temp_c
@@ -291,20 +479,20 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "mastech-ms8250b", fs9721,
                "MASTECH", "MS8250B", "2400/8n1/rts=0/dtr=1",
-               2400, FS9721_PACKET_SIZE, 0, 0, NULL,
+               FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                NULL
        ),
        DMM(
                "pce-pce-dm32", fs9721,
-               "PCE", "PCE-DM32", "2400/8n1", 2400,
+               "PCE", "PCE-DM32", "2400/8n1",
                FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_01_10_temp_f_c
        ),
        DMM(
                "peaktech-3330", fs9721,
-               "PeakTech", "3330", "2400/8n1/dtr=1", 2400,
+               "PeakTech", "3330", "2400/8n1/dtr=1",
                FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_01_10_temp_f_c
@@ -312,13 +500,13 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "tecpel-dmm-8061-ser", fs9721,
                "Tecpel", "DMM-8061 (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9721_PACKET_SIZE, 0, 0, NULL,
+               FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_00_temp_c
        ),
        DMM(
                "tekpower-tp4000ZC", fs9721,
-               "TekPower", "TP4000ZC", "2400/8n1/dtr=1", 2400,
+               "TekPower", "TP4000ZC", "2400/8n1/dtr=1",
                FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_10_temp_c
@@ -326,34 +514,34 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "tenma-72-7745-ser", fs9721,
                "Tenma", "72-7745 (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9721_PACKET_SIZE, 0, 0, NULL,
+               FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_00_temp_c
        ),
        DMM(
                "uni-t-ut60a-ser", fs9721,
                "UNI-T", "UT60A (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9721_PACKET_SIZE, 0, 0, NULL,
+               FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                NULL
        ),
        DMM(
                "uni-t-ut60e-ser", fs9721,
                "UNI-T", "UT60E (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9721_PACKET_SIZE, 0, 0, NULL,
+               FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_00_temp_c
        ),
        DMM(
                "va-va18b", fs9721,
-               "V&A", "VA18B", "2400/8n1", 2400,
+               "V&A", "VA18B", "2400/8n1",
                FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_01_temp_c
        ),
        DMM(
                "va-va40b", fs9721,
-               "V&A", "VA40B", "2400/8n1", 2400,
+               "V&A", "VA40B", "2400/8n1",
                FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_max_c_min
@@ -361,41 +549,58 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "voltcraft-vc820-ser", fs9721,
                "Voltcraft", "VC-820 (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9721_PACKET_SIZE, 0, 0, NULL,
+               FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                NULL
        ),
        DMM(
                "voltcraft-vc840-ser", fs9721,
                "Voltcraft", "VC-840 (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9721_PACKET_SIZE, 0, 0, NULL,
+               FS9721_PACKET_SIZE, 0, 0, NULL,
                sr_fs9721_packet_valid, sr_fs9721_parse,
                sr_fs9721_00_temp_c
        ),
        /* }}} */
        /* fs9922 based meters {{{ */
+       DMM(
+               "gwinstek-gdm-397", fs9922,
+               "GW Instek", "GDM-397", "2400/8n1/rts=0/dtr=1",
+               FS9922_PACKET_SIZE, 0, 0, NULL,
+               sr_fs9922_packet_valid, sr_fs9922_parse, NULL
+       ),
+       DMM(
+               "peaktech-2025", fs9922,
+               "PeakTech", "2025", "2400/8n1/rts=0/dtr=1",
+               FS9922_PACKET_SIZE, 0, 0, NULL,
+               sr_fs9922_packet_valid, sr_fs9922_parse, NULL
+       ),
        DMM(
                "sparkfun-70c", fs9922,
                "SparkFun", "70C", "2400/8n1/rts=0/dtr=1",
-               2400, FS9922_PACKET_SIZE, 0, 0, NULL,
+               FS9922_PACKET_SIZE, 0, 0, NULL,
                sr_fs9922_packet_valid, sr_fs9922_parse, NULL
        ),
        DMM(
                "uni-t-ut61b-ser", fs9922,
                "UNI-T", "UT61B (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9922_PACKET_SIZE, 0, 0, NULL,
+               FS9922_PACKET_SIZE, 0, 0, NULL,
                sr_fs9922_packet_valid, sr_fs9922_parse, NULL
        ),
        DMM(
                "uni-t-ut61c-ser", fs9922,
                "UNI-T", "UT61C (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9922_PACKET_SIZE, 0, 0, NULL,
+               FS9922_PACKET_SIZE, 0, 0, NULL,
                sr_fs9922_packet_valid, sr_fs9922_parse, NULL
        ),
        DMM(
                "uni-t-ut61d-ser", fs9922,
                "UNI-T", "UT61D (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9922_PACKET_SIZE, 0, 0, NULL,
+               FS9922_PACKET_SIZE, 0, 0, NULL,
+               sr_fs9922_packet_valid, sr_fs9922_parse, NULL
+       ),
+       DMM_CONN(
+               "victor-dmm", fs9922, "Victor", "Victor DMMs",
+               "hid/victor", "2400/8n1", FS9922_PACKET_SIZE, 0, 0, NULL,
                sr_fs9922_packet_valid, sr_fs9922_parse, NULL
        ),
        DMM(
@@ -406,7 +611,7 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
                 */
                "voltcraft-vc830-ser", fs9922,
                "Voltcraft", "VC-830 (UT-D02 cable)", "2400/8n1/rts=0/dtr=1",
-               2400, FS9922_PACKET_SIZE, 0, 0, NULL,
+               FS9922_PACKET_SIZE, 0, 0, NULL,
                sr_fs9922_packet_valid, sr_fs9922_parse,
                &sr_fs9922_z1_diode
        ),
@@ -414,102 +619,150 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        /* m2110 based meters {{{ */
        DMM(
                "bbcgm-2010", m2110,
-               "BBC Goertz Metrawatt", "M2110", "1200/7n2", 1200,
+               "BBC Goertz Metrawatt", "M2110", "1200/7n2",
                BBCGM_M2110_PACKET_SIZE, 0, 0, NULL,
                sr_m2110_packet_valid, sr_m2110_parse,
                NULL
        ),
        /* }}} */
+       /* meterman_38xr based meters {{{ */
+       DMM(
+               "meterman-38xr", meterman_38xr,
+               "Meterman", "38XR", "9600/8n1/rts=0/dtr=1",
+               METERMAN_38XR_PACKET_SIZE, 0, 0, NULL,
+               meterman_38xr_packet_valid, meterman_38xr_parse,
+               NULL
+       ),
+       /* }}} */
        /* metex14 based meters {{{ */
        DMM(
                "mastech-mas345", metex14,
-               "MASTECH", "MAS345", "600/7n2/rts=0/dtr=1", 600,
+               "MASTECH", "MAS345", "600/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "metex-m3640d", metex14,
-               "Metex", "M-3640D", "1200/7n2/rts=0/dtr=1", 1200,
+               "Metex", "M-3640D", "1200/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
+       DMM(
+               "metex-m3860m", metex14,
+               "Metex", "M-3860M", "9600/7n2/rts=0/dtr=1",
+               4 * METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
+               sr_metex14_4packets_valid, sr_metex14_4packets_parse,
+               NULL
+       ),
        DMM(
                "metex-m4650cr", metex14,
-               "Metex", "M-4650CR", "1200/7n2/rts=0/dtr=1", 1200,
+               "Metex", "M-4650CR", "1200/7n2/rts=0/dtr=1",
+               METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
+               sr_metex14_packet_valid, sr_metex14_parse,
+               NULL
+       ),
+       DMM(
+               "metex-me21", metex14,
+               "Metex", "ME-21", "2400/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "metex-me31", metex14,
-               "Metex", "ME-31", "600/7n2/rts=0/dtr=1", 600,
+               "Metex", "ME-31", "600/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "peaktech-3410", metex14,
-               "PeakTech", "3410", "600/7n2/rts=0/dtr=1", 600,
+               "PeakTech", "3410", "600/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "peaktech-4370", metex14,
-               "PeakTech", "4370", "1200/7n2/rts=0/dtr=1", 1200,
+               "PeakTech", "4370", "1200/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
+       DMM(
+               "peaktech-4390a", metex14,
+               "PeakTech", "4390A", "9600/7n2/rts=0/dtr=1",
+               4 * METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
+               sr_metex14_4packets_valid, sr_metex14_4packets_parse,
+               NULL
+       ),
        DMM(
                "radioshack-22-168", metex14,
-               "RadioShack", "22-168", "1200/7n2/rts=0/dtr=1", 1200,
+               "RadioShack", "22-168", "1200/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "radioshack-22-805", metex14,
-               "RadioShack", "22-805", "600/7n2/rts=0/dtr=1", 600,
+               "RadioShack", "22-805", "600/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "voltcraft-m3650cr", metex14,
-               "Voltcraft", "M-3650CR", "1200/7n2/rts=0/dtr=1", 1200,
+               "Voltcraft", "M-3650CR", "1200/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 150, 20, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "voltcraft-m3650d", metex14,
-               "Voltcraft", "M-3650D", "1200/7n2/rts=0/dtr=1", 1200,
+               "Voltcraft", "M-3650D", "1200/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "voltcraft-m4650cr", metex14,
-               "Voltcraft", "M-4650CR", "1200/7n2/rts=0/dtr=1", 1200,
+               "Voltcraft", "M-4650CR", "1200/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 0, 0, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        DMM(
                "voltcraft-me42", metex14,
-               "Voltcraft", "ME-42", "600/7n2/rts=0/dtr=1", 600,
+               "Voltcraft", "ME-42", "600/7n2/rts=0/dtr=1",
                METEX14_PACKET_SIZE, 250, 60, sr_metex14_packet_request,
                sr_metex14_packet_valid, sr_metex14_parse,
                NULL
        ),
        /* }}} */
+       /* ms2115b based meters {{{ */
+       DMM(
+               "mastech-ms2115b", ms2115b,
+               "MASTECH", "MS2115B", "1200/8n1",
+               MS2115B_PACKET_SIZE, 0, 0, NULL,
+               sr_ms2115b_packet_valid, sr_ms2115b_parse,
+               NULL
+       ),
+       /* }}} */
+       /* ms8250d based meters {{{ */
+       DMM(
+               "mastech-ms8250d", ms8250d,
+               "MASTECH", "MS8250D", "2400/8n1/rts=0/dtr=1",
+               MS8250D_PACKET_SIZE, 0, 0, NULL,
+               sr_ms8250d_packet_valid, sr_ms8250d_parse,
+               NULL
+       ),
+       /* }}} */
        /* rs9lcd based meters {{{ */
        DMM(
                "radioshack-22-812", rs9lcd,
-               "RadioShack", "22-812", "4800/8n1/rts=0/dtr=1", 4800,
+               "RadioShack", "22-812", "4800/8n1/rts=0/dtr=1",
                RS9LCD_PACKET_SIZE, 0, 0, NULL,
                sr_rs9lcd_packet_valid, sr_rs9lcd_parse,
                NULL
@@ -519,67 +772,73 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "tenma-72-7730-ser", ut71x,
                "Tenma", "72-7730 (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "tenma-72-7732-ser", ut71x,
                "Tenma", "72-7732 (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "tenma-72-9380a-ser", ut71x,
                "Tenma", "72-9380A (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "uni-t-ut71a-ser", ut71x,
                "UNI-T", "UT71A (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "uni-t-ut71b-ser", ut71x,
                "UNI-T", "UT71B (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "uni-t-ut71c-ser", ut71x,
                "UNI-T", "UT71C (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "uni-t-ut71d-ser", ut71x,
                "UNI-T", "UT71D (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "uni-t-ut71e-ser", ut71x,
                "UNI-T", "UT71E (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
+               sr_ut71x_packet_valid, sr_ut71x_parse, NULL
+       ),
+       DMM(
+               "uni-t-ut804-ser", ut71x,
+               "UNI-T", "UT804", "2400/7o1/rts=0/dtr=1",
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "voltcraft-vc920-ser", ut71x,
                "Voltcraft", "VC-920 (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "voltcraft-vc940-ser", ut71x,
                "Voltcraft", "VC-940 (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        DMM(
                "voltcraft-vc960-ser", ut71x,
                "Voltcraft", "VC-960 (UT-D02 cable)", "2400/7o1/rts=0/dtr=1",
-               2400, UT71X_PACKET_SIZE, 0, 0, NULL,
+               UT71X_PACKET_SIZE, 0, 0, NULL,
                sr_ut71x_packet_valid, sr_ut71x_parse, NULL
        ),
        /* }}} */
@@ -587,10 +846,19 @@ SR_REGISTER_DEV_DRIVER_LIST(serial_dmm_drivers,
        DMM(
                "voltcraft-vc870-ser", vc870,
                "Voltcraft", "VC-870 (UT-D02 cable)", "9600/8n1/rts=0/dtr=1",
-               9600, VC870_PACKET_SIZE, 0, 0, NULL,
+               VC870_PACKET_SIZE, 0, 0, NULL,
                sr_vc870_packet_valid, sr_vc870_parse, NULL
        ),
        /* }}} */
+       /* vc96 based meters {{{ */
+       DMM(
+               "voltcraft-vc96", vc96,
+               "Voltcraft", "VC-96", "1200/8n2",
+               VC96_PACKET_SIZE, 0, 0, NULL,
+               sr_vc96_packet_valid, sr_vc96_parse,
+               NULL
+       ),
+       /* }}} */
        /*
         * The list is sorted. Add new items in the respective chip's group.
         */