]> sigrok.org Git - libsigrok.git/blobdiff - src/hardware/serial-dmm/protocol.c
serial-dmm: only do bytes to text conversion at higher log levels
[libsigrok.git] / src / hardware / serial-dmm / protocol.c
index bcac8500c4acfededb67461b12488ac5658b0257..71fc1a4635c480f646859f8ed62a2f63f0b66eca 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+#include <config.h>
 #include <stdlib.h>
 #include <math.h>
 #include <string.h>
-#include <errno.h>
 #include <glib.h>
-#include "libsigrok.h"
+#include <libsigrok/libsigrok.h>
 #include "libsigrok-internal.h"
 #include "protocol.h"
 
-static void log_dmm_packet(const uint8_t *buf)
+static void log_dmm_packet(const uint8_t *buf, size_t len)
 {
-       sr_dbg("DMM packet: %02x %02x %02x %02x %02x %02x %02x "
-              "%02x %02x %02x %02x %02x %02x %02x %02x %02x "
-              "%02x %02x %02x %02x %02x %02x %02x",
-              buf[0], buf[1], buf[2], buf[3], buf[4], buf[5], buf[6],
-              buf[7], buf[8], buf[9], buf[10], buf[11], buf[12], buf[13],
-              buf[14], buf[15], buf[16], buf[17], buf[18], buf[19], buf[20],
-              buf[21], buf[22]);
+       GString *text;
+
+       if (sr_log_loglevel_get() < SR_LOG_DBG)
+               return;
+
+       text = sr_hexdump_new(buf, len);
+       sr_dbg("DMM packet: %s", text->str);
+       sr_hexdump_free(text);
 }
 
 static void handle_packet(const uint8_t *buf, struct sr_dev_inst *sdi,
-                         int dmm, void *info)
+                         void *info)
 {
+       struct dmm_info *dmm;
        float floatval;
        struct sr_datafeed_packet packet;
        struct sr_datafeed_analog analog;
+       struct sr_analog_encoding encoding;
+       struct sr_analog_meaning meaning;
+       struct sr_analog_spec spec;
        struct dev_context *devc;
+       gboolean sent_sample;
+       struct sr_channel *channel;
+       size_t ch_idx;
 
-       log_dmm_packet(buf);
-       devc = sdi->priv;
-
-       memset(&analog, 0, sizeof(struct sr_datafeed_analog));
+       dmm = (struct dmm_info *)sdi->driver;
 
-       analog.channels = sdi->channels;
-       analog.num_samples = 1;
-       analog.mq = -1;
-
-       dmms[dmm].packet_parse(buf, &floatval, &analog, info);
-       analog.data = &floatval;
+       log_dmm_packet(buf, dmm->packet_size);
+       devc = sdi->priv;
 
-       /* If this DMM needs additional handling, call the resp. function. */
-       if (dmms[dmm].dmm_details)
-               dmms[dmm].dmm_details(&analog, info);
+       sent_sample = FALSE;
+       memset(info, 0, dmm->info_size);
+       for (ch_idx = 0; ch_idx < dmm->channel_count; ch_idx++) {
+               /* Note: digits/spec_digits will be overridden by the DMM parsers. */
+               sr_analog_init(&analog, &encoding, &meaning, &spec, 0);
+
+               channel = g_slist_nth_data(sdi->channels, ch_idx);
+               analog.meaning->channels = g_slist_append(NULL, channel);
+               analog.num_samples = 1;
+               analog.meaning->mq = 0;
+
+               dmm->packet_parse(buf, &floatval, &analog, info);
+               analog.data = &floatval;
+
+               /* If this DMM needs additional handling, call the resp. function. */
+               if (dmm->dmm_details)
+                       dmm->dmm_details(&analog, info);
+
+               if (analog.meaning->mq != 0 && channel->enabled) {
+                       /* Got a measurement. */
+                       packet.type = SR_DF_ANALOG;
+                       packet.payload = &analog;
+                       sr_session_send(sdi, &packet);
+                       sent_sample = TRUE;
+               }
+       }
 
-       if (analog.mq != -1) {
-               /* Got a measurement. */
-               packet.type = SR_DF_ANALOG;
-               packet.payload = &analog;
-               sr_session_send(devc->cb_data, &packet);
-               devc->num_samples++;
+       if (sent_sample) {
+               sr_sw_limits_update_samples_read(&devc->limits, 1);
        }
 }
 
 /** Request packet, if required. */
-SR_PRIV int req_packet(struct sr_dev_inst *sdi, int dmm)
+SR_PRIV int req_packet(struct sr_dev_inst *sdi)
 {
+       struct dmm_info *dmm;
        struct dev_context *devc;
        struct sr_serial_dev_inst *serial;
        int ret;
 
-       if (!dmms[dmm].packet_request)
+       dmm = (struct dmm_info *)sdi->driver;
+
+       if (!dmm->packet_request)
                return SR_OK;
 
        devc = sdi->priv;
@@ -90,24 +113,27 @@ SR_PRIV int req_packet(struct sr_dev_inst *sdi, int dmm)
                return SR_OK;
        }
 
-       ret = dmms[dmm].packet_request(serial);
+       ret = dmm->packet_request(serial);
        if (ret < 0) {
                sr_err("Failed to request packet: %d.", ret);
                return ret;
        }
 
-       if (dmms[dmm].req_timeout_ms)
-               devc->req_next_at = g_get_monotonic_time() + (dmms[dmm].req_timeout_ms * 1000);
+       if (dmm->req_timeout_ms)
+               devc->req_next_at = g_get_monotonic_time() + (dmm->req_timeout_ms * 1000);
 
        return SR_OK;
 }
 
