]> sigrok.org Git - libsigrok.git/blobdiff - src/hardware/raspberrypi-pico/protocol.c
raspberrypi-pico: Cleanup whitespace, add a serial_close to api.c scan function
[libsigrok.git] / src / hardware / raspberrypi-pico / protocol.c
index 0e954e8d427bb794e8404f671ca0aef617ce8a6e..a9ca47238d7116407c6ac24f2c6104350fcf730d 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * This file is part of the libsigrok project.
  *
- * Copyright (C) 2022 AC0BI <ac0bi00@gmail.com>
+ * Copyright (C) 2022 Shawn Walker <ac0bi00@gmail.com>
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+#define _GNU_SOURCE
+
 #include <config.h>
+#include <errno.h>
+#include <glib.h>
+#include <math.h>
+#include <stdlib.h>
+#include <stdarg.h>
+#include <string.h>
+#include <time.h>
+#include <unistd.h>
+#include <libsigrok/libsigrok.h>
+#include "libsigrok-internal.h"
 #include "protocol.h"
 
-SR_PRIV int raspberrypi_pico_receive_data(int fd, int revents, void *cb_data)
+SR_PRIV int send_serial_str(struct sr_serial_dev_inst *serial, char *str)
 {
-       const struct sr_dev_inst *sdi;
-       struct dev_context *devc;
+       int len = strlen(str);
+       if ((len > 15) || (len < 1)) {  //limit length to catch errant strings
+               sr_err("ERROR:Serial string len %d invalid ", len);
+               return SR_ERR;
+       }
+       //100ms timeout. With USB CDC serial we can't define the timeout
+       //based on link rate, so just pick something large as we shouldn't normally see them
+       if (serial_write_blocking(serial, str, len, 100) != len) {
+               sr_err("ERROR:Serial str write failed");
+               return SR_ERR;
+       }
+
+       return SR_OK;
+}
+
+SR_PRIV int send_serial_char(struct sr_serial_dev_inst *serial, char ch)
+{
+       char buf[1];
+       buf[0] = ch;
+       if (serial_write_blocking(serial, buf, 1, 100) != 1) {  //100ms
+               sr_err("ERROR:Serial char write failed");
+               return SR_ERR;
+       }
+       return SR_OK;
+}
+
+//Issue a command that expects a string return, return length of string
+int send_serial_w_resp(struct sr_serial_dev_inst *serial, char *str,
+                      char *resp, size_t cnt)
+{
+       int num_read, i;
+       send_serial_str(serial, str);
+       //Using the serial_read_blocking function when reading a response of unknown length requires 
+       //a long worst case timeout to always be taken.  So, instead loop waiting for a first byte, and
+       //then a final small delay for the rest. 
+       for (i = 0; i < 1000; i++) {    //wait up to 1 second in ms increments
+               num_read = serial_read_blocking(serial, resp, cnt, 1);
+               if (num_read > 0)
+                       break;
+       }
+       //sr_spew("rwprsp1 i %d nr %d",i,num_read);
+       //Since the serial port is usb CDC we can't calculate timeouts based on baud rate but
+       //even if the response is split between two USB transfers 10ms should be plenty.
+       num_read +=
+           serial_read_blocking(serial, &(resp[num_read]), cnt - num_read,
+                                10);
+       //sr_spew("rwrsp2 nr %d",num_read);
+
+       if ((num_read < 1) || (num_read > 30)) {
+               sr_err("ERROR:Serial_w_resp failed (%d).", num_read);
+               return -1;
+       } else {
+               return num_read;
+       }
+}
+
+//Issue a command that expects a single char ack 
+SR_PRIV int send_serial_w_ack(struct sr_serial_dev_inst *serial, char *str)
+{
+       char buf[2];
+       int num_read;
+       //In case we have left over transfer from the device, drain them
+       while ((num_read = serial_read_blocking(serial, buf, 2, 10))) {
+               //sr_dbg("swack drops 2 previous bytes %d %d",buf[0],buf[1]);
+       }
+       send_serial_str(serial, str);
+       //1000ms timeout
+       num_read = serial_read_blocking(serial, buf, 1, 1000);
+       if ((num_read == 1) && (buf[0] == '*')) {
+               return SR_OK;
+       } else {
+               sr_err("ERROR:Serial_w_ack %s failed (%d).", str,
+                      num_read);
+               if (num_read) {
+                       sr_err("ack resp char %c d %d\n\r", buf[0],
+                              buf[0]);
+               }
+               return SR_ERR;
+       }
+}
+
+//Process incoming data stream assuming it is optimized packing of 4 channels or less
+//Each byte is 4 channels of data and a 3 bit rle value, or a larger rle value, or a control signal.
+//This also checks for aborts and ends.
+//If an end is seen we stop processing but do not check the byte_cnt
+//The output is a set of samples fed to process group to perform sw triggering and sending of data to the session
+//as well as maintenance of the serial rx byte cnt.
+//Since we can get huge rle values we chop them up for processing into smaller groups
+//In this mode we can always consume all bytes because there are no cases where the processing of one 
+//byte requires the one after it.
+void process_D4(struct sr_dev_inst *sdi, struct dev_context *d)
+{
+       uint32_t j;
+       uint8_t cbyte;
+       uint8_t cval;
+       uint32_t rlecnt = 0;
+       uint32_t sampcnt = 0;   //number of samples received with no rles
+       while (d->ser_rdptr < d->bytes_avail) {
+               cbyte = d->buffer[(d->ser_rdptr)];
+               //RLE only byte
+               if (cbyte >= 48 && cbyte <= 127) {
+                       rlecnt += (cbyte - 47) * 8;
+                       d->byte_cnt++;
+               } else if (cbyte >= 0x80) {     //sample with possible rle
+                       rlecnt += (cbyte & 0x70) >> 4;
+                       if (rlecnt) {
+                               //On a value change, duplicate the previous values first.
+                               //The maximum value of one rle is 640.
+                               //To ensure we don't overflow the sample buffer but still send it large chunks of data 
+                               //(to make the packet sends to the session efficient) only call process group after
+                               //a large number of samples have been seen.
+                               //Likely we could use the max rle value of 640 but 2048 gives some extra room.
+                               if ((rlecnt + d->cbuf_wrptr) >
+                                   (d->sample_buf_size - 2048)) {
+                                       //process_group is sent the number of slices which is just the cbufwrptr divided by the slice size
+                                       //This modulo check should never happen as long the calculations for dig_sample_bytes etc are 
+                                       //correct, but it's a good cross check for code development.
+                                       if ((d->cbuf_wrptr) %
+                                           (d->dig_sample_bytes)) {
+                                               sr_err
+                                                   ("Modulo fail %d %d ",
+                                                    d->cbuf_wrptr,
+                                                    d->dig_sample_bytes);
+                                       }
+                                       process_group(sdi, d,
+                                                     (d->cbuf_wrptr /
+                                                      d->dig_sample_bytes));
+                               }
+                               rle_memset(d, rlecnt);
+                               rlecnt = 0;
+                               sampcnt = 0;
+                       }
+                       //Finally add in the new values
+                       cval = cbyte & 0xF;
+                       d->d_data_buf[d->cbuf_wrptr++] = cval;
+                       //pad in all other bytes since the sessions even wants disabled channels reported
+                       for (j = 1; j < d->dig_sample_bytes; j++) {
+                               d->d_data_buf[d->cbuf_wrptr++] = 0;
+                       }
+                       sampcnt++;
+                       d->byte_cnt++;
+                       sr_spew
+                           ("Dchan4 rdptr %d wrptr %d bytein 0x%X rle %d cval 0x%X\n",
+                            (d->ser_rdptr) - 1, d->cbuf_wrptr, cbyte,
+                            rlecnt, cval);
+                       rlecnt = 0;
+
+                       d->d_last[0] = cval;
+               }
+               //Any other character ends parsing - it could be a frame error or a start of the final byte cnt
+               else {
+                       if (cbyte == '$') {
+                               sr_info
+                                   ("D4 Data stream stops with cbyte %d char %c rdidx %d cnt %llu",
+                                    cbyte, cbyte, d->ser_rdptr,
+                                    d->byte_cnt);
+                               d->rxstate = RX_STOPPED;
+                       } else {
+                               sr_err
+                                   ("D4 Data stream aborts with cbyte %d char %c rdidx %d cnt %llu",
+                                    cbyte, cbyte, d->ser_rdptr,
+                                    d->byte_cnt);
+                               d->rxstate = RX_ABORT;
+                       }
+                       break;  //break from while loop
+               }
+               (d->ser_rdptr)++;
+       }                       //while rdptr < wrptr
+       sr_spew("D4 while done rdptr %d", d->ser_rdptr);
+       //If we reach the end of the serial input stream send any remaining values or rles to the session
+       /*this can also be skipped now the rle_memset handles cbufwrptr
+          if(sampcnt){
+          process_group(sdi,d,sampcnt);
+          sampcnt=0;
+          }   
+        */
+       if (rlecnt) {
+               sr_spew("Residual D4 slice rlecnt %d", rlecnt);
+               rle_memset(d, rlecnt);
+       }
+       if (d->cbuf_wrptr) {
+               sr_spew("Residual D4 data wrptr %d", d->cbuf_wrptr);
+               process_group(sdi, d, d->cbuf_wrptr / d->dig_sample_bytes);
+
+       }
+
+}                              //Process_D4
+
+//Process incoming data stream and forward to trigger processing with process_group
+//The final value of ser_rdptr indicates how many bytes were processed.
+//This version handles all other enabled channel configurations that Process_D4 doesn't
+void process_slice(struct sr_dev_inst *sdi, struct dev_context *devc)
+{
+       int32_t i;
+       uint32_t tmp32;
+       uint8_t cbyte;
+       uint32_t slices_avail = 0;
+       uint32_t cword;
+       uint32_t slice_bytes;   //number of bytes that have legal slice values
+       //Only process legal data values for this mode which are >=0x80
+       for (slice_bytes = 1; (slice_bytes < devc->bytes_avail)
+            && (devc->buffer[slice_bytes - 1] >= 0x80); slice_bytes++);
+       if (slice_bytes != devc->bytes_avail) {
+               cbyte = devc->buffer[slice_bytes - 1];
+               slice_bytes--;  //Don't process the ending character
+               if (cbyte == '$') {
+                       sr_info
+                           ("Data stream stops with cbyte %d char %c rdidx %d sbytes %d cnt %llu",
+                            cbyte, cbyte, devc->ser_rdptr, slice_bytes,
+                            devc->byte_cnt);
+                       devc->rxstate = RX_STOPPED;
+               } else {
+                       sr_err
+                           ("Data stream aborts with cbyte %d char %c rdidx %d sbytes %d cnt %llu",
+                            cbyte, cbyte, devc->ser_rdptr, slice_bytes,
+                            devc->byte_cnt);
+                       devc->rxstate = RX_ABORT;
+               }
+       }
+       //If the wrptr is non-zero due to a residual from the previous serial transfer don't double count it towards byte_cnt
+       devc->byte_cnt += slice_bytes - (devc->wrptr);
+       sr_spew("process slice avail %d rdptr %d sb %d byte_cnt %lld",
+               devc->bytes_avail, devc->ser_rdptr, slice_bytes,
+               devc->byte_cnt);
+       //Must have a full slice
+       while ((devc->ser_rdptr + devc->bytes_per_slice) <= slice_bytes) {
+               //The use of devc->cbuf_wrptr is different between analog and digital.
+               //For analog it targets a float sized offset for that channel's buffer 
+               //For digital it targets a bit, so the 3 lsbs are bit offsets within a byte
+               slices_avail++;
+               cword = 0;
+               //build up a word 7 bits at a time, using only enabled channels
+               for (i = 0; i < devc->num_d_channels; i += 7) {
+                       if (((devc->d_chan_mask) >> i) & 0x7F) {
+                               cword |=
+                                   ((devc->buffer[devc->ser_rdptr]) &
+                                    0x7F) << i;
+                               (devc->ser_rdptr)++;
+                       }
+               }
+               //and then distribute 8 bits at a time to all possible channels
+               for (i = 0; i < devc->num_d_channels; i += 8) {
+                       uint32_t idx =
+                           ((devc->cbuf_wrptr) * devc->dig_sample_bytes) +
+                           (i >> 3);
+                       devc->d_data_buf[idx] = cword & 0xFF;
+                       sr_spew
+                           ("Dchan i %d wrptr %d idx %d char 0x%X cword 0x%X",
+                            i, devc->cbuf_wrptr, idx,
+                            devc->d_data_buf[idx], cword);
+                       cword >>= 8;
+               }
+               //Each analog value is a 7 bit value
+               for (i = 0; i < devc->num_a_channels; i++) {
+                       if ((devc->a_chan_mask >> i) & 1) {
+                               //a_size is depracted and must always be 1B
+                               tmp32 =
+                                   devc->buffer[devc->ser_rdptr] - 0x80;
+                               devc->a_data_bufs[i][devc->cbuf_wrptr] =
+                                   ((float) tmp32 * devc->a_scale[i]) +
+                                   devc->a_offset[i];
+                               devc->a_last[i] =
+                                   devc->a_data_bufs[i][devc->cbuf_wrptr];
+                               sr_spew
+                                   ("AChan %d t32 %d value %f wrptr %d rdptr %d sc %f off %f",
+                                    i, tmp32,
+                                    devc->
+                                    a_data_bufs[i][devc->cbuf_wrptr],
+                                    devc->cbuf_wrptr, devc->ser_rdptr,
+                                    devc->a_scale[i], devc->a_offset[i]);
+                               devc->ser_rdptr++;
+                       }       //if channel enabled
+               }               //for num_a_channels
+               devc->cbuf_wrptr++;
+       }                       //While another slice available
+       if (slices_avail) {
+               process_group(sdi, devc, slices_avail);
+       }
+
+}
+
+//Send the processed analog values to the session
+int send_analog(struct sr_dev_inst *sdi, struct dev_context *devc,
+               uint32_t num_samples, uint32_t offset)
+{
+       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 sr_channel *ch;
+       uint32_t i;
+       float *fptr;
 
-       (void)fd;
+       sr_analog_init(&analog, &encoding, &meaning, &spec, ANALOG_DIGITS);
+       for (i = 0; i < devc->num_a_channels; i++) {
+               if ((devc->a_chan_mask >> i) & 1) {
+                       ch = devc->analog_groups[i]->channels->data;
+                       analog.meaning->channels =
+                           g_slist_append(NULL, ch);
+                       analog.num_samples = num_samples;
+                       analog.data = (devc->a_data_bufs[i]) + offset;
+                       fptr = analog.data;
+                       sr_spew
+                           ("send analog num %d offset %d first %f 2 %f",
+                            num_samples, offset, *(devc->a_data_bufs[i]),
+                            *fptr);
+                       analog.meaning->mq = SR_MQ_VOLTAGE;
+                       analog.meaning->unit = SR_UNIT_VOLT;
+                       analog.meaning->mqflags = 0;
+                       packet.type = SR_DF_ANALOG;
+                       packet.payload = &analog;
+                       sr_session_send(sdi, &packet);
+                       g_slist_free(analog.meaning->channels);
+               }               //if enabled
+       }                       //for channels
+       return 0;
+
+}
+
+//Send the ring buffer of pre-trigger analog samples.
+//  The entire buffer is sent (as long as it filled once), but need send two payloads split at the 
+//  the writeptr 
+int send_analog_ring(struct sr_dev_inst *sdi, struct dev_context *devc,
+                    uint32_t num_samples)
+{
+       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 sr_channel *ch;
+       int i;
+       uint32_t num_pre, start_pre;
+       uint32_t num_post, start_post;
+       num_pre =
+           (num_samples >=
+            devc->pretrig_wr_ptr) ? devc->pretrig_wr_ptr : num_samples;
+       start_pre = devc->pretrig_wr_ptr - num_pre;
+       num_post = num_samples - num_pre;
+       start_post = devc->pretrig_entries - num_post;
+       sr_spew
+           ("send_analog ring wrptr %u ns %d npre %u spre %u npost %u spost %u",
+            devc->pretrig_wr_ptr, num_samples, num_pre, start_pre,
+            num_post, start_post);
+       float *fptr;
+       sr_analog_init(&analog, &encoding, &meaning, &spec, ANALOG_DIGITS);
+       for (i = 0; i < devc->num_a_channels; i++) {
+               if ((devc->a_chan_mask >> i) & 1) {
+                       ch = devc->analog_groups[i]->channels->data;
+                       analog.meaning->channels =
+                           g_slist_append(NULL, ch);
+                       analog.meaning->mq = SR_MQ_VOLTAGE;
+                       analog.meaning->unit = SR_UNIT_VOLT;
+                       analog.meaning->mqflags = 0;
+                       packet.type = SR_DF_ANALOG;
+                       packet.payload = &analog;
+                       //First send what is after the write pointer because it is oldest
+                       if (num_post) {
+                               analog.num_samples = num_post;
+                               analog.data =
+                                   (devc->a_pretrig_bufs[i]) + start_post;
+                               //sr_spew("ring buf %d starts at %p",i,(void *) devc->a_pretrig_bufs[i]);
+                               //sr_spew("analog data %d starts at %p",i,(void *) analog.data);
+                               //sr_spew("Sending A%d ring buffer oldest ",i);
+                               for (uint32_t j = 0;
+                                    j < analog.num_samples; j++) {
+                                       fptr =
+                                           analog.data +
+                                           (j * sizeof(float));
+                                       //sr_spew("RNGDCT%d j %d %f %p",i,j,*fptr,(void *)fptr);
+                               }
+                               sr_session_send(sdi, &packet);
+                       }
+                       if (num_pre) {
+                               analog.num_samples = num_pre;
+                               analog.data =
+                                   (devc->a_pretrig_bufs[i]) + start_pre;
+                               sr_dbg("Sending A%d ring buffer newest ",
+                                      i);
+                               for (uint32_t j = 0;
+                                    j < analog.num_samples; j++) {
+                                       fptr =
+                                           analog.data +
+                                           (j * sizeof(float));
+                                       sr_spew("RNGDCW%d j %d %f %p", i,
+                                               j, *fptr, (void *) fptr);
+                               }
+                               sr_session_send(sdi, &packet);
+                       }
+                       g_slist_free(analog.meaning->channels);
+                       sr_dbg("Sending A%d ring buffer done ", i);
+               }               //if enabled
+       }                       //for channels
+       return 0;
+
+}
+
+//Given a chunk of slices forward to trigger check or session as appropriate and update state
+//these could be real slices or those generated by rles
+int process_group(struct sr_dev_inst *sdi, struct dev_context *devc,
+                 uint32_t num_slices)
+{
+       int trigger_offset;
+       int pre_trigger_samples;
+       //These are samples sent to session and are less than num_slices if we reach limit_samples
+       size_t num_samples;
+       struct sr_datafeed_logic logic;
+       struct sr_datafeed_packet packet;
+       int i;
+       size_t cbuf_wrptr_cpy;
+       cbuf_wrptr_cpy = devc->cbuf_wrptr;
+       //regardless of whether we forward samples on or not (because we aren't triggered), always reset the 
+       //pointer into the device data buffers 
+       devc->cbuf_wrptr = 0;
+       if (devc->trigger_fired) {      //send directly to session
+               if (devc->limit_samples &&
+                   num_slices >
+                   devc->limit_samples - devc->sent_samples) {
+                       num_samples =
+                           devc->limit_samples - devc->sent_samples;
+               } else {
+                       num_samples = num_slices;
+               }
+               if (num_samples > 0) {
+                       sr_spew
+                           ("Process_group sending %d post trig samples dsb %d",
+                            num_samples, devc->dig_sample_bytes);
+                       //for(int z=0;(z<num_samples);z+=2){
+                       //  sr_spew("0x%X ",devc->d_data_buf[z]);
+                       //}
+                       if (devc->num_d_channels) {
+                               packet.type = SR_DF_LOGIC;
+                               packet.payload = &logic;
+                               //Size the number of bytes required to fit all of the channels
+                               logic.unitsize = devc->dig_sample_bytes;
+                               //The total length of the array sent
+                               logic.length =
+                                   num_samples * logic.unitsize;
+                               logic.data = devc->d_data_buf;
+                               sr_session_send(sdi, &packet);
+                       }
+                       send_analog(sdi, devc, num_samples, 0);
+               }               //num_sample>0
+               devc->sent_samples += num_samples;
+               return 0;
+       }                       //trigger_fired
+       else {
+               size_t num_ring_samples;
+               size_t sptr;
+               size_t eptr;
+               size_t numtail;
+               size_t numwrap;
+               size_t srcptr;
+               //sr_spew("Process_group check %d pre trig samples",num_slices);
+               //The trigger_offset is -1 if no trigger is found, but if a trigger is found
+               //then trigger_offset is the offset into the data buffer sent to it.
+               //The pre_trigger_samples is the total number of samples before the trigger, but limited to
+               //the size of the ring buffer set by the capture_ratio. So the pre_trigger_samples can include both the new samples
+               //and the ring buffer, but trigger_offset is only in relation to the new samples
+               trigger_offset = soft_trigger_logic_check(devc->stl,
+                                                         devc->d_data_buf,
+                                                         num_slices *
+                                                         devc->dig_sample_bytes,
+                                                         &pre_trigger_samples);
+               //A trigger offset >=0 indicate a trigger was seen.  The stl will isue the trigger to the session
+               //and will forward all pre trigger logic samples, but we must send any post trigger logic 
+               //and all pre and post trigger analog signals
+               // sr_dbg("trggr_off %d",trigger_offset);
+               // sr_dbg("pre_samp  %d",pre_trigger_samples);
+               if (trigger_offset > -1) {
+                       devc->trigger_fired = TRUE;
+                       devc->sent_samples += pre_trigger_samples;
+                       packet.type = SR_DF_LOGIC;
+                       packet.payload = &logic;
+                       num_samples = num_slices - trigger_offset;
+//Since we are in continuous mode for SW triggers it is possible to get more samples than limit_samples, so
+//once the trigger fires make sure we don't get beyond limit samples. At this point sent_samples should
+//be equal to pre_trigger_samples (just added above) because without being triggered we'd never increment
+//sent_samples.
+//This number is the number of post trigger logic samples to send to the session, the number of floats
+//is larger because of the analog ring buffer we track.
+                       if (devc->limit_samples &&
+                           num_samples >
+                           devc->limit_samples - devc->sent_samples)
+                               num_samples =
+                                   devc->limit_samples -
+                                   devc->sent_samples;
+                       //The soft trigger logic issues the trigger and sends packest for all logic data that was pretrigger
+                       //so only send what is left
+                       if (num_samples > 0) {
+                               sr_dbg
+                                   ("Sending post trigger logical remainder of %d",
+                                    num_samples);
+                               logic.length =
+                                   num_samples * devc->dig_sample_bytes;
+                               logic.unitsize = devc->dig_sample_bytes;
+                               logic.data =
+                                   devc->d_data_buf +
+                                   (trigger_offset *
+                                    devc->dig_sample_bytes);
+                               devc->sent_samples += num_samples;
+                               sr_session_send(sdi, &packet);
+                       }
+                       size_t new_start, new_end, new_samples,
+                           ring_samples;
+                       //Figure out the analog data to send.
+                       //We might need to send:
+                       //-some or all of incoming data
+                       //-all of incoming data and some of ring buffer
+                       //-all of incoming data and all of ring buffer (and still might be short)
+                       //We don't need to compare to limit_samples because pretrig_entries can never be more than limit_samples
+                       //trigger offset indicatese where in the new samples the trigger was, but we need to go back pretrig_entries before it             
+                       new_start =
+                           (trigger_offset >
+                            (int) devc->pretrig_entries) ? trigger_offset
+                           - devc->pretrig_entries : 0;
+                       //Note that we might not have gotten all the pre triggerstore data we were looking for. In such a case the sw trigger
+                       //logic seems to fill up to the limit_samples and thus the ratio is off, but we get the full number of samples
+                       //The number of entries in the ring buffer is pre_trigger_samples-trigger_offset so subtract that from limit samples
+                       //as a threshold
+                       new_end =
+                           MIN(num_slices - 1,
+                               devc->limit_samples -
+                               (pre_trigger_samples - trigger_offset) -
+                               1);
+                       //This includes pre and post trigger storage.
+                       new_samples = new_end - new_start + 1;
+                       //pre_trigger_samples can never be greater than trigger_offset by more than the ring buffer depth (pretrig entries) 
+                       ring_samples =
+                           (pre_trigger_samples >
+                            trigger_offset) ? pre_trigger_samples -
+                           trigger_offset : 0;
+                       sr_spew
+                           ("SW trigger float info newstart %zu new_end %zu new_samp %zu ring_samp %zu",
+                            new_start, new_end, new_samples,
+                            ring_samples);
+                       if (ring_samples > 0) {
+                               send_analog_ring(sdi, devc, ring_samples);
+                       }
+                       if (new_samples) {
+                               send_analog(sdi, devc, new_samples,
+                                           new_start);
+                       }
+
+               }               //if trigger_offset 
+               else {          //We didn't trigger but need to copy to ring buffer
+                       if ((devc->a_chan_mask) && (devc->pretrig_entries)) {
+                               //The incoming data buffer could be much larger than the ring buffer, so never copy more than 
+                               //the size of the ring buffer
+                               num_ring_samples =
+                                   num_slices >
+                                   devc->
+                                   pretrig_entries ? devc->pretrig_entries
+                                   : num_slices;
+                               sptr = devc->pretrig_wr_ptr;    //starting pointer to copy to
+                               //endptr can't go past the end
+                               eptr =
+                                   (sptr + num_ring_samples) >=
+                                   devc->
+                                   pretrig_entries ? devc->pretrig_entries
+                                   - 1 : sptr + num_ring_samples - 1;
+                               numtail = (eptr - sptr) + 1;    //number of samples to copy to the tail of ring buffer without wrapping
+                               numwrap =
+                                   (num_ring_samples >
+                                    numtail) ? num_ring_samples -
+                                   numtail : 0;
+                               //cbuf_wrptr points to where the next write should go, not  theactual write data
+                               srcptr = cbuf_wrptr_cpy - num_ring_samples;
+                               sr_spew("RNG num %zu sptr %zu eptr %zu ",
+                                       num_ring_samples, sptr, eptr);
+                               //sr_spew("RNG srcptr %zu nt %zu nw %zu",srcptr,numtail,numwrap);
+                               for (i = 0; i < devc->num_a_channels; i++) {
+                                       if ((devc->a_chan_mask >> i) & 1) {
+                                               //copy tail
+                                               for (uint32_t j = 0;
+                                                    j < numtail; j++) {
+                                                       devc->a_pretrig_bufs
+                                                           [i][sptr + j] =
+                                                           devc->a_data_bufs
+                                                           [i]
+                                                           [srcptr + j];
+                                                       //sr_spew("RNGCpyT C%d src %zu dest %zu",i,srcptr+j,sptr+j);
+                                               }       //for j
+                                       }       //if chan_mask
+                               }       //for channels
+                               //Copy wrap
+                               srcptr += numtail;
+                               for (i = 0; i < devc->num_a_channels; i++) {
+                                       if ((devc->a_chan_mask >> i) & 1) {
+                                               for (uint32_t j = 0;
+                                                    j < numwrap; j++) {
+                                                       devc->a_pretrig_bufs
+                                                           [i][j] =
+                                                           devc->a_data_bufs
+                                                           [i]
+                                                           [srcptr + j];
+                                                       //sr_spew("RNGCpyW C%d src %zu dest %zu",i,srcptr+j,j);
+                                               }       //for j
+                                       }       //if chan_mask
+                               }       //for channels
+                               devc->pretrig_wr_ptr =
+                                   (numwrap) ? numwrap : (eptr +
+                                                          1) %
+                                   devc->pretrig_entries;
+                               //sr_dbg("RNG pwrptr new %u",devc->pretrig_wr_ptr);
+                       }       //if any analog channel enabled and pretrig_entries
+               }               //else (trigger not detected)
+       }                       //trigger not set on function entry
+       return 0;
+}                              //process_group
+
+
+//Duplicate previous sample values
+//This function relies on the caller to ensure d_data_buf has samples to handle the full value of the rle
+void rle_memset(struct dev_context *devc, uint32_t num_slices)
+{
+       uint32_t j, k;
+       sr_spew("rle_memset val 0x%X,slices %d dsb %d\n", devc->d_last[0],
+               num_slices, devc->dig_sample_bytes);
+       //Even if a channel is disabled, PV expects the same location and size for the enabled
+       // channels as if the channel were enabled.
+       for (j = 0; j < num_slices; j++) {
+               for (k = 0; k < devc->dig_sample_bytes; k++) {
+                       devc->d_data_buf[devc->cbuf_wrptr++] =
+                           devc->d_last[k];
+                       //sr_spew("k %d j %d v 0x%X",k,j,devc->d_data_buf[(devc->cbuf_wrptr)-1]);
+               }
+       }
+}
+
+//This callback function is mapped from api.c with serial_source_add and is created after a capture
+//has been setup and is responsible for querying the device trigger status, downloading data
+//and forwarding packets
+SR_PRIV int raspberrypi_pico_receive(int fd, int revents, void *cb_data)
+{
+       struct sr_dev_inst *sdi;
+       struct dev_context *devc;
+       struct sr_serial_dev_inst *serial;
+       uint32_t i;
+       int len;
+       uint32_t bytes_rem;
+       uint32_t residual_bytes;
+       (void) fd;
 
        if (!(sdi = cb_data))
                return TRUE;
 
        if (!(devc = sdi->priv))
                return TRUE;
+       if (devc->rxstate != RX_ACTIVE) {
+               //This condition is normal operation and expected to happen 
+               //but printed as information
+               sr_dbg("Reached non active state in receive %d",
+                      devc->rxstate);
+               //don't return - we may be waiting for a final bytecnt
+               //return TRUE;
+       }
+       if (devc->rxstate == RX_IDLE) {
+               //This is the normal end condition where we do one more receive
+               //to make sure we get the full byte_cnt
+               sr_dbg("Reached idle state in receive %d", devc->rxstate);
+               return FALSE;
+       }
+
+       serial = sdi->conn;
+       //return true if it is some kind of event we don't handle
+       if (!(revents == G_IO_IN || revents == 0))
+               return TRUE;
+       //Fill the buffer, note the end may have partial slices
+       bytes_rem = devc->serial_buffer_size - devc->wrptr;
+       //Read one byte less so that we can null it and print as a string
+       //Do a small 10ms timeout, if we get nothing, we'll always come back again
+       len =
+           serial_read_blocking(serial, &(devc->buffer[devc->wrptr]),
+                                bytes_rem - 1, 10);
+       sr_spew("Entry wrptr %u bytes_rem %u len %d", devc->wrptr,
+               bytes_rem, len);
+
+       if (len > 0) {
+               devc->buffer[devc->wrptr + len] = 0;
+               //Add the "#" so that spaces are clearly seen
+               sr_dbg("rx string %s#", devc->buffer);
+               //This is not guaranteed to be a dataloss condition, but definitely indicates we are 
+               //processing data right at the incoming rate.
+               //With the addition of the byte_cnt sent at the end we will detect any dataloss conditions
+               //and thus this is disabled
+               //if(len>=(int)bytes_rem-8){
+               //  sr_err("ERROR: Serial buffer near or at max depth, data from device may have been lost");
+               //}
+               devc->bytes_avail = (devc->wrptr + len);
+               sr_spew
+                   ("rx len %d bytes_avail %ul sent_samples %ul wrptr %u",
+                    len, devc->bytes_avail, devc->sent_samples,
+                    devc->wrptr);
+               //sr_err("rx len %d ",len);
+       } else if (len == 0) {
+               return TRUE;
+       } else {
+               sr_err("ERROR:Negative serial read code %d", len);
+               sdi->driver->dev_acquisition_stop(sdi);
+               return FALSE;
+       }                       //len>0
+       //This can be used as a bit bucket to drop all samples to see how host processing time effects
+       //the devices ability to send data. Obviously no data will be forwarded to the session so it will hang
+       //        return TRUE; 
 
-       if (revents == G_IO_IN) {
-               /* TODO */
+       //Process the serial read data
+       devc->ser_rdptr = 0;
+       if (devc->rxstate == RX_ACTIVE) {
+               if ((devc->a_chan_mask == 0)
+                   && ((devc->d_chan_mask & 0xFFFFFFF0) == 0)) {
+                       process_D4(sdi, devc);
+               } else {
+                       process_slice(sdi, devc);
+               }
        }
+       //process_slice/process_D4 increment ser_rdptr as bytes of the serial buffer are used
+       //But they may not use all of it, and thus the residual unused bytes are shifted to the start of the buffer
+       //for the next call.
+       residual_bytes = devc->bytes_avail - devc->ser_rdptr;
+       //sr_spew("Residuals resid %d avail %d rdptr %d wrptr %d\n",residual_bytes,devc->bytes_avail,devc->ser_rdptr,devc->wrptr);
+       if (residual_bytes) {
+               for (i = 0; i < residual_bytes; i++) {
+                       devc->buffer[i] =
+                           devc->buffer[i + devc->ser_rdptr];
+               }
+               devc->ser_rdptr = 0;
+               devc->wrptr = residual_bytes;
+               sr_spew("Residual shift rdptr %u wrptr %u",
+                       devc->ser_rdptr, devc->wrptr);
+       } else {
+               //If there are no residuals shifted then zero the wrptr since all data is used
+               devc->wrptr = 0;
+       }
+       //ABORT ends immediately
+       if (devc->rxstate == RX_ABORT) {
+               sr_err("Ending receive on abort");
+               sdi->driver->dev_acquisition_stop(sdi);
+               return FALSE;   //
+       }
+       //if stopped look for final '+' indicating the full byte_cnt is received
+       if (devc->rxstate == RX_STOPPED) {
+               sr_dbg("Stopped, checking byte_cnt");
+               if (devc->buffer[0] != '$') {
+                       //If this happens it means that we got a set of data that was not processed as
+                       //whole groups of slice bytes. So either we lost data or are not parsing it correctly.
+                       sr_err("ERROR: Stop marker should be byte zero");
+                       devc->rxstate = RX_ABORT;
+                       sdi->driver->dev_acquisition_stop(sdi);
+                       return FALSE;
+               }
+               for (i = 1; i < devc->wrptr; i++) {
+                       if (devc->buffer[i] == '+') {
+                               devc->buffer[i] = 0;
+                               uint64_t rxbytecnt;
+                               rxbytecnt = atol((char *)&(devc->buffer[1]));
+                               sr_dbg
+                                   ("Byte_cnt check device cnt %llu host cnt %llu",
+                                    rxbytecnt, devc->byte_cnt);
+                               if (rxbytecnt != devc->byte_cnt) {
+                                       sr_err
+                                           ("ERROR: received %llu and counted %llu bytecnts don't match, data may be lost",
+                                            rxbytecnt, devc->byte_cnt);
+                               }
+                               //Since we got the bytecnt we know the device is done sending data
+                               devc->rxstate = RX_IDLE;
+                               //We must always call acquisition_stop on all completed runs
+                               sdi->driver->dev_acquisition_stop(sdi);
+                               return TRUE;
+                       }
+               }
+               //It's possible we need one more serial transfer to get the byte_cnt, so print that here
+               sr_dbg("Haven't seen byte_cnt + yet");
+       }                       //RX_STOPPED
+       //If at the sample limit, send a "+" in case we are in continuous mode and need
+       //to stop the device.  Not that even in non continous mode there might be cases where get an extra
+       //sample or two...
 
+       if ((devc->sent_samples >= devc->limit_samples)
+           && (devc->rxstate == RX_ACTIVE)) {
+               sr_dbg
+                   ("Ending: sent %u of limit %llu samples byte_cnt %llu",
+                    devc->sent_samples, devc->limit_samples,
+                    devc->byte_cnt);
+               send_serial_char(serial, '+');
+
+       }
+       sr_spew
+           ("Receive function done: sent %u limit %llu wrptr %u len %d",
+            devc->sent_samples, devc->limit_samples, devc->wrptr, len);
        return TRUE;
+}                              //raspberrypi_pico_receive
+
+//Read device specific information from the device
+SR_PRIV int raspberrypi_pico_get_dev_cfg(const struct sr_dev_inst *sdi)
+{
+       struct dev_context *devc;
+       struct sr_serial_dev_inst *serial;
+       char *cmd, response[20];
+       gchar **tokens;
+       unsigned int i;
+       int ret, num_tokens;
+
+       devc = sdi->priv;
+       sr_dbg("At get_dev_cfg");
+       serial = sdi->conn;
+       for (i = 0; i < devc->num_a_channels; i++) {
+               cmd = g_strdup_printf("a%d\n", i);
+               ret = send_serial_w_resp(serial, cmd, response, 20);
+               if (ret <= 0) {
+                       sr_err
+                           ("ERROR:No response from device for analog channel query");
+                       return SR_ERR;
+               }
+               //null end of string for strsplit
+               response[ret] = 0;
+               tokens = NULL;
+               tokens = g_strsplit(response, "x", 0);
+               num_tokens = g_strv_length(tokens);
+               if (num_tokens == 2) {
+                       devc->a_scale[i] =
+                           ((float) atoi(tokens[0])) / 1000000.0;
+                       devc->a_offset[i] =
+                           ((float) atoi(tokens[1])) / 1000000.0;
+                       sr_dbg
+                           ("A%d scale %f offset %f response #%s# tokens #%s# #%s#\n",
+                            i, devc->a_scale[i], devc->a_offset[i],
+                            response, tokens[0], tokens[1]);
+               } else {
+                       sr_err
+                           ("ERROR:Ascale read c%d got unparseable response %s tokens %d",
+                            i, response, num_tokens);
+                       //force a legal fixed value assuming a 3.3V scale
+                       //a failue in parsing the scale
+                       devc->a_scale[i] = 0.0257;
+                       devc->a_offset[i] = 0.0;
+               }
+               g_strfreev(tokens);
+               g_free(cmd);
+       }
+
+
+       return SR_OK;
+
 }