-static void handle_new_data(struct sr_dev_inst *sdi, int dmm, void *info)
+static void handle_new_data(struct sr_dev_inst *sdi, void *info)
 {
+       struct dmm_info *dmm;
        struct dev_context *devc;
-       int len, i, offset = 0;
+       int len, offset;
        struct sr_serial_dev_inst *serial;
 
+       dmm = (struct dmm_info *)sdi->driver;
+
        devc = sdi->priv;
        serial = sdi->conn;
 
@@ -123,34 +149,36 @@ static void handle_new_data(struct sr_dev_inst *sdi, int dmm, void *info)
        devc->buflen += len;
 
        /* Now look for packets in that data. */
-       while ((devc->buflen - offset) >= dmms[dmm].packet_size) {
-               if (dmms[dmm].packet_valid(devc->buf + offset)) {
-                       handle_packet(devc->buf + offset, sdi, dmm, info);
-                       offset += dmms[dmm].packet_size;
+       offset = 0;
+       while ((devc->buflen - offset) >= dmm->packet_size) {
+               if (dmm->packet_valid(devc->buf + offset)) {
+                       handle_packet(devc->buf + offset, sdi, info);
+                       offset += dmm->packet_size;
 
                        /* Request next packet, if required. */
-                       if (!dmms[dmm].packet_request)
+                       if (!dmm->packet_request)
                                break;
-                       if (dmms[dmm].req_timeout_ms || dmms[dmm].req_delay_ms)
+                       if (dmm->req_timeout_ms || dmm->req_delay_ms)
                                devc->req_next_at = g_get_monotonic_time() +
-                                       dmms[dmm].req_delay_ms * 1000;
-                       req_packet(sdi, dmm);
+                                       dmm->req_delay_ms * 1000;
+                       req_packet(sdi);
                } else {
                        offset++;
                }
        }
 
        /* If we have any data left, move it to the beginning of our buffer. */
-       for (i = 0; i < devc->buflen - offset; i++)
-               devc->buf[i] = devc->buf[offset + i];
+       if (devc->buflen > offset)
+               memmove(devc->buf, devc->buf + offset, devc->buflen - offset);
        devc->buflen -= offset;
 }
 
-static int receive_data(int fd, int revents, int dmm, void *info, void *cb_data)
+int receive_data(int fd, int revents, void *cb_data)
 {
        struct sr_dev_inst *sdi;
        struct dev_context *devc;
-       int64_t time;
+       struct dmm_info *dmm;
+       void *info;
 
        (void)fd;
 
@@ -160,80 +188,21 @@ static int receive_data(int fd, int revents, int dmm, void *info, void *cb_data)
        if (!(devc = sdi->priv))
                return TRUE;
 
+       dmm = (struct dmm_info *)sdi->driver;
+
        if (revents == G_IO_IN) {
                /* Serial data arrived. */
-               handle_new_data(sdi, dmm, info);
+               info = g_malloc(dmm->info_size);
+               handle_new_data(sdi, info);
+               g_free(info);
        } else {
                /* Timeout; send another packet request if DMM needs it. */
-               if (dmms[dmm].packet_request && (req_packet(sdi, dmm) < 0))
+               if (dmm->packet_request && (req_packet(sdi) < 0))
                        return FALSE;
        }
 
-       if (devc->limit_samples && devc->num_samples >= devc->limit_samples) {
-               sr_info("Requested number of samples reached.");
-               sdi->driver->dev_acquisition_stop(sdi, cb_data);
-               return TRUE;
-       }
-
-       if (devc->limit_msec) {
-               time = (g_get_monotonic_time() - devc->starttime) / 1000;
-               if (time > (int64_t)devc->limit_msec) {
-                       sr_info("Requested time limit reached.");
-                       sdi->driver->dev_acquisition_stop(sdi, cb_data);
-                       return TRUE;
-               }
-       }
+       if (sr_sw_limits_check(&devc->limits))
+               sr_dev_acquisition_stop(sdi);
 
        return TRUE;
 }
-
-#define RECEIVE_DATA(ID_UPPER, DMM_DRIVER) \
-SR_PRIV int receive_data_##ID_UPPER(int fd, int revents, void *cb_data) { \
-       struct DMM_DRIVER##_info info; \
-       return receive_data(fd, revents, ID_UPPER, &info, cb_data); }
-
-/* Driver-specific receive_data() wrappers */
-RECEIVE_DATA(BBCGM_M2110, metex14) /* metex14_info used as a dummy. */
-RECEIVE_DATA(DIGITEK_DT4000ZC, fs9721)
-RECEIVE_DATA(TEKPOWER_TP4000ZC, fs9721)
-RECEIVE_DATA(METEX_ME31, metex14)
-RECEIVE_DATA(PEAKTECH_3410, metex14)
-RECEIVE_DATA(MASTECH_MAS345, metex14)
-RECEIVE_DATA(MASTECH_MS8250B, fs9721)
-RECEIVE_DATA(VA_VA18B, fs9721)
-RECEIVE_DATA(VA_VA40B, fs9721)
-RECEIVE_DATA(METEX_M3640D, metex14)
-RECEIVE_DATA(METEX_M4650CR, metex14)
-RECEIVE_DATA(PEAKTECH_4370, metex14)
-RECEIVE_DATA(PCE_PCE_DM32, fs9721)
-RECEIVE_DATA(RADIOSHACK_22_168, metex14)
-RECEIVE_DATA(RADIOSHACK_22_805, metex14)
-RECEIVE_DATA(RADIOSHACK_22_812, rs9lcd)
-RECEIVE_DATA(TECPEL_DMM_8061_SER, fs9721)
-RECEIVE_DATA(VOLTCRAFT_M3650CR, metex14)
-RECEIVE_DATA(VOLTCRAFT_M3650D, metex14)
-RECEIVE_DATA(VOLTCRAFT_M4650CR, metex14)
-RECEIVE_DATA(VOLTCRAFT_ME42, metex14)
-RECEIVE_DATA(VOLTCRAFT_VC820_SER, fs9721)
-RECEIVE_DATA(VOLTCRAFT_VC830_SER, fs9922)
-RECEIVE_DATA(VOLTCRAFT_VC840_SER, fs9721)
-RECEIVE_DATA(VOLTCRAFT_VC870_SER, vc870)
-RECEIVE_DATA(VOLTCRAFT_VC920_SER, ut71x)
-RECEIVE_DATA(VOLTCRAFT_VC940_SER, ut71x)
-RECEIVE_DATA(VOLTCRAFT_VC960_SER, ut71x)
-RECEIVE_DATA(UNI_T_UT60A_SER, fs9721)
-RECEIVE_DATA(UNI_T_UT60E_SER, fs9721)
-RECEIVE_DATA(UNI_T_UT60G_SER, es519xx)
-RECEIVE_DATA(UNI_T_UT61B_SER, fs9922)
-RECEIVE_DATA(UNI_T_UT61C_SER, fs9922)
-RECEIVE_DATA(UNI_T_UT61D_SER, fs9922)
-RECEIVE_DATA(UNI_T_UT61E_SER, es519xx)
-RECEIVE_DATA(UNI_T_UT71A_SER, ut71x)
-RECEIVE_DATA(UNI_T_UT71B_SER, ut71x)
-RECEIVE_DATA(UNI_T_UT71C_SER, ut71x)
-RECEIVE_DATA(UNI_T_UT71D_SER, ut71x)
-RECEIVE_DATA(UNI_T_UT71E_SER, ut71x)
-RECEIVE_DATA(ISO_TECH_IDM103N, es519xx)
-RECEIVE_DATA(TENMA_72_7745_SER, fs9721)
-RECEIVE_DATA(TENMA_72_7750_SER, es519xx)
-RECEIVE_DATA(BRYMEN_BM25X, bm25x)